thecoop opened a new pull request, #11847:
URL: https://github.com/apache/lucene/pull/11847
Use a shared buffer for decoding short strings
--
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 spec
rmuir commented on PR #11847:
URL: https://github.com/apache/lucene/pull/11847#issuecomment-1277440222
I don't know what this string interning is here, but I am strongly opposed
to it
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
rmuir merged PR #11844:
URL: https://github.com/apache/lucene/pull/11844
--
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.apach
rmuir closed issue #11842: TestLongBitSet.testHugeCapacity OOM
URL: https://github.com/apache/lucene/issues/11842
--
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
rmuir merged PR #11846:
URL: https://github.com/apache/lucene/pull/11846
--
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.apach
rmuir closed issue #11845: WrapperDownloader should retry on Layer3/Layer4
network errors
URL: https://github.com/apache/lucene/issues/11845
--
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
jpountz commented on PR #11843:
URL: https://github.com/apache/lucene/pull/11843#issuecomment-1277592908
> I wonder if we are running benchmarks with the cancellation/timeout
checker?
We recently introduced support for benchmarking the impact of timeouts in
the benchmark suite, but i
jpountz merged PR #11841:
URL: https://github.com/apache/lucene/pull/11841
--
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
jpountz commented on issue #11761:
URL: https://github.com/apache/lucene/issues/11761#issuecomment-1277704569
Closing: https://github.com/apache/lucene/pull/11831 has been merged.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
jpountz closed issue #11761: Expand TieredMergePolicy deletePctAllowed limits
URL: https://github.com/apache/lucene/issues/11761
--
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.
rmuir opened a new issue, #11848:
URL: https://github.com/apache/lucene/issues/11848
### Description
When looking at #11843, I noticed code of the following in several places in
ExitableDirectoryReader:
```
if (calls++ % MAX_CALLS_XXX== 0) {
checkAndThrow();
}
```
benwtrent opened a new pull request, #11849:
URL: https://github.com/apache/lucene/pull/11849
When running the `reindex` task with KnnGraphTest, exceptionally large
datasets can be used. Since mmap is used to read the data, we need to know the
buffer size. This size is limited to Integer.MA
jtibshirani merged PR #11843:
URL: https://github.com/apache/lucene/pull/11843
--
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
benwtrent closed pull request #11849: Fix failure to load larger data sets in
KnnGraphTest
URL: https://github.com/apache/lucene/pull/11849
--
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 specif
benwtrent commented on PR #11849:
URL: https://github.com/apache/lucene/pull/11849#issuecomment-1277932553
@jtibshirani or @msokolov care to review? The bug was introduced back in
https://github.com/apache/lucene/pull/1054
--
This is an automated message from the Apache Git Service.
To re
jtibshirani commented on PR #11849:
URL: https://github.com/apache/lucene/pull/11849#issuecomment-1278034553
Thanks for fixing this @benwtrent ! I wonder if we could take the simpler
approach of just opening the file, and iterating through the vectors one by
one. I don't think there's a cle
benwtrent commented on PR #11849:
URL: https://github.com/apache/lucene/pull/11849#issuecomment-1278060063
@jtibshirani My goal here was to fix the bug with as much as the original
design as possible. I didn't want to spend a bunch of time re-factoring this
code.
I am open to simply
rmuir opened a new pull request, #11850:
URL: https://github.com/apache/lucene/pull/11850
If it's performance sensitive enough that we should do sampling, then we
should avoid integer division too.
Closes #11848
--
This is an automated message from the Apache Git Service.
To respon
msokolov opened a new issue, #11851:
URL: https://github.com/apache/lucene/issues/11851
### Description
I threw together a demo for ApacheCon to show off vector search and I wanted
a scrappy UI I could hack on. Luke seemed like a good place to start since it
is already in the Lucene
msokolov opened a new pull request, #11852:
URL: https://github.com/apache/lucene/pull/11852
See #11851 for an overview of what this is
--
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 sp
msokolov commented on PR #11852:
URL: https://github.com/apache/lucene/pull/11852#issuecomment-1278129891
So -- this is just a scrappy start I wanted to post to get an idea if people
think this is worth including. The initial "overview" page is functionally
equivalent to the Luke overview s
rmuir commented on code in PR #11852:
URL: https://github.com/apache/lucene/pull/11852#discussion_r995094470
##
gradle/testing/randomization/policies/luke-tests.policy:
##
@@ -0,0 +1,141 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributo
rmuir commented on code in PR #11852:
URL: https://github.com/apache/lucene/pull/11852#discussion_r995097222
##
gradle/testing/randomization/policies/luke-tests.policy:
##
@@ -0,0 +1,141 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributo
msokolov commented on code in PR #11852:
URL: https://github.com/apache/lucene/pull/11852#discussion_r995097975
##
gradle/testing/randomization/policies/luke-tests.policy:
##
@@ -0,0 +1,141 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contrib
dsmiley commented on issue #11851:
URL: https://github.com/apache/lucene/issues/11851#issuecomment-1278229377
I believe @romseygeek worked on a HTTP based Luke-like thing and invested a
lot of time into it.
--
This is an automated message from the Apache Git Service.
To respond to the mes
dsmiley commented on PR #11847:
URL: https://github.com/apache/lucene/pull/11847#issuecomment-1278243705
This PR does not use `String.intern` which was the previous concern. So
what's wrong here?
--
This is an automated message from the Apache Git Service.
To respond to the message, plea
dsmiley commented on PR #1069:
URL: https://github.com/apache/lucene/pull/1069#issuecomment-1278245086
If only we renamed "Highlighter" to "OriginalHighlighter", maybe folks
wouldn't continue to using this thing. Is the UnifiedHighlighter not
satisfying you, and if so, why not?
--
This
rmuir commented on PR #11847:
URL: https://github.com/apache/lucene/pull/11847#issuecomment-1278380008
It does essentially the same thing. Leaking memory on purpose into static
finals.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
dsmiley commented on PR #11847:
URL: https://github.com/apache/lucene/pull/11847#issuecomment-1278448167
The map isn't static.
Even if there was a static map, *if* it was expressly used for known static
strings, then it wouldn't be a leak but just re-use of constants.
--
This is
stefanvodita commented on PR #11815:
URL: https://github.com/apache/lucene/pull/11815#issuecomment-1278524902
The second revision comes with a lot more changes to support selecting
deletes in the same fashion as segment content. I’ve reworked the tests to be
more thorough, especially about
zhaih commented on code in PR #11840:
URL: https://github.com/apache/lucene/pull/11840#discussion_r995382563
##
lucene/classification/src/java/org/apache/lucene/classification/utils/NearestFuzzyQuery.java:
##
@@ -31,13 +31,7 @@
import org.apache.lucene.index.TermStates;
import
zhaih commented on code in PR #11840:
URL: https://github.com/apache/lucene/pull/11840#discussion_r995390126
##
lucene/core/src/java/org/apache/lucene/document/FeatureQuery.java:
##
@@ -50,12 +49,12 @@ final class FeatureQuery extends Query {
}
@Override
- public Query
32 matches
Mail list logo
