-
Notifications
You must be signed in to change notification settings - Fork 2.8k
SQLITE_CONSTRAINT: UNIQUE constraint failed: global_cache. cacheKey, global_cache. dir, global_cache. branch, global_cache. artifactId] #1840
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
Comments
Hey @joshvera, curious if you ran into this error at all? @sestinj and I ran into it a few times locally when switching between branches but I had assumed that for the regular user flow the deduplication logic you had here would prevent unique constraint errors. We haven't had any user reports from VS Code of this issue yet, so I'm wondering if for some reason that code path isn't being invoked in JetBrains. |
I didn't. I'd be surprised if the codepath isn't triggering, otherwise I assume the tables wouldn't exist to begin with. The easiest solution would be to turn The issue could also be the following: await Promise.all([
...remove.map(({ cacheKey }) => {
return this.deleteOrRemoveTag(cacheKey, tag);
}),
...add.map(({ cacheKey }) => {
return this.computeOrAddTag(cacheKey, tag);
}),
]); Which resolves adds and removes concurrently. If we add before we remove, then we'd see this issue. I'd check here first. AFK today, so I can't look into it but I can take a look tomorrow if it's not too pressing. I'd try removing |
I have this error as well. |
Thanks for the writeup @joshvera - I'm leaning towards just making the @levdad , I will circle back here once we get this pushed to pre-release. Please let me know if you continue to have issues once you upgrade. |
Hi, I am also experiencing this issue leading to an unusable plugin. Is there a way to use the merged solution in Webstorm/JetBrains in general or do I have to wait for a new release? If so, when should I expect the latter? Kind regards |
已收到你的邮件,谢谢~~~
|
@Patrick-Erichsen do we need to update both plugins to prerelease? |
Nevermind, it's easier to build it from sources... FIY, the issue is fixed in the dev branch. |
@Patrick-Erichsen - will there be a new version of the plugin for the JetBrains Suite at all? |
已收到你的邮件,谢谢~~~
|
Before submitting your bug report
Relevant environment info
Description
Some users are receiving the error in the title while indexing after upgrading from previous version of Continue.
The unique constraint was introduced here: https://github.com/continuedev/continue/pull/1795/files#diff-b1fd84889028918892926feb0f8dd8c5f5b574119374d8efd474fb9438430b6aR47-R76
User reports
https://discord.com/channels/1108621136150929458/1156679146932535376/1266320982809903186
To reproduce
Not exactly sure when/how users are experiencing this. I need to pinpoint which versions for VS Code/JetBrains this change was introduced.
The issue arises when attempting to insert new rows into the DB while indexing, e.g. on load, adding docs, etc.
Log output
The text was updated successfully, but these errors were encountered: