Skip to content

Added conn.commit() to actually save the table in the db#204

Merged
mattmakai merged 1 commit intomattmakai:masterfrom
josephpaulmckenzie:master
Feb 15, 2019
Merged

Added conn.commit() to actually save the table in the db#204
mattmakai merged 1 commit intomattmakai:masterfrom
josephpaulmckenzie:master

Conversation

@josephpaulmckenzie
Copy link
Copy Markdown
Contributor

Needed to add conn.commit() to save the table else when you go to view the table in a db manager it will not exist.Also added a conn.close() to close out the connection after creating the table or inserting new data

Need to add conn.commit() to save the table else when you go to view the table in a db manager it will not exsist
@mattmakai mattmakai merged commit b0402e3 into mattmakai:master Feb 15, 2019
@mattmakai
Copy link
Copy Markdown
Owner

cool thanks @josephpaulmckenzie. hmm it's been awhile since I wrote the post I could've sworn only DML changes needed a commit not DDL. but if you tested and it's required that makes sense

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants