Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit cc0a054

Browse files
committed
fix warnings in clang about long long formats
1 parent e62c700 commit cc0a054

File tree

4 files changed

+23
-23
lines changed

4 files changed

+23
-23
lines changed

Makefile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,15 +3,15 @@ OBJS = multimaster.o raftable.o arbiter.o bytebuf.o bgwpool.o pglogical_output.o
33

44
override CPPFLAGS += -I../raftable
55

6-
SCRIPTS_built = tests/dtmbench
6+
#SCRIPTS_built = tests/dtmbench
77
EXTRA_INSTALL = contrib/raftable contrib/mmts
88

99
EXTENSION = multimaster
1010
DATA = multimaster--1.0.sql
1111

1212
.PHONY: all
1313

14-
all: multimaster.so tests/dtmbench
14+
all: multimaster.so #tests/dtmbench
1515

1616
tests/dtmbench:
1717
make -C tests

arbiter.c

Lines changed: 13 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -105,17 +105,18 @@ static int gateway;
105105
static void MtmTransSender(Datum arg);
106106
static void MtmTransReceiver(Datum arg);
107107

108-
static char const* const messageText[] =
109-
{
110-
"INVALID",
111-
"HANDSHAKE",
112-
"READY",
113-
"PREPARE",
114-
"PREPARED",
115-
"ABORTED",
116-
"STATUS"
117-
};
118-
108+
/*
109+
* static char const* const messageText[] =
110+
* {
111+
* "INVALID",
112+
* "HANDSHAKE",
113+
* "READY",
114+
* "PREPARE",
115+
* "PREPARED",
116+
* "ABORTED",
117+
* "STATUS"
118+
*};
119+
*/
119120

120121
static BackgroundWorker MtmSender = {
121122
"mtm-sender",
@@ -715,7 +716,7 @@ static void MtmTransReceiver(Datum arg)
715716
/* Coordinator's disabled mask is wider than of this node: so reject such transaction to avoid
716717
commit on smaller subset of nodes */
717718
elog(WARNING, "Coordinator of distributed transaction see less nodes than node %d: %lx instead of %lx",
718-
msg->node, Mtm->disabledNodeMask, msg->disabledNodeMask);
719+
msg->node, (long) Mtm->disabledNodeMask, (long) msg->disabledNodeMask);
719720
MtmAbortTransaction(ts);
720721
}
721722

multimaster.c

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1137,7 +1137,7 @@ MtmCheckClusterLock()
11371137
} else {
11381138
/* All lockers are synchronized their logs */
11391139
/* Remove lock and mark them as receovered */
1140-
elog(WARNING, "Complete recovery of %d nodes (node mask %lx)", Mtm->nLockers, Mtm->nodeLockerMask);
1140+
elog(WARNING, "Complete recovery of %d nodes (node mask %lx)", Mtm->nLockers, (long) Mtm->nodeLockerMask);
11411141
Assert(Mtm->walSenderLockerMask == 0);
11421142
Assert((Mtm->nodeLockerMask & Mtm->disabledNodeMask) == Mtm->nodeLockerMask);
11431143
Mtm->disabledNodeMask &= ~Mtm->nodeLockerMask;
@@ -1197,7 +1197,7 @@ bool MtmRefreshClusterStatus(bool nowait)
11971197

11981198
clique = MtmFindMaxClique(matrix, MtmNodes, &clique_size);
11991199
if (clique_size >= MtmNodes/2+1) { /* have quorum */
1200-
elog(WARNING, "Find clique %lx, disabledNodeMask %lx", clique, Mtm->disabledNodeMask);
1200+
elog(WARNING, "Find clique %lx, disabledNodeMask %lx", (long) clique, (long) Mtm->disabledNodeMask);
12011201
MtmLock(LW_EXCLUSIVE);
12021202
mask = ~clique & (((nodemask_t)1 << MtmNodes)-1) & ~Mtm->disabledNodeMask; /* new disabled nodes mask */
12031203
for (i = 0; mask != 0; i++, mask >>= 1) {
@@ -1225,7 +1225,7 @@ bool MtmRefreshClusterStatus(bool nowait)
12251225
MtmSwitchClusterMode(MTM_RECOVERY);
12261226
}
12271227
} else {
1228-
elog(WARNING, "Clique %lx has no quorum", clique);
1228+
elog(WARNING, "Clique %lx has no quorum", (long) clique);
12291229
MtmSwitchClusterMode(MTM_IN_MINORITY);
12301230
}
12311231
return true;
@@ -1235,12 +1235,12 @@ void MtmCheckQuorum(void)
12351235
{
12361236
if (Mtm->nNodes < MtmNodes/2+1) {
12371237
if (Mtm->status == MTM_ONLINE) { /* out of quorum */
1238-
elog(WARNING, "Node is in minority: disabled mask %lx", Mtm->disabledNodeMask);
1238+
elog(WARNING, "Node is in minority: disabled mask %lx", (long) Mtm->disabledNodeMask);
12391239
MtmSwitchClusterMode(MTM_IN_MINORITY);
12401240
}
12411241
} else {
12421242
if (Mtm->status == MTM_IN_MINORITY) {
1243-
elog(WARNING, "Node is in majority: dissbled mask %lx", Mtm->disabledNodeMask);
1243+
elog(WARNING, "Node is in majority: dissbled mask %lx", (long) Mtm->disabledNodeMask);
12441244
MtmSwitchClusterMode(MTM_ONLINE);
12451245
}
12461246
}
@@ -2229,7 +2229,7 @@ MtmNoticeReceiver(void *i, const PGresult *res)
22292229
/* Strip "NOTICE: " from beginning and "\n" from end of error string */
22302230
strncpy(stripped_notice, notice + 9, len - 1 - 9);
22312231

2232-
elog(NOTICE, stripped_notice);
2232+
elog(NOTICE, "%s", stripped_notice);
22332233
pfree(stripped_notice);
22342234
}
22352235

@@ -2273,7 +2273,7 @@ static void MtmBroadcastUtilityStmt(char const* sql, bool ignoreError)
22732273
if (MtmGUCBufferAllocated && !MtmRunUtilityStmt(conns[i], MtmGUCBuffer->data, &utility_errmsg) && !ignoreError)
22742274
{
22752275
errorMsg = "Failed to set GUC variables at node %d";
2276-
elog(ERROR, utility_errmsg);
2276+
elog(NOTICE, "%s", utility_errmsg);
22772277
failedNode = i;
22782278
break;
22792279
}
@@ -2289,7 +2289,7 @@ static void MtmBroadcastUtilityStmt(char const* sql, bool ignoreError)
22892289
errorMsg = utility_errmsg;
22902290
else
22912291
{
2292-
elog(ERROR, utility_errmsg);
2292+
elog(ERROR, "%s", utility_errmsg);
22932293
errorMsg = "Failed to run command at node %d";
22942294
}
22952295

pglogical_output.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -493,7 +493,6 @@ pg_decode_origin_filter(LogicalDecodingContext *ctx,
493493
}
494494

495495
if (!data->forward_changesets && origin_id != InvalidRepOriginId) {
496-
*(int*)0 = 0;
497496
return true;
498497
}
499498

0 commit comments

Comments
 (0)