Hi, Sounds good to me.
Regards, Thomas On 09/12/14 14:56, "Davide Giannella" <[email protected]> wrote: >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 > >
