Skip to content

gh-79009: sqlite3.iterdump now correctly handles tables with autoincrement #9621

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

Merged
erlend-aasland merged 15 commits into from
Jun 19, 2022
Merged

gh-79009: sqlite3.iterdump now correctly handles tables with autoincrement #9621

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
9bf9eb7
bpo-34828 sqlite3.iterdump now correctly handles tables with autoincr…
itssme Sep 28, 2018
5d7b0ae
bpo-34828: added entry in NEWS.d/next/Library and in ACKS
itssme Sep 28, 2018
1ac20b0
Merge remote-tracking branch 'origin/master' into fix-issue-34828
itssme Jul 3, 2021
365a2f1
bpo-34828 Removed redundant code and made some small formatting chang…
itssme Jul 3, 2021
7c5901b
Merge branch 'main' into fix-issue-34828
itssme Oct 15, 2021
b259219
bpo-34828 formatting changes as well as readability improvements in s…
itssme Oct 15, 2021
3812bf6
Merge branch 'main' into fix-issue-34828
itssme May 22, 2022
daed6e5
bpo-34828: reformatted some tests in test_sqlite3/test.dump.py as wel…
itssme May 22, 2022
d5daa91
Merge branch 'main' into fix-issue-34828
erlend-aasland Jun 9, 2022
4e573e9
bpo-34828: minified test case for some sqlite3 dump command tests, fi…
itssme Jun 17, 2022
64f4101
Merge branch 'main' into fix-issue-34828
erlend-aasland Jun 19, 2022
1abc7d0
Clean up dump.py
erlend-aasland Jun 19, 2022
6bfac7e
Cleanup: don't use abbreviated link in NEWS entry
erlend-aasland Jun 19, 2022
103c50f
Cleanup: tests
erlend-aasland Jun 19, 2022
53076e9
Fix 1abc7d05b7a605224611a09eaa74e50ef1cc38e1
erlend-aasland Jun 19, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Clean up dump.py 
- consolidate construction of sqlite_sequence
- use existing quoting style
  • Loading branch information
@erlend-aasland
erlend-aasland authored Jun 19, 2022
commit 1abc7d05b7a605224611a09eaa74e50ef1cc38e1
18 changes: 10 additions & 8 deletions Lib/sqlite3/dump.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,12 +30,13 @@ def _iterdump(connection):
schema_res = cu.execute(q)
sqlite_sequence = []
for table_name, type, sql in schema_res.fetchall():
if table_name == "sqlite_sequence":
sqlite_sequence = ["DELETE FROM \"sqlite_sequence\""]
rows = cu.execute("SELECT * FROM \"sqlite_sequence\";").fetchall()
sqlite_sequence.extend(["INSERT INTO \"sqlite_sequence\" VALUES('{0}',{1})"
.format(row[0], row[1]) for row in rows])
continue
if table_name == 'sqlite_sequence':
rows = cu.execute('SELECT * FROM "sqlite_sequence";').fetchall()
sqlite_sequence = ['DELETE FROM "sqlite_sequence"']
sqlite_sequence += [
f'INSERT INTO "sqlite_sequence" VALUES(\'{row[0]}\',{row[1]})'
for row in rows
]
elif table_name == 'sqlite_stat1':
yield('ANALYZE "sqlite_master";')
elif table_name.startswith('sqlite_'):
Expand Down Expand Up @@ -72,8 +73,9 @@ def _iterdump(connection):
for name, type, sql in schema_res.fetchall():
yield('{0};'.format(sql))

# Yield statements concerning the sqlite_sequence table at the end of the transaction: (bpo-34828)
# gh-79009: Yield statements concerning the sqlite_sequence table at the
# end of the transaction.
for row in sqlite_sequence:
yield("{0};".format(row))
yield('{0};'.format(row))

yield('COMMIT;')