Michae94 commented on PR #34927:
URL: https://github.com/apache/airflow/pull/34927#issuecomment-1787522055

   Hey @baruch-shteken , thanks for the answer.
   I agree with everything you mentioned above, except the conclusion.
   
   Let's say we have 2 concurrent tasks, completely isolated (not run on the 
same machine). Both of them being will use the same PAT to authenticate. 
   **Example of the sequence:**
   - First job authenticate, and start to access Tableau using the 
session_token retrieved.
   - While the first job is still running, we have our 2nd job that now starts 
to authenticate and retrieve a new session_token while first job is not finished
   - **From my understanding**: the first job will interrupt and fail because 
the session_token used for the job is not viable anymore.
   
   Maybe there is something I missed 🤔 
   As you said, I recommend testing your implementation with 1 PAT to see if 
it's working (pull this version of the `apache` package instead on a testing 
environment, try to run 2 jobs).


-- 
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]

Reply via email to