Skip to content

Add colsUsed to vtable BestIndex callback#1378

Open
theimpostor wants to merge 2 commits intomattn:masterfrom
theimpostor:pr-1323-bestindex-colused
Open

Add colsUsed to vtable BestIndex callback#1378
theimpostor wants to merge 2 commits intomattn:masterfrom
theimpostor:pr-1323-bestindex-colused

Conversation

@theimpostor
Copy link
Copy Markdown
Contributor

Split out of #1336 per review feedback.

Summary

  • pass sqlite3_index_info.colUsed through to the Go VTab.BestIndex callback
  • update the examples and tests for the new callback signature
  • add test coverage for the propagated colsUsed mask

Fixes #1323.

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.

vtable BestIndex callback missing colUsed input

1 participant