Marcel Reutegger

Results 10 comments of Marcel Reutegger

> one concern that just came to mind with this: obviously this would increase the number of queries executed - and that scales with the number of clusterNodeIds in use......

@Joscorbe, this PR currently has conflicts. Can you please rebase your changes and resolve the conflicts?

I think the import package in `oak-store-document/pom.xml` must be adjusted as well. It currently limits the version scope to something less than 4.x.

@Joscorbe it seems to me only a few changes would be necessary to avoid use of the legacy driver, but I may be missing some less trivial changes. https://www.mongodb.com/docs/drivers/java/sync/current/legacy/#api-changes WDYT?

@mohammadMahdavi can you please add an Apache license header to your test class `FullTextSearchImplTest`? It currently fails the build. It would also be good if the PR does not contain...

@MarcyGO please create a JIRA issue for this PR. Preferably the PR should also have a unit test that reproduces the issue reliably. I'm not sure the fix is done...

Change has been committed to SVN: http://svn.apache.org/viewvc?rev=1879352&view=rev

> i haven't seen `org.apache.jackrabbit.oak.plugins.document.VersionGCWithSplitTest` failing when i ran the build locally. does that sound like being related to my patch? The test failure is most likely unrelated.