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.

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

test: use precompiled statements in examples #7

Merged
merged 1 commit into from
May 27, 2024

Conversation

ligurio
Copy link
Owner

@ligurio ligurio commented May 22, 2024

Prepared statements improve performance by caching the execution plan for a query after the query optimizer has found the best plan.

If the query doesn't have a complicated plan (such as simple selects/inserts with no joins), then prepared statements won't give you a big improvement since the optimizer will quickly find the best plan.

SQLite helpers has been renamed in test examples.

@coveralls
Copy link

coveralls commented May 22, 2024

Pull Request Test Coverage Report for Build 9254214822

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 4 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.6%) to 93.419%

Files with Coverage Reduction New Missed Lines %
molly/tests.lua 4 87.93%
Totals Coverage Status
Change from base Build 9254137959: -0.6%
Covered Lines: 653
Relevant Lines: 699

💛 - Coveralls

@ligurio ligurio force-pushed the ligurio/use-precompiled-queries branch 2 times, most recently from 9fa6c6f to 36c50ee Compare May 27, 2024 11:40
Prepared statements improve performance by caching the execution
plan for a query after the query optimizer has found the best plan.
However, with simple queries (such as simple selects/inserts with
no joins) prepared statements won't give a big improvement since
the optimizer will quickly find the best plan. This commit enables
using prepared statements in test examples and renames SQLite
helpers.
@ligurio ligurio force-pushed the ligurio/use-precompiled-queries branch from 36c50ee to 52b1428 Compare May 27, 2024 11:42
@ligurio ligurio merged commit e35a45a into master May 27, 2024
5 checks passed
@ligurio ligurio deleted the ligurio/use-precompiled-queries branch May 27, 2024 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants