[ 
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)

Reply via email to