maoling created ZOOKEEPER-3821: ---------------------------------- Summary: improve getting snapCount and globalOutstandingLimit from System's Property Key: ZOOKEEPER-3821 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-3821 Project: ZooKeeper Issue Type: Improvement Components: server Reporter: maoling
We have System.getProperty(SNAP_COUNT, defaultValue), we don't need to set the default value in an exception, that's not elegantĀ {code:java} public static int getSnapCount() { String sc = System.getProperty(SNAP_COUNT); try { int snapCount = Integer.parseInt(sc); // snapCount must be 2 or more. See org.apache.zookeeper.server.SyncRequestProcessor if (snapCount < 2) { LOG.warn("SnapCount should be 2 or more. Now, snapCount is reset to 2"); snapCount = 2; } return snapCount; } catch (Exception e) { return 100000; } }public int getGlobalOutstandingLimit() { String sc = System.getProperty(GLOBAL_OUTSTANDING_LIMIT); int limit; try { limit = Integer.parseInt(sc); } catch (Exception e) { limit = 1000; } return limit; } {code} -- This message was sent by Atlassian Jira (v8.3.4#803005)