Marcel Reutegger
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.
@dependabot rebase
I think this is still relevant.