You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "shashbha14 (via GitHub)" <gi...@apache.org> on 2026/02/23 15:36:58 UTC
[PR] Fix HiveServer2Hook password handling for PLAIN auth (#62338) [airflow]
shashbha14 opened a new pull request, #62364:
URL: https://github.com/apache/airflow/pull/62364
When creating a Hive Server 2 Thrift connection using the PLAIN authentication mechanism (the default), Airflow was failing to pass the provided password to the underlying pyhive library. This caused authentication failures even when correct credentials were provided.
This PR adds PLAIN to the list of mechanisms that correctly retrieve and pass the password in HiveServer2Hook.get_conn.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fix HiveServer2Hook password handling for PLAIN auth (#62338) [airflow]
Posted by "vincbeck (via GitHub)" <gi...@apache.org>.
vincbeck commented on PR #62364:
URL: https://github.com/apache/airflow/pull/62364#issuecomment-3945561751
Please rebase correctly your PR, it contains bunch of unrelated commits
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fix HiveServer2Hook password handling for PLAIN auth (#62338) [airflow]
Posted by "shashbha14 (via GitHub)" <gi...@apache.org>.
shashbha14 commented on PR #62364:
URL: https://github.com/apache/airflow/pull/62364#issuecomment-3952324522
I've rebased the branch from the latest main and removed all unrelated commits. The PR now only contains the fix for #62338 and the corresponding unit test. Take another look, @vincbeck @amoghrajesh.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fix HiveServer2Hook password handling for PLAIN auth (#62338) [airflow]
Posted by "amoghrajesh (via GitHub)" <gi...@apache.org>.
amoghrajesh commented on PR #62364:
URL: https://github.com/apache/airflow/pull/62364#issuecomment-3949414894
Not really @shashbha14, I still see a large unrelated diff
<img width="325" height="964" alt="image" src="https://github.com/user-attachments/assets/747b4553-1f9c-42a2-a724-9a4d010f6dc5" />
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fix HiveServer2Hook password handling for PLAIN auth (#62338) [airflow]
Posted by "boring-cyborg[bot] (via GitHub)" <gi...@apache.org>.
boring-cyborg[bot] commented on PR #62364:
URL: https://github.com/apache/airflow/pull/62364#issuecomment-3954782825
Awesome work, congrats on your first merged pull request! You are invited to check our [Issue Tracker](https://github.com/apache/airflow/issues) for additional contributions.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fix HiveServer2Hook password handling for PLAIN auth (#62338) [airflow]
Posted by "jscheffl (via GitHub)" <gi...@apache.org>.
jscheffl merged PR #62364:
URL: https://github.com/apache/airflow/pull/62364
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fix HiveServer2Hook password handling for PLAIN auth (#62338) [airflow]
Posted by "shashbha14 (via GitHub)" <gi...@apache.org>.
shashbha14 commented on PR #62364:
URL: https://github.com/apache/airflow/pull/62364#issuecomment-3945752940
Hi @vincbeck, I've rebased the branch on the latest main to remove the unrelated commits. It should now only contain the fix for #62338 and the corresponding unit test. Ready for another look.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org