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

Close db engine #561

Merged
merged 4 commits into from
Sep 22, 2016
Merged

Close db engine #561

merged 4 commits into from
Sep 22, 2016

Conversation

jhamrick
Copy link
Member

Fixes #560

This adds a close() method to the Gradebook object and modifies everything to use that rather than Gradebook.db.close(). This ensures that the engine is also shutdown properly.

@jhamrick jhamrick added this to the 0.4.0 milestone Sep 22, 2016
@jhamrick jhamrick modified the milestones: 0.3.1, 0.4.0 Sep 22, 2016
@jhamrick jhamrick merged commit cef150b into jupyter:master Sep 22, 2016
@jhamrick jhamrick deleted the close_db_engine branch September 22, 2016 20:23
jhamrick added a commit that referenced this pull request Sep 22, 2016
Fixes #560

This adds a close() method to the Gradebook object and modifies everything to use that rather than Gradebook.db.close(). This ensures that the engine is also shutdown properly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sqlalchemy error when invoking autograder
1 participant