[ES-1614176] Fix: Databricks OSS JDBC Driver Fails to Authenticate After Token Update#1043
[ES-1614176] Fix: Databricks OSS JDBC Driver Fails to Authenticate After Token Update#1043shivam2680 merged 4 commits intomainfrom
Conversation
jayantsing-db
left a comment
There was a problem hiding this comment.
u2m not working seems like a major bug. what versions we have u2m not working? curious how did this slip past testing?
looks like a specific CUJ in U2M doesn't work and typical U2M works? |
nikhilsuri-db
left a comment
There was a problem hiding this comment.
LGTM, Can we write end to end test to verify this fix?
|
@nikhilsuri-db - that is not possible at the moment. @jayantsing-db |
Description
Problem: When using con.setClientInfo("Auth_AccessToken", <new_token>) to update the access token, subsequent queries continued using the old token instead of the new one. This caused authentication failures when the new token was invalid or when the old token expired.
fix: Made databricksConfig mutable in DatabricksThriftAccessor and update it with new config (token) on con.setClientInfo("Auth_AccessToken", <new_token>) calls.
Testing
Unit tests
Additional Notes to the Reviewer