Good afternoon everyone,

while I was looking into other issues, I found that it would be useful
if we pass to the IndexStoreStrategy.update() either the indexName, or
even better a NodeState of the indexDefinition. This will allow us to
have more clear logging in the strategies.

Something like what we did with IndexStoreStrategy.query().

Someone against passing down the NodeState of the index definition to
the update?

I opt for a NodeState so that the strategy won't be able, even
accidentally, to change the index definition.

Thoughts?

If no one against it I will take care of updating it and correct the
existing callers and implementaions

# Callers

PropertyIndexEditor.leave()

# Implementors

ContentMirrorStoreStrategy
UniqueEntryStoreStrategy

Davide


Reply via email to