[GitHub] [hive] soumyakanti3578 commented on a change in pull request #1390: [DRAFT] Add getTable and getAggrColStatsFor to the HS2 local cache

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

soumyakanti3578 commented on a change in pull request #1390:
URL: https://github.com/apache/hive/pull/1390#discussion_r468766052



##########
File path: standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
##########
@@ -1056,7 +1056,7 @@ public AggrStats getAggrColStatsFor(String catName, String dbName, String tblNam
       req.setCatName(catName);
       req.setValidWriteIdList(writeIdList);
 
-      return client.get_aggr_stats_for(req);
+      return getAggrStatsFromLocalCache(req);

Review comment:
       I'm thinking of renaming it to simply `getAggrStatsFor`. That'd be cleaner I think.




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



---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org