GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — [sqlite3] speed up cursor.execute*() · Issue #90792 · python/cpython

[sqlite3] speed up cursor.execute*() #90792

New issue
Copy link
New issue
Copy link
Closed
Closed
[sqlite3] speed up cursor.execute*()#90792
Copy link
Labels
3.11only security fixesonly security fixesextension-modulesC modules in the Modules dirC modules in the Modules dirtopic-sqlite3

Description

@erlend-aasland
erlend-aasland
opened on Feb 4, 2022
Issue body actions
BPO 46634
Nosy @erlend-aasland
PRs
  • bpo-46634: Inline calls to Connection.cursor() in sqlite3 #31127
    		•

    Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields:

    assignee = None
closed_at = <Date 2022-02-17.08:26:39.839>
created_at = <Date 2022-02-04.10:38:53.668>
labels = ['extension-modules', '3.11']
title = '[sqlite3]\xc2\xa0speed up cursor.execute*()'
updated_at = <Date 2022-02-17.08:26:39.838>
user = 'https://github.com/erlend-aasland'

    bugs.python.org fields:

    activity = <Date 2022-02-17.08:26:39.838>
actor = 'erlendaasland'
assignee = 'none'
closed = True
closed_date = <Date 2022-02-17.08:26:39.839>
closer = 'erlendaasland'
components = ['Extension Modules']
creation = <Date 2022-02-04.10:38:53.668>
creator = 'erlendaasland'
dependencies = []
files = []
hgrepos = []
issue_num = 46634
keywords = ['patch']
message_count = 2.0
messages = ['412503', '413390']
nosy_count = 1.0
nosy_names = ['erlendaasland']
pr_nums = ['31127']
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue46634'
versions = ['Python 3.11']

    Activity

    1. erlend-aasland commented on Feb 4, 2022

      @erlend-aasland
      erlend-aasland
      on Feb 4, 2022
      ContributorAuthor
      More actions

      pysqlite_connection_execute_impl() and friends (executemany, executescript) goes all the way through the Call API just to call pysqlite_connection_cursor_impl. We can same a lot of calls by calling the cursor _impl function directly; after all, it does live in the same file scope as the callers.

      A quick bench (sqlitesynth) shows a small speedup:

      Mean +- std dev: [main] 9.55 us +- 0.25 us -> [patched] 9.32 us +- 0.23 us: 1.02x faster

      (Side effect: will get rid of _Py_IDENTIFIER(cursor) in sqlite3)

    2. erlend-aasland
      added
      3.11only security fixesonly security fixes
      extension-modulesC modules in the Modules dirC modules in the Modules dir
      on Feb 4, 2022

    3. erlend-aasland commented on Feb 17, 2022

      @erlend-aasland
      erlend-aasland
      on Feb 17, 2022
      ContributorAuthor
      More actions

      The change proposed in #75310 was included in #75532. Closing this.

    4. erlend-aasland
      closed this as completedon Feb 17, 2022
    5. erlend-aasland
      closed this as completedon Feb 17, 2022
    6. ezio-melotti
      transferred this issue fromon Apr 10, 2022
    7. erlend-aasland
      added
      topic-sqlite3
      on May 16, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Metadata

    Metadata

    Assignees

    No one assigned

      Labels

      3.11only security fixesonly security fixesextension-modulesC modules in the Modules dirC modules in the Modules dirtopic-sqlite3

      Projects

      No projects

        Milestone

        No milestone

        Relationships

        None yet

        Development

        No branches or pull requests

        Issue actions
        You are visitor 01402520 — GeezerVison does not store the pages you look up.
        Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.