Skip to content

Commit

Permalink
Tune the test case (#4093)
Browse files Browse the repository at this point in the history
### Motivation

When migrating the TestReplicationWorker to pulsar(apache/pulsar#21188),  the CI always failed.

We need to reset the config after test passing
  • Loading branch information
horizonzy authored Dec 4, 2023
1 parent 7a5f891 commit 2c2c583
Showing 1 changed file with 3 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1246,6 +1246,7 @@ private void testRepairedNotAdheringPlacementPolicyLedgerFragments(

baseClientConf.setProperty("reppDnsResolverClass", StaticDNSResolver.class.getName());
baseClientConf.setProperty("enforceStrictZoneawarePlacement", false);
bkc.close();
bkc = new BookKeeperTestClient(baseClientConf) {
@Override
protected EnsemblePlacementPolicy initializeEnsemblePlacementPolicy(ClientConfiguration conf,
Expand Down Expand Up @@ -1357,6 +1358,8 @@ protected EnsemblePlacementPolicy initializeEnsemblePlacementPolicy(ClientConfig
if (checkReplicationStats == null) {
rw.shutdown();
}
baseConf.setRepairedPlacementPolicyNotAdheringBookieEnable(false);
bookKeeper.close();
}

private EnsemblePlacementPolicy buildRackAwareEnsemblePlacementPolicy(List<BookieId> bookieIds) {
Expand Down

0 comments on commit 2c2c583

Please sign in to comment.