You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/02/02 06:50:31 UTC

[GitHub] [airflow] blag commented on pull request #21167: Add a session backend to store session data in the database

blag commented on pull request #21167:
URL: https://github.com/apache/airflow/pull/21167#issuecomment-1027637286


   @jedcunningham Some of the code your wanted changes to is the vendored in code. I intentionally made as minimal modifications to that code as I needed, with the intent to possibly make a PR to upstream Flask-Session, get that merged, and migrate to using that (and removing our vendored code). The more and more in-depth modifications we make to the vendored code, the more difficult it will be to switch in the future.
   
   I'm happy to make the changes you noted if you'd still like me to (please let me know), I just wanted to bring that to your attention first.


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