astrodbtoolkit / astrodbtoolkit/AstrodbKit

Check if conn.commit() is required

オープン
#103 コメント 0 件 リアクション 0 件 担当者 1 名 @dr-rodriguez が担当を希望しています GitHub で見る
bug
主要言語
Python
スター
12
フォーク
5
PR マージ指標
30日以内にマージされた PR はありません

説明

We have several methods that load data into the database from the JSON files. Because we have been using SQLite a lot, we typically don't encounter issues. However, other databases may explicitly require us to use `conn.commit()` or similar to commit things like inserts or updates to the database. This is a newer requirement of SQLAlchemy 2.0. I think this is an un-caught bug in our codebase.

We should run some tests with a Postgres database to see if this is indeed the case and add them as part of the underlying methods.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。