rda3mon commented on a change in pull request #3359:
URL: https://github.com/apache/hbase/pull/3359#discussion_r677947714
##########
File path:
hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/BackupInfo.java
##########
@@ -351,19 +355,19 @@ public void setIncrBackupFileList(List<String>
incrBackupFileList) {
/**
* Set the new region server log timestamps after distributed log roll
- * @param newTableSetTimestampMap table timestamp map
+ * @param prevTableSetTimestampMap table timestamp map
*/
- public void setIncrTimestampMap(HashMap<TableName,
- HashMap<String, Long>> newTableSetTimestampMap) {
- this.tableSetTimestampMap = newTableSetTimestampMap;
+ public void setIncrTimestampMap(Map<TableName,
+ Map<String, Long>> prevTableSetTimestampMap) {
+ this.incrTimestampMap = prevTableSetTimestampMap;
Review comment:
It looks like. What do you suggest to do here? Since backup is not part
of any release, I took the liberty to correct this. But open for suggestions.
--
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]
