jiajunwang commented on a change in pull request #671: Add ZooKeeperAccessor to helix-rest URL: https://github.com/apache/helix/pull/671#discussion_r366723208
########## File path: helix-rest/src/main/java/org/apache/helix/rest/server/ServerContext.java ########## @@ -47,8 +47,8 @@ private ZKHelixAdmin _zkHelixAdmin; private ClusterSetup _clusterSetup; private ConfigAccessor _configAccessor; - // The lazy initialized base data accessor that reads/writes byte array to ZK - private ZkBaseDataAccessor<byte[]> _byteArrayBaseDataAccessor; + // A lazily-initialized base data accessor that reads/writes byte array to ZK Review comment: Is it only supporting read at this version? If so, can we update the name accordingly? _byteArrayZkReadOnlyAccessor? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@helix.apache.org For additional commands, e-mail: reviews-h...@helix.apache.org