Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

DM-37744: Make daf_butler compatible with sqlalchemy 2 #788

Merged
merged 4 commits into from
Feb 11, 2023
Merged

DM-37744: Make daf_butler compatible with sqlalchemy 2 #788

merged 4 commits into from
Feb 11, 2023

Commits on Feb 10, 2023

  1. Get rid of RemovedIn20Warning showing up with SQLALCHEMY_WARN_20=1

    @andy-slac
    andy-slac committed Feb 10, 2023
    Configuration menu
    Copy the full SHA
    ff00d6a View commit details
    Browse the repository at this point in the history

  2. Fix for issues uncovered with SA 2.0 

    There are two issues fixed in this commit:
- SA now handles UUID internally for postgresql and returns UUID.
- Transaction handling needs more care as there is no autocommit.
- str(URL) replaces password with starts.
    @andy-slac
    andy-slac committed Feb 10, 2023
    Configuration menu
    Copy the full SHA
    107f5d6 View commit details
    Browse the repository at this point in the history

  3. Mypy fixes for sqlalchemy 2

    @andy-slac
    andy-slac committed Feb 10, 2023
    Configuration menu
    Copy the full SHA
    24230ca View commit details
    Browse the repository at this point in the history

  4. Allow sqlalchemy 2 in requirements

    @andy-slac
    andy-slac committed Feb 10, 2023
    Configuration menu
    Copy the full SHA
    2c47f31 View commit details
    Browse the repository at this point in the history