Skip to content

Commit

Permalink
[Broker] Fix compilation error related to SystemTopicClient (apache#1…
Browse files Browse the repository at this point in the history
…4774)

### Motivation

master branch is broken. It broke after merging apache#12025.


Compiling code fails with this error:
```
Error:  Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.9.0:compile (default-compile) on project pulsar-broker: Compilation failure
Error:  /home/runner/work/pulsar/pulsar/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java:[1528,36] cannot find symbol
Error:    symbol:   variable SystemTopicClient
Error:    location: class org.apache.pulsar.broker.service.BrokerService
Error:  -> [Help 1]
Error:  
```

apache#14671 had removed the `SystemTopicClient.isSystemTopic` method.


### Modifications

replace `SystemTopicClient.isSystemType` with `isSystemTopic` since BrokerService contains a method called `isSystemTopic`.
  • Loading branch information
lhotari authored Mar 21, 2022
1 parent ddf229e commit b17f86c
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -1525,7 +1525,7 @@ public CompletableFuture<ManagedLedgerConfig> getManagedLedgerConfig(TopicName t
properties.put(IsolatedBookieEnsemblePlacementPolicy.SECONDARY_ISOLATION_BOOKIE_GROUPS,
localPolicies.get().bookieAffinityGroup.getBookkeeperAffinityGroupSecondary());
managedLedgerConfig.setBookKeeperEnsemblePlacementPolicyProperties(properties);
} else if (SystemTopicClient.isSystemTopic(topicName)) {
} else if (isSystemTopic(topicName)) {
Map<String, Object> properties = Maps.newHashMap();
properties.put(IsolatedBookieEnsemblePlacementPolicy.ISOLATION_BOOKIE_GROUPS, "*");
properties.put(IsolatedBookieEnsemblePlacementPolicy
Expand Down

0 comments on commit b17f86c

Please sign in to comment.