mikemccand commented on PR #12415:
URL: https://github.com/apache/lucene/pull/12415#issuecomment-1645462761
> counting title OR 12 is now 80% faster compared to main.
Wow! I'll try to review soon. Thanks @jpountz!
--
This is an automated message from the Apache Git Service.
To res
gashutos commented on issue #12448:
URL: https://github.com/apache/lucene/issues/12448#issuecomment-1645589833
@backslasht The overhead of `Index Sort` is very high. I ingested above 36
million documents with/without @timestamp indexsort and different is minimum
20% plus. Refer below number
heemin32 commented on PR #12287:
URL: https://github.com/apache/lucene/pull/12287#issuecomment-1646099677
> Nice find! I wonder if we could put a test in place that would have
reliably detected this issue.
This is a code for test actually. If we are writing a test using this
method,
mayya-sharipova commented on code in PR #12436:
URL: https://github.com/apache/lucene/pull/12436#discussion_r1270997853
##
lucene/core/src/java/org/apache/lucene/index/IndexingChain.java:
##
@@ -621,6 +621,12 @@ private void initializeFieldInfo(PerField pf) throws
IOException {
stefanvodita commented on PR #12287:
URL: https://github.com/apache/lucene/pull/12287#issuecomment-1646142364
I should have been clearer. I'm assuming the existing tests that call
`nextPolygon()` are passing before and after the fix, which warrants some
suspicion. Could we devise a test tha
mayya-sharipova commented on code in PR #12436:
URL: https://github.com/apache/lucene/pull/12436#discussion_r1270997853
##
lucene/core/src/java/org/apache/lucene/index/IndexingChain.java:
##
@@ -621,6 +621,12 @@ private void initializeFieldInfo(PerField pf) throws
IOException {
heemin32 commented on PR #12287:
URL: https://github.com/apache/lucene/pull/12287#issuecomment-1646175009
> I should have been clearer. I'm assuming the existing tests that call
`nextPolygon()` are passing before and after the fix, which warrants some
suspicion. Could we devise a test that
jbellis commented on PR #12421:
URL: https://github.com/apache/lucene/pull/12421#issuecomment-1646504742
Parallel code with one thread will build the same graph as serial code.
(This is validated by unit tests.) With multiple threads, parallel code
errs on the side of potentially addin
