[ https://issues.apache.org/jira/browse/ZOOKEEPER-3509?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16948647#comment-16948647 ]
Hudson commented on ZOOKEEPER-3509: ----------------------------------- SUCCESS: Integrated in Jenkins build ZooKeeper-trunk #732 (See [https://builds.apache.org/job/ZooKeeper-trunk/732/]) ZOOKEEPER-3509: Revisit log format (andor: rev 11c07921c15e2fb7692375327b53f26a583b77ca) * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/ZooKeeper.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/JUnit4ZKTestRunner.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/SyncRequestProcessor.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/SaslServerPrincipal.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/SaslClientCallbackHandler.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/ZkDatabaseCorruptionTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/LearnerSessionTracker.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/QuorumCnxManager.java * (edit) zookeeper-recipes/zookeeper-recipes-lock/src/main/java/org/apache/zookeeper/recipes/lock/ProtocolSupport.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/SessionUpgradeQuorumTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/FourLetterWordsWhiteListTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/auth/SaslQuorumAuthServer.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/LeaderSessionTracker.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/LocalSessionsOnlyTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/persistence/FileSnap.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/ClientTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/ClientCnxnSocketNIO.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/util/RequestPathMetricsCollector.java * (edit) zookeeper-recipes/zookeeper-recipes-queue/src/main/java/org/apache/zookeeper/recipes/queue/DistributedQueue.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/UpgradeableSessionTracker.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/util/MessageTracker.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/SessionTrackerImpl.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/FollowerResyncConcurrencyTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/FinalRequestProcessor.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/Follower.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/Login.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/persistence/SnapStream.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/BlueThrottle.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/NIOConnectionFactoryFdLeakTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/QuorumTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/ClientCnxnSocket.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/ZooKeeperServer.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/NettyServerCnxnFactory.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/AsyncHammerTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/AsyncOpsTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/FourLetterWordsTest.java * (edit) zookeeper-recipes/zookeeper-recipes-lock/src/main/java/org/apache/zookeeper/recipes/lock/WriteLock.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/auth/SaslQuorumAuthLearner.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/jmx/MBeanRegistry.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/ClientHammerTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/common/SSLContextAndOptions.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/FLEZeroWeightTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/Learner.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/DuplicateLocalSessionUpgradeTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/admin/JettyAdminServerTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/CommitProcessorConcurrencyTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/DatadirCleanupManager.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/LearnerHandler.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/NettyServerCnxn.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/LoadFromLogTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/ConnectionBean.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/FourLetterWordsQuorumTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/SaslAuthFailTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/AuthFastLeaderElection.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/util/PortForwarder.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/persistence/FileTxnLogTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/ObserverZooKeeperServer.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/JMXEnv.java * (edit) zookeeper-recipes/zookeeper-recipes-election/src/test/java/org/apache/zookeeper/recipes/leader/LeaderElectionSupportTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/SaslAuthFailDesignatedClientTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/FLERestartTest.java * (edit) zookeeper-recipes/zookeeper-recipes-election/src/main/java/org/apache/zookeeper/recipes/leader/LeaderElectionSupport.java * (edit) zookeeper-recipes/zookeeper-recipes-queue/src/test/java/org/apache/zookeeper/recipes/queue/DistributedQueueTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/ClientPortBindTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/flexible/QuorumHierarchical.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/LocalSessionRequestTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/InvalidSnapshotTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/auth/SaslServerCallbackHandler.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/ContainerManager.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/QuorumZooKeeperServer.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/KeyAuthClientTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/LoadFromLogNoServerTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/IntegrityCheck.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/SessionTest.java * (edit) zookeeper-metrics-providers/zookeeper-prometheus-metrics/src/main/java/org/apache/zookeeper/metrics/prometheus/PrometheusMetricsProvider.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/ReferenceCountedACLCache.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/StandaloneDisabledTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/RecoveryTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/common/ZKTrustManager.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/DeserializationPerfTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/ZooTrace.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/PurgeTxnTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/FastLeaderElection.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/ReadOnlyRequestProcessor.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/QuorumPeer.java * (edit) zookeeper-recipes/zookeeper-recipes-lock/src/main/java/org/apache/zookeeper/recipes/lock/ZNodeName.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/common/FileChangeWatcher.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/DataTree.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/QuorumPeerMainTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/ServerCnxnTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/EmptiedSnapshotRecoveryTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/LeaderRequestProcessor.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/FLETest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/CnxManagerTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/SyncCallTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/common/FileChangeWatcherTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/Zab1_0Test.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/QuorumUtil.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/QuorumPeerConfig.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/LearnerMaster.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/util/SecurityUtils.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/UnifiedServerSocketModeDetectionTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/PurgeTxnLog.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/CRCTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/OSMXBeanTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/PrepRequestProcessor.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/ACLTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/ZKDatabase.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/QuorumPeerMain.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/UnifiedServerSocket.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/ClientCnxnSocketNetty.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/common/X509Util.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/auth/MiniKdc.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/SerializationPerfTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/NIOServerCnxn.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/persistence/FileTxnLog.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/ChrootAsyncTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/QuorumBase.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/common/AtomicFileOutputStream.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/PortAssignment.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/LeaderSessionTrackerTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/LearnerHandlerBean.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/common/NettyUtils.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/SessionUpgradeTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/QuorumPeerTestBase.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/common/ZKHostnameVerifier.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/NIOServerCnxnFactory.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/common/ZKConfig.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/CommitProcessorTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/Observer.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/ObserverMaster.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/FuzzySnapshotRelatedTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/ClientBase.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/ClientCnxn.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/Leader.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/FollowerRequestProcessor.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/admin/JettyAdminServer.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/auth/QuorumAuthUpgradeTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/ChrootClientTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/ZooKeeperServerStartupTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/LearnerMetricsTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/ObserverMasterTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/ZooKeeperMain.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/persistence/FilePadding.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/RemoveWatchesTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/watch/WatcherCleaner.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/HierarchicalQuorumTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/SaslAuthMissingClientConfigTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/BufferSizeTest.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/ZKTestCase.java * (edit) zookeeper-server/src/test/java/org/apache/zookeeper/test/FLENewEpochTest.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/client/ZooKeeperSaslClient.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/auth/ProviderRegistry.java * (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/CommitProcessor.java > Revisit log format > ------------------ > > Key: ZOOKEEPER-3509 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-3509 > Project: ZooKeeper > Issue Type: Improvement > Components: server > Reporter: Zili Chen > Assignee: Zili Chen > Priority: Major > Labels: pull-request-available > Fix For: 3.6.0 > > Time Spent: 4h > Remaining Estimate: 0h > > Currently ZooKeeper mixes up different log format and even a number of log > statements are buggy. It is an opportunity that we revisit log format in > ZooKeeper and do a pass to fix all log format related issues. -- This message was sent by Atlassian Jira (v8.3.4#803005)