From notifications-return-1295-archive-asf-public=cust-asf.ponee.io@zookeeper.apache.org Mon Aug 12 07:41:14 2019 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [207.244.88.153]) by mx-eu-01.ponee.io (Postfix) with SMTP id BA14A180637 for ; Mon, 12 Aug 2019 09:41:13 +0200 (CEST) Received: (qmail 34232 invoked by uid 500); 12 Aug 2019 07:41:12 -0000 Mailing-List: contact notifications-help@zookeeper.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@zookeeper.apache.org Delivered-To: mailing list notifications@zookeeper.apache.org Received: (qmail 34223 invoked by uid 99); 12 Aug 2019 07:41:12 -0000 Received: from ec2-52-202-80-70.compute-1.amazonaws.com (HELO gitbox.apache.org) (52.202.80.70) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 12 Aug 2019 07:41:12 +0000 From: GitBox To: notifications@zookeeper.apache.org Subject: [GitHub] [zookeeper] nkalmar commented on a change in pull request #1049: ZOOKEEPER-3475 Enable Checkstyle configuration on zookeeper-server Message-ID: <156559567261.27232.13626085139755590912.gitbox@gitbox.apache.org> Date: Mon, 12 Aug 2019 07:41:12 -0000 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit nkalmar commented on a change in pull request #1049: ZOOKEEPER-3475 Enable Checkstyle configuration on zookeeper-server URL: https://github.com/apache/zookeeper/pull/1049#discussion_r312805241 ########## File path: zookeeper-server/src/main/java/org/apache/zookeeper/server/ZooKeeperServerMXBean.java ########## @@ -25,192 +25,233 @@ /** * @return the server socket port number */ - public String getClientPort(); + String getClientPort(); + /** * @return the zookeeper server version */ - public String getVersion(); + String getVersion(); + /** * @return time the server was started */ - public String getStartTime(); + String getStartTime(); + /** * @return min request latency in ms */ - public long getMinRequestLatency(); + long getMinRequestLatency(); + /** * @return average request latency in ms */ - public double getAvgRequestLatency(); + double getAvgRequestLatency(); + /** * @return max request latency in ms */ - public long getMaxRequestLatency(); + long getMaxRequestLatency(); + /** * @return number of packets received so far */ - public long getPacketsReceived(); + long getPacketsReceived(); + /** * @return number of packets sent so far */ - public long getPacketsSent(); + long getPacketsSent(); + /** * @return number of fsync threshold exceeds so far */ - public long getFsyncThresholdExceedCount(); + long getFsyncThresholdExceedCount(); + /** * @return number of outstanding requests. */ - public long getOutstandingRequests(); + long getOutstandingRequests(); + /** - * Current TickTime of server in milliseconds + * Current TickTime of server in milliseconds. */ - public int getTickTime(); + int getTickTime(); + /** - * Set TickTime of server in milliseconds + * Set TickTime of server in milliseconds. */ - public void setTickTime(int tickTime); + void setTickTime(int tickTime); - /** Current maxClientCnxns allowed from a particular host */ - public int getMaxClientCnxnsPerHost(); + /** + * Current maxClientCnxns allowed from a particular host. + */ + int getMaxClientCnxnsPerHost(); - /** Set maxClientCnxns allowed from a particular host */ - public void setMaxClientCnxnsPerHost(int max); + /** + * Set maxClientCnxns allowed from a particular host. + */ + void setMaxClientCnxnsPerHost(int max); /** - * Current minSessionTimeout of the server in milliseconds + * Current minSessionTimeout of the server in milliseconds. */ - public int getMinSessionTimeout(); + int getMinSessionTimeout(); + /** - * Set minSessionTimeout of server in milliseconds + * Set minSessionTimeout of server in milliseconds. */ - public void setMinSessionTimeout(int min); + void setMinSessionTimeout(int min); /** - * Current maxSessionTimeout of the server in milliseconds + * Current maxSessionTimeout of the server in milliseconds. */ - public int getMaxSessionTimeout(); + int getMaxSessionTimeout(); + /** - * Set maxSessionTimeout of server in milliseconds + * Set maxSessionTimeout of server in milliseconds. */ - public void setMaxSessionTimeout(int max); + void setMaxSessionTimeout(int max); + + boolean getResponseCachingEnabled(); - public boolean getResponseCachingEnabled(); - public void setResponseCachingEnabled(boolean isEnabled); + void setResponseCachingEnabled(boolean isEnabled); /* Connection throttling settings */ - public int getConnectionMaxTokens(); - public void setConnectionMaxTokens(int val); Review comment: It's pretty much just the getters and setters grouped together I think. I'm fine with the standardized format, but always open for suggestions, so this is a +0 from me :) ---------------------------------------------------------------- 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: users@infra.apache.org With regards, Apache Git Services