chenxu14 commented on a change in pull request #656: HBASE-23063 Add an option to enable multiget in parallel URL: https://github.com/apache/hbase/pull/656#discussion_r333322142
########## File path: hbase-common/src/main/java/org/apache/hadoop/hbase/HConstants.java ########## @@ -972,6 +972,17 @@ /** Whether nonces are enabled; default is true. */ public static final String HBASE_RS_NONCES_ENABLED = "hbase.regionserver.nonces.enabled"; + /** Some configuration param about multiget in parallel */ + public static final String HBASE_RS_PARALLEL_GET_ENABLED = + "hbase.regionserver.parallel.get.enabled"; + public static final boolean DEFAULT_PARALLEL_GET_ENABLED = false; Review comment: Each multi request will be handle by one Handler thread, if there are 10 multi request, the concurrency is 10 before enable this feature. But if we set the parGet threadpool’s capacity to 5 and enable this feature, the concurrency will be reduced to 5, will be some performance loss. That's what I thought before. ---------------------------------------------------------------- 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
