Skip to content

Commit a12943d

Browse files
authored
Change the new ensemble log to info level (#4566)
--- ### Motivation Change the new ensemble log to the info level. Sometimes, the ensemble may not satisfied with the placement policy. The log should be info level but not a warn level. Because it will fix by the auto recovery later, so this just a information from the ensemble choose.
1 parent 149d055 commit a12943d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

bookkeeper-server/src/main/java/org/apache/bookkeeper/client/BookieWatcherImpl.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -273,7 +273,7 @@ public List<BookieId> newEnsemble(int ensembleSize, int writeQuorumSize,
273273
if (isEnsembleAdheringToPlacementPolicy == PlacementPolicyAdherence.FAIL) {
274274
ensembleNotAdheringToPlacementPolicy.inc();
275275
if (ensembleSize > 1) {
276-
log.warn("New ensemble: {} is not adhering to Placement Policy. quarantinedBookies: {}",
276+
log.info("New ensemble: {} is not adhering to Placement Policy. quarantinedBookies: {}",
277277
socketAddresses, quarantinedBookiesSet);
278278
}
279279
}
@@ -289,7 +289,7 @@ public List<BookieId> newEnsemble(int ensembleSize, int writeQuorumSize,
289289
isEnsembleAdheringToPlacementPolicy = newEnsembleResponse.getAdheringToPolicy();
290290
if (isEnsembleAdheringToPlacementPolicy == PlacementPolicyAdherence.FAIL) {
291291
ensembleNotAdheringToPlacementPolicy.inc();
292-
log.warn("New ensemble: {} is not adhering to Placement Policy", socketAddresses);
292+
log.info("New ensemble: {} is not adhering to Placement Policy", socketAddresses);
293293
}
294294
newEnsembleTimer.registerFailedEvent(MathUtils.nowInNano() - startTime, TimeUnit.NANOSECONDS);
295295
}

0 commit comments

Comments
 (0)