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

switch to go-based SQLite driver (colin) #128

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

erikdw
Copy link
Collaborator

@erikdw erikdw commented Oct 27, 2023

Note
This is a resurrection of Colin King's attempt to do this:

See Collin Van Dyck's version here: #129


  • switch from CGO-based driver to Go-based
  • "fix" sqlite watch failures by removing connect hook

Warning
For this PR to be viable, we'll need to either:

  • fix that by adding connect hooks with the Go-based sqlite driver somehow -OR-
  • change the build & module import setup to somehow support using the Go-based driver for the library we produce, but the CGO-based library for the ctlstore-reflector and other ctlstore daemons.

@erikdw erikdw changed the title switch to go-based SQLite driver switch to go-based SQLite driver (colin) Oct 27, 2023
@erikdw erikdw added the Don't Merge This is only for performance testing and to resolve OOM issue for consent feature in warehouses label Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Don't Merge This is only for performance testing and to resolve OOM issue for consent feature in warehouses
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants