kishendas commented on a change in pull request #1095:
URL: https://github.com/apache/hive/pull/1095#discussion_r445347854



##########
File path: 
storage-api/src/java/org/apache/hadoop/hive/common/ValidTxnWriteIdList.java
##########
@@ -32,12 +32,18 @@
    */
   public static final String VALID_TABLES_WRITEIDS_KEY = 
"hive.txn.tables.valid.writeids";
 
+  /**
+   * Key used to store valid write id list for compactor in a
+   * {@link org.apache.hadoop.conf.Configuration} object.
+   */
+  public static final String COMPACTOR_VALID_TABLES_WRITEIDS_KEY = 
"hive.compactor.txn.tables.valid.writeids";
+
   // Transaction for which the list of tables valid write Ids are populated
   private Long txnId;
 
   // Map of valid write ids list for all the tables read by the current txn
   // Key is full table name string of format <db_name>.<table_name>
-  private Map<String, ValidWriteIdList> tablesValidWriteIdList = new 
HashMap<>();
+  final private Map<String, ValidWriteIdList> tablesValidWriteIdList = new 
HashMap<>();

Review comment:
       Sure




----------------------------------------------------------------
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:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to