Skip to content

Conversation

@erikdw
Copy link
Contributor

@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.

3 participants