[GitHub] [hbase] Apache9 commented on a change in pull request #147: HBASE-22238 Fix TestRpcControllerFactory

[GitBox <gi...@apache.org>]

Apache9 commented on a change in pull request #147: HBASE-22238 Fix TestRpcControllerFactory
URL: https://github.com/apache/hbase/pull/147#discussion_r275644499
 
 

 ##########
 File path: hbase-endpoint/src/test/java/org/apache/hadoop/hbase/client/TestRpcControllerFactory.java
 ##########
 @@ -85,32 +86,15 @@ public HBaseRpcController newController(final List<CellScannable> cellIterables)
 
     private static Multiset<Integer> GROUPED_PRIORITY = ConcurrentHashMultiset.create();
     private static AtomicInteger INT_PRIORITY = new AtomicInteger();
-    private static AtomicInteger TABLE_PRIORITY = new AtomicInteger();
 
 Review comment:
   We do not call the setPriority(TableName) method any more in async client, we will decide the priority outside the RpcController and then set with the setPriority(int) method. This is why we need to change the UT itself.

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