You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucene.apache.org by "ameyakarve (via GitHub)" <gi...@apache.org> on 2024/07/03 06:24:23 UTC
[I] Merge on Commit: No merges if new data is flushed (but not committed) [lucene]
ameyakarve opened a new issue, #13537:
URL: https://github.com/apache/lucene/issues/13537
https://github.com/apache/lucene/blob/f4cd4b46fc1ccb497624c5c5454785078aca6501/lucene/core/src/java/org/apache/lucene/index/IndexWriter.java#L2361
Please see https://gist.github.com/ameyakarve/693a2a57f8836c9bce4e3eb59e85ec31 for a UT to reproduce:
I was working on something that uses the merge-on-commit feature. I noticed a place where the behavior was not per (my) expectation - if we try to perform a commit when the latest data is already flushed, then the merge never gets invoked.
This attached Gist reproduces the behavior. Commenting out `indexWriter.flush();` fixes the test.
I will be happy to dive into the behavior and fix if needed
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
Re: [I] Merge on Commit: No merges if new data is flushed (but not committed) [lucene]
Posted by "ameyakarve (via GitHub)" <gi...@apache.org>.
ameyakarve commented on issue #13537:
URL: https://github.com/apache/lucene/issues/13537#issuecomment-2226555771
Yeah this works on 9.9
Thanks for pointing it out. I'll close this issue
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
Re: [I] Merge on Commit: No merges if new data is flushed (but not committed) [lucene]
Posted by "ameyakarve (via GitHub)" <gi...@apache.org>.
ameyakarve closed issue #13537: Merge on Commit: No merges if new data is flushed (but not committed)
URL: https://github.com/apache/lucene/issues/13537
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
Re: [I] Merge on Commit: No merges if new data is flushed (but not committed) [lucene]
Posted by "ameyakarve (via GitHub)" <gi...@apache.org>.
ameyakarve commented on issue #13537:
URL: https://github.com/apache/lucene/issues/13537#issuecomment-2225559097
I think we're running on 9.4
Let me quickly try with the latest and reconfirm
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
Re: [I] Merge on Commit: No merges if new data is flushed (but not committed) [lucene]
Posted by "jpountz (via GitHub)" <gi...@apache.org>.
jpountz commented on issue #13537:
URL: https://github.com/apache/lucene/issues/13537#issuecomment-2219745791
What version are you using? We fixed a similar problem in version 9.9, I wonder if the problem that you are reporting is the same one or a new one: https://github.com/apache/lucene/pull/12549.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org