[GitHub] [hbase] saintstack commented on pull request #2504: HBASE-22976 [HBCK2] Add RecoveredEditsPlayer

2020-10-09 Thread GitBox
saintstack commented on pull request #2504: URL: https://github.com/apache/hbase/pull/2504#issuecomment-705680049 Addressed @wchevreuil review. No plans to move WALPlayer to hbck2. I'd added the hbck2 prefix since it tooling to do fixup (though it core). Thanks @wchevreuil

[GitHub] [hbase] saintstack commented on pull request #2504: HBASE-22976 [HBCK2] Add RecoveredEditsPlayer

2020-10-08 Thread GitBox
saintstack commented on pull request #2504: URL: https://github.com/apache/hbase/pull/2504#issuecomment-705680049 Addressed @wchevreuil review. No plans to move WALPlayer to hbck2. I'd added the hbck2 prefix since it tooling to do fixup (though it core). Thanks @wchevreuil

[GitHub] [hbase] saintstack commented on pull request #2504: HBASE-22976 [HBCK2] Add RecoveredEditsPlayer

2020-10-07 Thread GitBox
saintstack commented on pull request #2504: URL: https://github.com/apache/hbase/pull/2504#issuecomment-705081673 Fix tests w/ latest push. The tests were not factoring fact that now the file timestamp is considered. This

[GitHub] [hbase] saintstack commented on pull request #2504: HBASE-22976 [HBCK2] Add RecoveredEditsPlayer

2020-10-06 Thread GitBox
saintstack commented on pull request #2504: URL: https://github.com/apache/hbase/pull/2504#issuecomment-704630324 New push updates doc. This is an automated message from the Apache Git Service. To respond to the message,