benchaplin opened a new pull request, #13887:
URL: https://github.com/apache/lucene/pull/13887
### Description
[This issue](https://github.com/apache/lucene/issues/13234) raises a
question about the QueryParser's ability to handle escaped brackets in a range
query's terms.
```
dweiss commented on PR #13884:
URL: https://github.com/apache/lucene/pull/13884#issuecomment-2406582594
Thank you!
--
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 unsubs
dweiss merged PR #13884:
URL: https://github.com/apache/lucene/pull/13884
--
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-unsubscr...@lucene.apac
bugmakerr commented on PR #13884:
URL: https://github.com/apache/lucene/pull/13884#issuecomment-2406512304
@dweiss Thanks. I've fixed the lint error, PTAL.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL ab
BrianWoolfolk commented on PR #13886:
URL: https://github.com/apache/lucene/pull/13886#issuecomment-2406383720
There are some failing tests because of the unused methods and imports from
the older logic, and I wanted to see if my implementation of the doc skipper is
good before removing the
BrianWoolfolk opened a new pull request, #13886:
URL: https://github.com/apache/lucene/pull/13886
Fixes #13840
`IndexSortSortedNumericDocValuesRangeQuery` now implements a similar logic
(using `DocValuesSkipper`) as `SortedNumericDocValuesRangeQuery`'s
`getDocIdSetIteratorOrNullForP
jpountz commented on PR #13885:
URL: https://github.com/apache/lucene/pull/13885#issuecomment-2405769428
If you're looking at this sort of allocation, you may also want to
specialize BlockDocsEnum into one class that decodes only doc IDs and another
one that decodes docs and freqs. The form
jpountz commented on PR #13885:
URL: https://github.com/apache/lucene/pull/13885#issuecomment-2405765367
Even if it doesn't show up in benchmarks it's disappointing to have these
conditions in hot code paths. Could we instead initialize these objects in
reset() if `docFreq >= BLOCK_SIZE`?
dweiss commented on PR #13884:
URL: https://github.com/apache/lucene/pull/13884#issuecomment-2405653923
Sure, thank you. The linter has failed - seems like an unused method. Please
correct and perhaps add a CHANGES.txt entry with an attribution to you?
--
This is an automated message from
original-brownbear opened a new pull request, #13885:
URL: https://github.com/apache/lucene/pull/13885
Lazy initialize these fields. They consume/cause a lot of memory/GC because
they are allocated frequently (~7% of all allocations in luceneutil's wikimedia
medium run for me). This does no
bugmakerr commented on PR #13884:
URL: https://github.com/apache/lucene/pull/13884#issuecomment-2405554212
@dweiss would you mind taking a look at this if you get a chance
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
bugmakerr opened a new pull request, #13884:
URL: https://github.com/apache/lucene/pull/13884
### Description
Fixes #13761
--
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 speci
bugmakerr commented on issue #13761:
URL: https://github.com/apache/lucene/issues/13761#issuecomment-2405515035
@dweiss I think that `CharObjectHashMap` and `LongObjectHashMap` need to be
fixed as well, I can help to create a PR.
--
This is an automated message from the Apache Git Ser
jpountz opened a new issue, #13883:
URL: https://github.com/apache/lucene/issues/13883
### Description
`ConcurrentMergeScheduler` computes max thread counts assuming a single
`IndexWriter` in the JVM. But it's common with Solr or Elasticsearch to have
tens of active `IndexWriter`s ru
ChrisHegarty merged PR #13882:
URL: https://github.com/apache/lucene/pull/13882
--
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-unsubscr...@lucen
mikemccand commented on code in PR #13879:
URL: https://github.com/apache/lucene/pull/13879#discussion_r1795372954
##
lucene/backward-codecs/src/test/org/apache/lucene/backward_index/TestGenerateBwcIndices.java:
##
@@ -82,6 +82,16 @@ public void testCreateSortedIndex() throws IO
jpountz commented on PR #13882:
URL: https://github.com/apache/lucene/pull/13882#issuecomment-2404950119
LGTM
--
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
jpountz commented on PR #13871:
URL: https://github.com/apache/lucene/pull/13871#issuecomment-2404883858
Cool, I pushed an annotation.
--
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
original-brownbear commented on PR #13871:
URL: https://github.com/apache/lucene/pull/13871#issuecomment-2404878244
Now it's there:
``
IntervalsOrdered 19.1(2.6%) 22.0(2.4%) 1.2 X 0.000
```
:)
--
This is an automated message from the Apache Git Service
ChrisHegarty commented on PR #13882:
URL: https://github.com/apache/lucene/pull/13882#issuecomment-2404854635
--
This is an automated message from the Apache Git Servi
ChrisHegarty opened a new pull request, #13882:
URL: https://github.com/apache/lucene/pull/13882
This commit tombstones branch_9x - wipes all files and updates the readme to
point to more modern branches.
Not much point looking at the files changes. Maybe just update to the branch
an
dweiss merged PR #13835:
URL: https://github.com/apache/lucene/pull/13835
--
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-unsubscr...@lucene.apac
dweiss commented on PR #13835:
URL: https://github.com/apache/lucene/pull/13835#issuecomment-2404620384
Thank you. I'll merge this into branch_9x and branch_10x.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
jpountz commented on PR #12589:
URL: https://github.com/apache/lucene/pull/12589#issuecomment-2404503497
No, the sentence is correct. Non essential clauses are clauses that cannot
produce a match on their own because the score wouldn't be high enough. So the
more non-essential clauses, the
jpountz commented on PR #13800:
URL: https://github.com/apache/lucene/pull/13800#issuecomment-2404466253
@zhaih Indeed conjunctions need a similar fix, I'll look into it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
jpountz commented on code in PR #12868:
URL: https://github.com/apache/lucene/pull/12868#discussion_r1794974762
##
lucene/codecs/src/java/org/apache/lucene/codecs/bloom/FuzzySet.java:
##
@@ -150,9 +149,10 @@ private FuzzySet(FixedBitSet filter, int bloomSize, int
hashCount) {
jpountz merged PR #13596:
URL: https://github.com/apache/lucene/pull/13596
--
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-unsubscr...@lucene.apa
breskeby commented on PR #13835:
URL: https://github.com/apache/lucene/pull/13835#issuecomment-2404281566
> Part of this is actually already in gradle-archives.gradle - I think
adding this patch there would be better?
@dweiss updated this accordingly. boiled down to a one line change
ChrisHegarty commented on PR #13874:
URL: https://github.com/apache/lucene/pull/13874#issuecomment-2404222904
> if I separately run `./gradlew clean` and then `./gradlew check` then it's
fine ... weird.
We’ve seen similar recently, reported in #13567. Likely a grade bug, which
w
29 matches
Mail list logo
