Revert "[PIP 97] Update Authentication Interfaces to Include Async Au… #14424
Conversation
…thentication Methods (apache#12104)" This reverts commit 5868025.
|
@codelipenghui - since we need to do a second RC for 2.10.0, I realized I should revert PIP 97 while I have the chance. I plan to complete the work soon, but it didn't make 2.10. |
|
I am looking into why non of the tests ran. |
|
@codelipenghui - I had opened this PR with the intention of making sure tests pass. Apparently, we don't have tests configured to run on these PRs--I thought we did. I am going to merge this now since I believe it is the correct change and will therefore get included in the next 2.10 RC. |
|
@michaeljmarshall Ok, thanks for the update, any reason for reverting the change? |
|
@codelipenghui - I reverted the commit that was responsible for updating the interfaces because none of the other work has been completed. It'd be a bit misleading for third parties to implement the interface since it is still synchronous. I plan to implement PIP 97 for Pulsar 2.11. |
2 participants
…thentication Methods (#12104)"
This reverts commit 5868025.
Master Issue: #12105
Motivation
Because the PIP is not completely implemented, we should not publish the interface changes in 2.10.
Modifications
Verifying this change
This change is trivial, because the original change was completely backwards compatible.
Documentation
no-need-doc