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.

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

Update to SQLAlchemy 2.x #521

Merged
merged 4 commits into from
May 22, 2024
Merged

Conversation

andrii-i
Copy link
Collaborator

@andrii-i andrii-i commented May 21, 2024

Jupyter Scheduler uses subset of SQLAlchemy functionality supported in both SQLAlchemy 1.x and 2.x.

Updating to SQLAlchemy 2.x would allow people who want to use Jupyter Scheduler and other packages that depend on SQLAlchemy 2.x in the same environment (for example, pandas) to do so.

For reference: SQLAlchemy 2.0 - Major Migration Guide .

Fixes #478.
Will also fix a SQLAlchemy 1.x depreciation warning failing CI in #520: https://github.com/jupyter-server/jupyter-scheduler/actions/runs/9179340732/job/25241262049?pr=520#step:6:463

@andrii-i andrii-i added dependencies Pull requests that update a dependency file enhancement New feature or request labels May 21, 2024
@andrii-i andrii-i marked this pull request as ready for review May 21, 2024 19:00
Copy link
Collaborator

@srdas srdas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the change and also tested it. Works as expected.
image

@andrii-i
Copy link
Collaborator Author

Thank you @JasonWeill, @srdas.

README.md Show resolved Hide resolved
Copy link
Collaborator

@JasonWeill JasonWeill left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm a little concerned that we have two alert boxes right next to each other, but if we eventually phase out both of them, this won't be a problem in the long run.

@andrii-i andrii-i enabled auto-merge (squash) May 22, 2024 21:19
@andrii-i andrii-i merged commit d690ac8 into jupyter-server:main May 22, 2024
6 checks passed
@andrii-i andrii-i deleted the sqlalchemy-2.0 branch May 22, 2024 21:20
nsingl00 pushed a commit to nsingl00/jupyter-scheduler that referenced this pull request Jun 28, 2024
* update sqlalchemy to 2.x

* Add SQLAlchemy 2.x requirement warning to README and readthedocs

* change JupyterLab 3 "will reach" to "reached"

* Remove "anywhere on earth" as it's not a relevant information anymore
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade to SQLAlchemy 2.x
3 participants