eladkal commented on a change in pull request #16916:
URL: https://github.com/apache/airflow/pull/16916#discussion_r667366994
##########
File path: airflow/providers/tableau/hooks/tableau.py
##########
@@ -41,6 +40,16 @@ class TableauHook(BaseHook):
"""
Connects to the Tableau Server Instance and allows to communicate with it.
+ Can be used as a context manager: automatically authenticates the
connection
+ when opened and signs out when closed.
+
+ WARNING: authenticaton by personal token was deprecated as Tableau
automatically
+ invalidates opened personal token connection if one or more parallel
+ connections with the same token are opened. So, in the environments with
+ multiple parallel tasks this authentication method can lead to numerous
bugs
+ and all the jobs will not run as they intended. Therefore personal token
+ auth option is no more available for the end user.
Review comment:
The place for this is in the docs:
https://github.com/apache/airflow/blob/main/docs/apache-airflow-providers-tableau/connections/tableau.rst
--
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]
