Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

zk client config update and bugfix for ZKMetadataClientDriver #2958

Merged
merged 4 commits into from Jan 10, 2022

Conversation

StevenLuMT
Copy link
Contributor

Descriptions of the changes in this PR:

Motivation

  1. bug fix for error config for BoundExponentialBackoffRetryPolicy in class ZKMetadataClientDriver,if set MaxRetries zero, zk client will throw ConnectionLossException when the zk has some changing,for example: zk leader node changed.

  2. In Bookie's ZKClient, different BoundExponentialBackoffRetryPolicy set different MaxRetries,so change zkRetryBackoffMaxRetries to config in Bookie's AbstractConfiguration

Changes

  1. update a error config for BoundExponentialBackoffRetryPolicy in class ZKMetadataClientDriver
  2. change zkRetryBackoffMaxRetries to config

Master Issue: #2760

1. change zkRetryBackoffMaxRetries to config
2. update a error config for BoundExponentialBackoffRetryPolicy in class ZKMetadataClientDriver
@pkumar-singh
Copy link
Member

Very good.

Copy link
Member

@pkumar-singh pkumar-singh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@StevenLuMT StevenLuMT closed this Dec 26, 2021
@StevenLuMT StevenLuMT reopened this Dec 26, 2021
@StevenLuMT
Copy link
Contributor Author

rerun failure checks

@StevenLuMT
Copy link
Contributor Author

StevenLuMT commented Dec 26, 2021

@nicoloboschi @dlg99 @eolivelli @zymap
If you have time, please help me review it, thank you

@pkumar-singh
Copy link
Member

rerun failure checks

Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@zymap
Copy link
Member

zymap commented Dec 29, 2021

Looks like the tests have been failed multiple times. Could you please run them on your local and find what's happened?

org.apache.bookkeeper.conf.NoSystemPropertiesConfigurationTest

  Test testUseSystemProperty PASSED

org.apache.bookkeeper.replication.AuditorBookieTest

  Test testBookieClusterRestart FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)

  Test testEnsureOnlySingleAuditor FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)

  Test testSuccessiveAuditorCrashes FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)

  Test testShutdown FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)


org.apache.bookkeeper.replication.AuditorPeriodicCheckTest

  Test testPeriodicCheckWhenLedgerDeleted FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testInitialDelayOfReplicasCheck FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testInitialDelayOfPlacementPolicyCheck FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testGetLedgerFromZookeeperThrottled FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testInitialDelayOfCheckAllLedgers FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)


org.apache.bookkeeper.replication.AutoRecoveryMainTest

  Test testAutoRecoverySessionLoss FAILED (2.6s)

  java.lang.NullPointerException
      at org.apache.bookkeeper.replication.AutoRecoveryMainTest.testAutoRecoverySessionLoss(AutoRecoveryMainTest.java:118)


org.apache.bookkeeper.replication.TestReplicationWorker

  Test testDeferLedgerLockReleaseForReplicationWorker FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.TestReplicationWorker.testDeferLedgerLockReleaseForReplicationWorker(TestReplicationWorker.java:737)


org.apache.bookkeeper.test.ReadOnlyBookieTest

  Test testBookieShouldTurnWritableFromReadOnly PASSED (5s)

org.apache.bookkeeper.test.LedgerCreateDeleteTest

  Test testCreateLedgerWithZKException FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.test.LedgerCreateDeleteTest.testCreateLedgerWithZKException(LedgerCreateDeleteTest.java:92)


org.apache.bookkeeper.client.TestReadLastConfirmedLongPoll

  Test testReadLACLongPollWhenSomeBookiesDown[0] PASSED (6.5s)

org.apache.bookkeeper.replication.AuditorBookieTest

  Test testBookieClusterRestart FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)

  Test testEnsureOnlySingleAuditor FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)

  Test testSuccessiveAuditorCrashes FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)

  Test testShutdown FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)


org.apache.bookkeeper.replication.AuditorPeriodicCheckTest

  Test testPeriodicCheckWhenLedgerDeleted FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testInitialDelayOfReplicasCheck FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testInitialDelayOfPlacementPolicyCheck FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testGetLedgerFromZookeeperThrottled FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testInitialDelayOfCheckAllLedgers FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)


org.apache.bookkeeper.replication.AutoRecoveryMainTest

  Test testAutoRecoverySessionLoss FAILED (13.8s)

  java.lang.AssertionError: AR1's auditor should not be running
      at org.apache.bookkeeper.replication.AutoRecoveryMainTest.testAutoRecoverySessionLoss(AutoRecoveryMainTest.java:170)


org.apache.bookkeeper.replication.TestReplicationWorker

  Test testDeferLedgerLockReleaseForReplicationWorker FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.TestReplicationWorker.testDeferLedgerLockReleaseForReplicationWorker(TestReplicationWorker.java:737)


org.apache.bookkeeper.test.LedgerCreateDeleteTest

  Test testCreateLedgerWithZKException FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.test.LedgerCreateDeleteTest.testCreateLedgerWithZKException(LedgerCreateDeleteTest.java:92)


org.apache.bookkeeper.replication.AuditorBookieTest

  Test testBookieClusterRestart FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)

  Test testEnsureOnlySingleAuditor FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)

  Test testSuccessiveAuditorCrashes FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)

  Test testShutdown FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)


org.apache.bookkeeper.replication.AuditorPeriodicCheckTest

  Test testPeriodicCheckWhenLedgerDeleted FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testInitialDelayOfReplicasCheck FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testInitialDelayOfPlacementPolicyCheck FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testGetLedgerFromZookeeperThrottled FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testInitialDelayOfCheckAllLedgers FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)


org.apache.bookkeeper.replication.AutoRecoveryMainTest

  Test testAutoRecoverySessionLoss FAILED (13.8s)

  java.lang.AssertionError: AR1's auditor should not be running
      at org.apache.bookkeeper.replication.AutoRecoveryMainTest.testAutoRecoverySessionLoss(AutoRecoveryMainTest.java:170)


org.apache.bookkeeper.replication.TestReplicationWorker

  Test testDeferLedgerLockReleaseForReplicationWorker PASSED (8.3s)

org.apache.bookkeeper.test.LedgerCreateDeleteTest

  Test testCreateLedgerWithZKException FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.test.LedgerCreateDeleteTest.testCreateLedgerWithZKException(LedgerCreateDeleteTest.java:92)


org.apache.bookkeeper.replication.AuditorBookieTest

  Test testBookieClusterRestart FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)

  Test testEnsureOnlySingleAuditor FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)

  Test testSuccessiveAuditorCrashes FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)

  Test testShutdown FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)


org.apache.bookkeeper.replication.AuditorPeriodicCheckTest

  Test testPeriodicCheckWhenLedgerDeleted FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testInitialDelayOfReplicasCheck FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testInitialDelayOfPlacementPolicyCheck FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testGetLedgerFromZookeeperThrottled FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testInitialDelayOfCheckAllLedgers FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)


org.apache.bookkeeper.replication.AutoRecoveryMainTest

  Test testAutoRecoverySessionLoss FAILED (13.8s)

  java.lang.AssertionError: AR1's auditor should not be running
      at org.apache.bookkeeper.replication.AutoRecoveryMainTest.testAutoRecoverySessionLoss(AutoRecoveryMainTest.java:170)


org.apache.bookkeeper.test.LedgerCreateDeleteTest

  Test testCreateLedgerWithZKException FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.test.LedgerCreateDeleteTest.testCreateLedgerWithZKException(LedgerCreateDeleteTest.java:92)


org.apache.bookkeeper.replication.AuditorBookieTest

  Test testBookieClusterRestart FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)

  Test testEnsureOnlySingleAuditor FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)

  Test testSuccessiveAuditorCrashes FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)

  Test testShutdown FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorBookieTest.stopAuditorElectors(AuditorBookieTest.java:215)


org.apache.bookkeeper.replication.AuditorPeriodicCheckTest

  Test testPeriodicCheckWhenLedgerDeleted FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testInitialDelayOfReplicasCheck FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testInitialDelayOfPlacementPolicyCheck FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testGetLedgerFromZookeeperThrottled FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)

  Test testInitialDelayOfCheckAllLedgers FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.replication.AuditorPeriodicCheckTest.tearDown(AuditorPeriodicCheckTest.java:142)


org.apache.bookkeeper.replication.AutoRecoveryMainTest

  Test testAutoRecoverySessionLoss FAILED (13.9s)

  java.lang.AssertionError: AR1's auditor should not be running
      at org.apache.bookkeeper.replication.AutoRecoveryMainTest.testAutoRecoverySessionLoss(AutoRecoveryMainTest.java:170)


org.apache.bookkeeper.test.LedgerCreateDeleteTest

  Test testCreateLedgerWithZKException FAILED (2m)

  org.junit.runners.model.TestTimedOutException: test timed out after 120 seconds
      at org.apache.bookkeeper.test.LedgerCreateDeleteTest.testCreateLedgerWithZKException(LedgerCreateDeleteTest.java:92)



1808 tests completed, 71 failed, 21 skipped
FAILURE: Executed 1808 tests in 3h 44m (71 failed, 21 skipped)

@StevenLuMT
Copy link
Contributor Author

ok,I try to debug in my local project,thanks @zymap

@StevenLuMT
Copy link
Contributor Author

@eolivelli @zymap
Happy new year, if this pr has no problem , help me merge it ,thanks

@zymap zymap merged commit 84ddc91 into apache:master Jan 10, 2022
Shawyeok pushed a commit to Shawyeok/bookkeeper that referenced this pull request Oct 21, 2022
…#2958)

### Motivation

1. bug fix for error config for BoundExponentialBackoffRetryPolicy in class ZKMetadataClientDriver,if set MaxRetries zero, zk client will throw ConnectionLossException when the zk has some changing,for example: zk leader node changed.

2. In Bookie's ZKClient, different BoundExponentialBackoffRetryPolicy set different MaxRetries,so change zkRetryBackoffMaxRetries to config in Bookie's AbstractConfiguration

### Changes

1. update a error config for BoundExponentialBackoffRetryPolicy in class ZKMetadataClientDriver
2. change zkRetryBackoffMaxRetries to config

Master Issue: apache#2760

(cherry picked from commit 84ddc91)
@hangc0276 hangc0276 modified the milestones: 4.16.0, 4.15.0 Oct 21, 2022
dlg99 pushed a commit to dlg99/bookkeeper that referenced this pull request Oct 24, 2022
…#2958)

### Motivation

1. bug fix for error config for BoundExponentialBackoffRetryPolicy in class ZKMetadataClientDriver,if set MaxRetries zero, zk client will throw ConnectionLossException when the zk has some changing,for example: zk leader node changed.

2. In Bookie's ZKClient, different BoundExponentialBackoffRetryPolicy set different MaxRetries,so change zkRetryBackoffMaxRetries to config in Bookie's AbstractConfiguration

### Changes

1. update a error config for BoundExponentialBackoffRetryPolicy in class ZKMetadataClientDriver
2. change zkRetryBackoffMaxRetries to config

Master Issue: apache#2760

(cherry picked from commit 84ddc91)
dlg99 added a commit to datastax/bookkeeper that referenced this pull request Oct 24, 2022
zk client config update and bugfix for ZKMetadataClientDriver (apache#2958)
Shawyeok pushed a commit to Shawyeok/bookkeeper that referenced this pull request Oct 25, 2022
…#2958)

### Motivation

1. bug fix for error config for BoundExponentialBackoffRetryPolicy in class ZKMetadataClientDriver,if set MaxRetries zero, zk client will throw ConnectionLossException when the zk has some changing,for example: zk leader node changed.

2. In Bookie's ZKClient, different BoundExponentialBackoffRetryPolicy set different MaxRetries,so change zkRetryBackoffMaxRetries to config in Bookie's AbstractConfiguration

### Changes

1. update a error config for BoundExponentialBackoffRetryPolicy in class ZKMetadataClientDriver
2. change zkRetryBackoffMaxRetries to config

Master Issue: apache#2760

(cherry picked from commit 84ddc91)
@Shawyeok
Copy link
Contributor

Shawyeok commented Oct 25, 2022

@hangc0276
I just open a PR #3578 to cherry-pick this patch into 4.14 branch, now I see it has labeled release/4.14.6.
My question is: Is it going to pick into 4.14 branch automatically or not? Should I close PR #3578?

hangc0276 pushed a commit to hangc0276/bookkeeper that referenced this pull request Nov 7, 2022
…#2958)

### Motivation

1. bug fix for error config for BoundExponentialBackoffRetryPolicy in class ZKMetadataClientDriver,if set MaxRetries zero, zk client will throw ConnectionLossException when the zk has some changing,for example: zk leader node changed.

2. In Bookie's ZKClient, different BoundExponentialBackoffRetryPolicy set different MaxRetries,so change zkRetryBackoffMaxRetries to config in Bookie's AbstractConfiguration

### Changes

1. update a error config for BoundExponentialBackoffRetryPolicy in class ZKMetadataClientDriver
2. change zkRetryBackoffMaxRetries to config

Master Issue: apache#2760

(cherry picked from commit 84ddc91)
hangc0276 pushed a commit to hangc0276/bookkeeper that referenced this pull request Nov 7, 2022
…#2958)

### Motivation

1. bug fix for error config for BoundExponentialBackoffRetryPolicy in class ZKMetadataClientDriver,if set MaxRetries zero, zk client will throw ConnectionLossException when the zk has some changing,for example: zk leader node changed.

2. In Bookie's ZKClient, different BoundExponentialBackoffRetryPolicy set different MaxRetries,so change zkRetryBackoffMaxRetries to config in Bookie's AbstractConfiguration

### Changes

1. update a error config for BoundExponentialBackoffRetryPolicy in class ZKMetadataClientDriver
2. change zkRetryBackoffMaxRetries to config

Master Issue: apache#2760

(cherry picked from commit 84ddc91)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants