[ https://issues.apache.org/jira/browse/HBASE-20054?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16375150#comment-16375150 ]
Ben Lau commented on HBASE-20054: --------------------------------- [~yuzhih...@gmail.com] I guess it doesn't hurt to forward port the test improvement although the bug doesn't exist in master. For the other change though (queue adoption), I wonder. If master branch decided to abort the server if there is a replication error instead of doing a retry, maybe that is something we should go with instead. In other words, maybe we should backport the server abortion to branch-2, instead forward-porting the RPC retry to master. I also thought about this more and I wonder if indefinitely retrying is a good idea anyway. Maybe it would be better to have a limited # of retries and then abort. But it might be simpler to just abort immediately like in master-branch. Thoughts? > Forward port HBASE-18282 ReplicationLogCleaner can delete WALs not yet > replicated in case of KeeperException > ------------------------------------------------------------------------------------------------------------ > > Key: HBASE-20054 > URL: https://issues.apache.org/jira/browse/HBASE-20054 > Project: HBase > Issue Type: Bug > Reporter: Ted Yu > Assignee: Ted Yu > Priority: Major > Fix For: 3.0.0 > > Attachments: 20054.v1.txt > > > Some changes from HBASE-18282 were missing in master branch: > * The retry in ReplicationSourceManager#adoptAbandonedQueues > * new tests in TestLogsCleaner > This issue is to forward port the above to master branch -- This message was sent by Atlassian JIRA (v7.6.3#76005)