Skip to content
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

Jump to bottom

sqlite: cache column names in stmt.all() #55373

Merged
merged 1 commit into from
Oct 15, 2024
Merged

sqlite: cache column names in stmt.all() #55373

merged 1 commit into from
Oct 15, 2024

Commits on Oct 13, 2024

  1. sqlite: cache column names in stmt.all() 

    While the statement is running, it is impossible to modify the column
names and thus it is beneficial to create the host-language (JS) keys
once per all rows and reuse them for all results.

With this change the performance of `.all()` improves by around 25%
depending on the number of columns and rows in the result.
    @indutny
    indutny committed Oct 13, 2024
    Configuration menu
    Copy the full SHA
    b6f3f6d View commit details
    Browse the repository at this point in the history