Skip to content

ZOOKEEPER-4993: Lower "Attempting to delete candidate container..." logging to DEBUG - #2430

Open
Pyolar wants to merge 1 commit into
apache:masterfrom
Pyolar:ZOOKEEPER-4993
Open

ZOOKEEPER-4993: Lower "Attempting to delete candidate container..." logging to DEBUG#2430
Pyolar wants to merge 1 commit into
apache:masterfrom
Pyolar:ZOOKEEPER-4993

Conversation

@Pyolar

@Pyolar Pyolar commented Aug 1, 2026

Copy link
Copy Markdown

This change lowers the candidate container deletion log from INFO to DEBUG to avoid excessive log volume while preserving the constructor configuration log at INFO.

A regression test verifies that the candidate deletion message is emitted at DEBUG level.

Jira: https://issues.apache.org/jira/browse/ZOOKEEPER-4993

Tests:

  • mvn -pl zookeeper-server -am -Dtest=CreateContainerTest -Dsurefire.failIfNoSpecifiedTests=false test
  • mvn -pl zookeeper-server -am -DskipTests compile checkstyle:check spotbugs:check
  • mvn verify spotbugs:check checkstyle:check -Pfull-build -Dsurefire-forkcount=4 reached 3,239 tests. It reported 7 unrelated OCSP/SSL failures in ClientSSLRevocationTest and QuorumSSLTest; all CreateContainerTest tests passed.

@Pyolar

Pyolar commented Aug 1, 2026

Copy link
Copy Markdown
Author

/retest

@Pyolar

Pyolar commented Aug 1, 2026

Copy link
Copy Markdown
Author

Jenkins build #1 reached compilation and failed only in two unrelated tests: ServerCnxnFactoryTest.testShedConnections_SmallPercentageRoundsToZero{FactoryType}[2] (probabilistic 1% shedding returned 1 instead of the test expected 0) and QuorumRestartTest.testRestartingLeaderMultipleTimes (30-second leader-election timeout). The PR-specific CreateContainerTest passed in that run. Focused local validation also passes on this commit: ServerCnxnFactoryTest 10/10, QuorumRestartTest 3/3, and CreateContainerTest 14/14. No source changes are needed for PR #2430; please rerun the Jenkins check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant