[ 
https://issues.apache.org/jira/browse/CASSANDRA-21160?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18091937#comment-18091937
 ] 

Caleb Rackliffe commented on CASSANDRA-21160:
---------------------------------------------

[~arvindk12] Sorry it took so long for me to get back to this!

I think the easiest thing to do would be to add {{synchronized}} to the 
signature of {{updateKeyBounds()}}. That should protect the access from both 
{{index()}} and {{update()}}. I'll put up a quick patch...

> VectorMemoryIndex has unnecessary synchronization on the add method
> -------------------------------------------------------------------
>
>                 Key: CASSANDRA-21160
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-21160
>             Project: Apache Cassandra
>          Issue Type: Improvement
>          Components: Feature/Vector Search
>            Reporter: Michael Marshall
>            Priority: Normal
>             Fix For: 5.0.x, 6.x, 7.x
>
>
> The `VectorMemoryIndex` supports asynchronous additions to the graph. 
> However, the `synchronous` keyword for the `add` method prevents this and 
> therefore means we are adding a point of contention that is unnecessary.
> As part of this ticket, we should verify my understanding that writes can be 
> concurrent and then drop the keyword.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to