veghlaci05 commented on code in PR #3746:
URL: https://github.com/apache/hive/pull/3746#discussion_r1033542616
##########
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/TxnStore.java:
##########
@@ -115,6 +115,7 @@ enum MUTEX_KEY {
// Compactor types
char MAJOR_TYPE = 'a';
char MINOR_TYPE = 'i';
+ char REBALANCE_TYPE = 'r';
Review Comment:
We are using these consts only in
```
org.apache.hadoop.hive.metastore.txn.TxnUtils#thriftCompactionType2DbType
org.apache.hadoop.hive.metastore.txn.TxnUtils#dbCompactionType2ThriftType
```
methods, everywhere else `CompactionType` enum is used. Not sure what would
be the benefit of introducing one more enum here.
--
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]
