FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

bpo-44976: Lazy creation of `sqlite3` result rows by erlend-aasland · Pull Request #27884 · python/cpython · GitHub

/ cpython Public

bpo-44976: Lazy creation of sqlite3 result rows - #27884

Merged
pablogsal merged 1 commit into
python:mainfrom
erlend-aasland:sqlite-fetch-when-needed
Aug 25, 2021
Merged

bpo-44976: Lazy creation of sqlite3 result rows#27884
pablogsal merged 1 commit into
python:mainfrom
erlend-aasland:sqlite-fetch-when-needed

Conversation

erlend-aasland commented Aug 22, 2021
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Contributor
  • remove 'next_row' member from Cursor object
  • remove _pysqlite_fetch_one_row() call from query execute loop
  • as a result of the two above, pysqlite_cursor_iternext() is vastly simplified

https://bugs.python.org/issue44976

- remove 'next_row' member in Cursor object
- remove fetch-one-row call from query execute loop
- rewrite cursor iternext method to comply with the new behaviour

Copy link
Copy Markdown
Contributor Author

Internal change; no user-visible or backwards incompatible changes; skipping news.

erlend-aasland added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Aug 22, 2021

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @erlend-aasland for commit ce0d55f 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Aug 22, 2021

Copy link
Copy Markdown
Contributor Author

The AMD64 Arch Linux Asan Debug PR buildbot failure seems to be unrelated. I see that it's been failing a lot recently.

pablogsal merged commit 3df0fc8 into python:main Aug 25, 2021
erlend-aasland deleted the sqlite-fetch-when-needed branch August 25, 2021 10:31

Copy link
Copy Markdown
Contributor Author

Thanks, Pablo!

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL