deniskuzZ commented on code in PR #3708:
URL: https://github.com/apache/hive/pull/3708#discussion_r1018194153
##########
ql/src/java/org/apache/hadoop/hive/ql/io/AcidUtils.java:
##########
@@ -230,6 +230,20 @@ public boolean accept(Path path) {
}
};
+ public enum CompactionColumn {
Review Comment:
If you want to create a column name mapping function, please keep it
localized (i.e. ShowCompactionsAnalyzer) + add mapping for "SCHEMA" ->
"DATABASE". I don't think you need to define an enum, simply append "CC_".
Note: users should be able to order by any of the columns from `show
compactions` output
--
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]
