ken
> The change looks good, could you please help add a test for the new change? Thanks for your review. I have added a test. PTAL
> @TakaHiR07 Please rebase the master and resolve the conflict. @Jason918 done.
/pulsarbot run-failure-checks
I have added many log in the code, and found that it is blocked in topic.checkIfTransactionBufferRecoverCompletely in handleProducer(). Then I found that the object transactionBufferFuture in TopicTransactionBuffer was not completed,...
> hi @TakaHiR07 , sorry it took a long time to reply. #17051 This Pr will fix the `reader.hasMoreEvents` has been blocked problem. please re-build the branch-2.9 or wait for...
> @TakaHiR07 Hi what version of the client you are using? > > ``` > 18:27:19.260 [transaction_coordinator_TransactionCoordinatorID(id=6)thread_factory-70-1] ERROR org.apache.pulsar.broker.TransactionMetadataStoreService - End transaction fail! TxnId : (6,2309004), TxnAction : 1 org.apache.pulsar.transaction.coordinator.exceptions.CoordinatorException$CoordinatorNotFoundException:...
@nodece @Technoboy- @codelipenghui PTAL, this problem would result in permission can not be revoked
I have changed the proposal and I think it is the better way to solve this problem and meet the compatibility. Please take a look, @nodece @codelipenghui @BewareMyPower pulsar old...
> It looks like you didn't update the `org.apache.pulsar.broker.authorization.PulsarAuthorizationProvider#checkPermission`. Yes, I didn't update this for compatibility.
> As you saw, the CI fails. @nodece As you suggest, I have completely change the implementation of grant/revoke/get/check permission. Therefore many relevant unittest become flaky. What do you think...