congbo

Results 41 comments of congbo

one question. leader load the topic, may cause the normal topic become fence. may need to consider how to not affect the normal topic behavior.

Hi @syhily TransactionCoordinator loadBalance base on partition topic `persistent://pulsar/system/transaction_coordinator_assign` , so when starting transaction client, make sure that partition topic `persistent://pulsar/system/transaction_coordinator_assign` has been created. Otherwise `persistent://pulsar/system/transaction_coordinator_assign` will be automatically created...

@syhily What users should care about is whether the transaction is fully enabled, rather than using it when it is not fully enabled, isn't it? If there are no other...

> Just one suggestion about the new option name > > Maybe `messagesFromMultiTopicsEnabled=true` is more understandable. And add a description of the behavior if the user disables it. It looks...

https://github.com/apache/pulsar/blob/adae4ae2b06304eae9ec001357e0bd6ee6ffb053/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java#L3030-L3046 I think this method only need to check whether have schema, if doesn't hava schema then add the schema, if have shema then check the compatible. so the code...

@poorbarcode mainly to fix the flaky test, this is just to remove the wrong logic

@thetumbled hi, I move this PR to release/2.9.5, if you have any questions, please ping me. thanks.

@TakaHiR07 hi, I move this PR to release/2.9.5, if you have any questions, please ping me. thanks.

@thetumbled hi, could you please merge the `apache/master` branch, seem the test can't rerun

``` Error: org.apache.pulsar.broker.service.ReplicatorSubscriptionWithTransactionTest.testReplicatedSubscriptionWhenReplicatorProducerIsClosed Time elapsed: 12.567 s