eolivelli commented on pull request #11251: URL: https://github.com/apache/pulsar/pull/11251#issuecomment-878796558
@sijie > How is that related to CI in a very bad state since the integration tests for debezium is not enabled? Sorry, I wasn't clear. The point is that currently Debezium related tests are not running on CI, and I meant that if CI is not running the tests this is kind of a "bad state" :-) We cannot enable them (https://github.com/apache/pulsar/pull/11154) because currently they will fail due to the problem that this PR is trying to address. I would feel more confident with this change and any other proposal if we could revert #11056, commit #11154 and then be back in resolving the Client Auth problem. So my proposal is: - revert #11056 -> this prepares the way for #11154 - commit #11154 -> now we have CI that validates new changes - work on this fix or on #11293 Does it sound like a good plan ? because without #11154 we cannot validate on CI any patch related to Debezium -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
