Skip to content

CASSANDRA-21644 Fix concurrent SAI vector inserts failing once jvector's per-graph pool limit is exceeded - #5101

Open
JeremiahDJordan wants to merge 3 commits into
apache:trunkfrom
JeremiahDJordan:CASSANDRA-21644-vectormemoryindex-jvector-cap
Open

CASSANDRA-21644 Fix concurrent SAI vector inserts failing once jvector's per-graph pool limit is exceeded#5101
JeremiahDJordan wants to merge 3 commits into
apache:trunkfrom
JeremiahDJordan:CASSANDRA-21644-vectormemoryindex-jvector-cap

Conversation

@JeremiahDJordan

Copy link
Copy Markdown
Contributor

Fix concurrent SAI vector inserts failing once jvector's per-graph pool limit is exceeded.

…aph pool limit

patch by Jeremiah Jordan; reviewed by XXX for CASSANDRA-21644
…ol limit is exceeded

patch by Jeremiah Jordan; reviewed by XXX for CASSANDRA-21644
…urrent writers

Review update: TrieMemtable and the locking ShardedSkipListMemtable serialize updates per shard, so when they have no
more shards than jvector's per-builder limit there is nothing for the semaphore to bound. Add
Memtable.limitsConcurrentWritesTo(int), false by default, and let OnHeapGraph leave the semaphore out when it is true.

patch by Jeremiah Jordan; reviewed by XXX for CASSANDRA-21644
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.

1 participant