Conversation
|
@luoyuxia @leekeiabstraction PTAL 🙏 |
7a464b6 to
03de6a7
Compare
There was a problem hiding this comment.
Copilot reviewed 30 out of 30 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@luoyuxia fixed issue with cpp tests, caused by additional \ used by formatter. |
leekeiabstraction
left a comment
There was a problem hiding this comment.
TY for the PR for such an important feature! Left some comments.
|
@leekeiabstraction Ty, addressed comments, PTAL 🙏 |
36e8cdc to
8e94710
Compare
|
@luoyuxia @leekeiabstraction Rebased and fixed conflicts. PTAL |
leekeiabstraction
left a comment
There was a problem hiding this comment.
TY for the PR!
8e94710 to
5abbd87
Compare
|
rebased |
luoyuxia
left a comment
There was a problem hiding this comment.
@fresh-borzoni Thanks. LGTM. Merging..
|
@fresh-borzoni Seems ci fails after merge this pr. https://github.com/apache/fluss-rust/actions/runs/22520032930/job/65243381417 |
Looking into this |
Summary
closes #367 #374
Add SASL/PLAIN authentication support to the Rust client, matching the Java Fluss client behavior.
Authenticaterequest/response, exponential backoff with jitter (matching Java'sExponentialBackoff), retriable auth error handlingsecurity_protocol,security_sasl_mechanism,security_sasl_username,security_sasl_password,connect_timeout_msLazyLockcluster with dual listeners, enabling concurrent test execution