[GitHub] [hbase] virajjasani commented on issue #1006: HBASE-23045.currentPath may be stitched in a loop in replication source code.

2020-01-14 Thread GitBox
virajjasani commented on issue #1006: HBASE-23045.currentPath may be stitched in a loop in replication source code. URL: https://github.com/apache/hbase/pull/1006#issuecomment-574241320 Thanks for the fix @G302 Could you please also validate branch-1 patch to ensure if we can get

[GitHub] [hbase] virajjasani commented on issue #1006: HBASE-23045.currentPath may be stitched in a loop in replication source code.

2020-01-10 Thread GitBox
virajjasani commented on issue #1006: HBASE-23045.currentPath may be stitched in a loop in replication source code. URL: https://github.com/apache/hbase/pull/1006#issuecomment-573282022 Can you please also confirm @G302 ?

[GitHub] [hbase] virajjasani commented on issue #1006: HBASE-23045.currentPath may be stitched in a loop in replication source code.

2020-01-10 Thread GitBox
virajjasani commented on issue #1006: HBASE-23045.currentPath may be stitched in a loop in replication source code. URL: https://github.com/apache/hbase/pull/1006#issuecomment-573281891 This seems applicable to all active branch-1s only and not to branch-2+

[GitHub] [hbase] virajjasani commented on issue #1006: HBASE-23045.currentPath may be stitched in a loop in replication source code.

2020-01-09 Thread GitBox
virajjasani commented on issue #1006: HBASE-23045.currentPath may be stitched in a loop in replication source code. URL: https://github.com/apache/hbase/pull/1006#issuecomment-572488279 @G302 Thanks for the PR, changes look good, please incorporate Unit Test for this fix.