rda3mon commented on a change in pull request #3359:
URL: https://github.com/apache/hbase/pull/3359#discussion_r677948243
##########
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:
`incrTimestampMap` is used by incremental backup to check when was
previous backup logroll happened so that it can take backup since then. and
stored as part of backup manifest.
--
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: issues-unsubscr...@hbase.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
