deniskuzZ commented on code in PR #6379:
URL: https://github.com/apache/hive/pull/6379#discussion_r3001327158
##########
ql/src/java/org/apache/hadoop/hive/llap/ProactiveEviction.java:
##########
@@ -173,16 +175,31 @@ public boolean isEmpty() {
}
/**
- * Request often times only contains tables/partitions of 1 DB only.
- * @return the single DB name, null if the count of DBs present is not
exactly 1.
+ * Request often times only contains tables/partitions of 1 catalog only.
+ * @return the single catalog name, null if the count of catalogs present
is not exactly 1.
*/
- public String getSingleDbName() {
+ public String getSingleCatalogName() {
if (entities.size() == 1) {
return entities.keySet().stream().findFirst().get();
}
return null;
}
+ /**
+ * Request often times only contains tables/partitions of 1 DB only.
+ * @return the single DB name, null if the count of (catalog, DB) pairs
present is not exactly 1.
+ */
+ public String getSingleDbName() {
+ String catalog = getSingleCatalogName();
Review Comment:
shouldn't this be done for a specific catalog and not first?
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
