mikemccand commented on PR #11784:
URL: https://github.com/apache/lucene/pull/11784#issuecomment-1790576576
Thanks @msokolov. This looks like a nice tool, helpful for giving demos of
cool Lucene features at conferences, but it looks like consensus is we should
not add it to Lucene? Maybe
mikemccand closed pull request #11852: Luke Webapp
URL: https://github.com/apache/lucene/pull/11852
--
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: issu
mikemccand commented on PR #11852:
URL: https://github.com/apache/lucene/pull/11852#issuecomment-1790579836
Thanks @msokolov. This looks like a nice tool, helpful for giving demos of
cool Lucene features at conferences, but it looks like consensus is we should
not add it to Lucene? Maybe lu
mikemccand commented on PR #907:
URL: https://github.com/apache/lucene/pull/907#issuecomment-1790591979
Thank you for persisting so hard on this one @shahrs87 -- I'm sorry it looks
like we should close it at this point, but your efforts / iterations were
needed to see that we are mostly exc
mikemccand closed pull request #907: LUCENE-10357 Ghost fields and
postings/points
URL: https://github.com/apache/lucene/pull/907
--
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
mikemccand commented on PR #11903:
URL: https://github.com/apache/lucene/pull/11903#issuecomment-1790595404
> @mikemccand Merging this PR will require regolding nightly benchmarks.
Does it help if you can control when the PR gets merged?
Oh no, I failed to reply to this, until now! N
mikemccand commented on PR #11934:
URL: https://github.com/apache/lucene/pull/11934#issuecomment-1790597593
Looks like we have since removed `IOContext` from `openChecksumInput` since
such an `IndexInput` must always be `READONCE` anyways.
--
This is an automated message from the Apache G
mikemccand closed pull request #11934: Fix a few calls to
`Directory#openChecksumInput` to pass the right `IOContext`.
URL: https://github.com/apache/lucene/pull/11934
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
jpountz commented on PR #1052:
URL: https://github.com/apache/lucene/pull/1052#issuecomment-1790599704
+1 I fell a bit into a trap by trying to make long shared prefixes less
adversarial. Let's do progress over perfection and start with a simple approach
and look into whether/how we can bet
jpountz commented on PR #1052:
URL: https://github.com/apache/lucene/pull/1052#issuecomment-1790603130
For reference, it should speed up:
- OrdinalMap construction
- Merging of terms in the inverted index
- Merging of terms in doc values (as a side-effect of the OrdinalMap
speedu
dungba88 commented on code in PR #12738:
URL: https://github.com/apache/lucene/pull/12738#discussion_r1380001349
##
lucene/core/src/java/org/apache/lucene/util/fst/NodeHash.java:
##
@@ -328,7 +298,100 @@ private void rehash(long lastNodeAddress) throws
IOException {
}
dungba88 commented on code in PR #12738:
URL: https://github.com/apache/lucene/pull/12738#discussion_r1380001349
##
lucene/core/src/java/org/apache/lucene/util/fst/NodeHash.java:
##
@@ -328,7 +298,100 @@ private void rehash(long lastNodeAddress) throws
IOException {
}
mikemccand commented on code in PR #11847:
URL: https://github.com/apache/lucene/pull/11847#discussion_r1380002867
##
lucene/core/src/java/org/apache/lucene/codecs/lucene94/Lucene94FieldInfosFormat.java:
##
@@ -145,8 +145,10 @@ public FieldInfos read(
// previous field'
slow-J commented on PR #12741:
URL: https://github.com/apache/lucene/pull/12741#issuecomment-1790605861
Thanks @mikemccand and yes, the codec version bump is the majority of this
change :D
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
mikemccand commented on PR #11847:
URL: https://github.com/apache/lucene/pull/11847#issuecomment-1790607699
It looks like there are strong objections to sharing string instances here,
and there is a JVM command-line flag that may achieve similar gains for many
indices X segments X fields so
mikemccand closed pull request #11847: Add a method allowing canonical strings
to be returned from DataInput
URL: https://github.com/apache/lucene/pull/11847
--
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 t
mikemccand commented on PR #11955:
URL: https://github.com/apache/lucene/pull/11955#issuecomment-1790613216
It looks like this is ready to be merged @rmuir? open-nlp may have thread
safety issues but 1) Lucene should not work around those bugs, and 2) the user
(of open-nlp tokenizers in Lu
dungba88 commented on code in PR #12738:
URL: https://github.com/apache/lucene/pull/12738#discussion_r1380008658
##
lucene/core/src/java/org/apache/lucene/util/fst/NodeHash.java:
##
@@ -328,7 +298,100 @@ private void rehash(long lastNodeAddress) throws
IOException {
}
dungba88 commented on code in PR #12738:
URL: https://github.com/apache/lucene/pull/12738#discussion_r1380008658
##
lucene/core/src/java/org/apache/lucene/util/fst/NodeHash.java:
##
@@ -328,7 +298,100 @@ private void rehash(long lastNodeAddress) throws
IOException {
}
dungba88 commented on code in PR #12738:
URL: https://github.com/apache/lucene/pull/12738#discussion_r1380008658
##
lucene/core/src/java/org/apache/lucene/util/fst/NodeHash.java:
##
@@ -328,7 +298,100 @@ private void rehash(long lastNodeAddress) throws
IOException {
}
rmuir commented on code in PR #12747:
URL: https://github.com/apache/lucene/pull/12747#discussion_r1380016895
##
lucene/benchmark-jmh/src/java/org/apache/lucene/benchmark/jmh/VectorUtilBenchmark.java:
##
@@ -24,8 +24,14 @@
@BenchmarkMode(Mode.Throughput)
@OutputTimeUnit(TimeUn
rmuir commented on code in PR #12747:
URL: https://github.com/apache/lucene/pull/12747#discussion_r1380017854
##
lucene/benchmark-jmh/src/java/org/apache/lucene/benchmark/jmh/VectorUtilBenchmark.java:
##
@@ -56,84 +62,72 @@ public void init() {
}
@Benchmark
- @Fork(valu
rmuir commented on code in PR #12747:
URL: https://github.com/apache/lucene/pull/12747#discussion_r1380018590
##
lucene/benchmark-jmh/src/java/org/apache/lucene/benchmark/jmh/VectorUtilBenchmark.java:
##
@@ -56,84 +62,72 @@ public void init() {
}
@Benchmark
- @Fork(valu
dungba88 commented on code in PR #12738:
URL: https://github.com/apache/lucene/pull/12738#discussion_r1380008658
##
lucene/core/src/java/org/apache/lucene/util/fst/NodeHash.java:
##
@@ -328,7 +298,100 @@ private void rehash(long lastNodeAddress) throws
IOException {
}
jpountz commented on PR #319:
URL: https://github.com/apache/lucene/pull/319#issuecomment-1790626991
It's still relevant but I'm not comfortable with the fact that it's a bit
fragile. I'll close for now and think more about it.
--
This is an automated message from the Apache Git Service.
jpountz closed pull request #319: LUCENE-10121: More skipping in WANDScorer.
URL: https://github.com/apache/lucene/pull/319
--
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 u
jpountz commented on PR #330:
URL: https://github.com/apache/lucene/pull/330#issuecomment-1790631133
Yes we do! I'll look into moving this forward...
--
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
mikemccand merged PR #12273:
URL: https://github.com/apache/lucene/pull/12273
--
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.
jpountz commented on code in PR #12729:
URL: https://github.com/apache/lucene/pull/12729#discussion_r1380023598
##
lucene/core/src/java/org/apache/lucene/codecs/lucene99/Lucene99HnswVectorsReader.java:
##
@@ -399,41 +281,30 @@ private HnswGraph getGraph(FieldEntry entry) throws
mikemccand commented on PR #12273:
URL: https://github.com/apache/lucene/pull/12273#issuecomment-1790634871
Merged & backported to 9.9.0. Sorry for the long delay @luyuncheng!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
slow-J opened a new issue, #12749:
URL: https://github.com/apache/lucene/issues/12749
### Description
Idea from @mikemccand 's comment in
https://github.com/apache/lucene/issues/12696#issuecomment-1770461719
```
Another exciting optimization such a "patch-less" encoding coul
mikemccand commented on PR #12277:
URL: https://github.com/apache/lucene/pull/12277#issuecomment-1790637499
Thanks for incorporating @rmuir's feedback @tang-hi! The change looks great
to me: we catch an invalid usage and throw a clean exception in that case.
I'll merge! Sorry for the lon
mikemccand merged PR #12277:
URL: https://github.com/apache/lucene/pull/12277
--
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.
dungba88 commented on code in PR #12738:
URL: https://github.com/apache/lucene/pull/12738#discussion_r1380008658
##
lucene/core/src/java/org/apache/lucene/util/fst/NodeHash.java:
##
@@ -328,7 +298,100 @@ private void rehash(long lastNodeAddress) throws
IOException {
}
slow-J commented on issue #12696:
URL: https://github.com/apache/lucene/issues/12696#issuecomment-1790638953
> Another exciting optimization such a "patch-less" encoding could implement
is within-block skipping (I believe Tantivy does this).
>
> Today, our skipper is forced to align t
mikemccand commented on PR #12277:
URL: https://github.com/apache/lucene/pull/12277#issuecomment-1790640840
Merged to 10.0 and 9.9.0. Thanks @tang-hi!
--
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 g
mikemccand commented on PR #12295:
URL: https://github.com/apache/lucene/pull/12295#issuecomment-1790653367
In general it's great for Lucene devs to use the new language features we
gain by setting a minimum Java version. This is (part of?) why we have such
minimums!
This nice `inst
s1monw commented on issue #12725:
URL: https://github.com/apache/lucene/issues/12725#issuecomment-1790655491
@robro612 please subscribe to the [dev
list](https://lucene.apache.org/core/discussion.html#developer-discussion-devluceneapacheorg)
and post your question there. We are more than ha
mikemccand commented on PR #12301:
URL: https://github.com/apache/lucene/pull/12301#issuecomment-1790672389
@gus-asf -- looks like this one is close? @uschindler had one more small
feeback (isolate the one line that requires suppression to its own method so we
don't suppress more than we n
mikemccand commented on PR #12306:
URL: https://github.com/apache/lucene/pull/12306#issuecomment-1790675801
We've since enabled Codec to set the limit, which is very expert and I think
a safer way to change the limit than a `sysprop`? So we can close this one?
--
This is an automated mes
mikemccand closed pull request #12306: Make MAX_DIMENSIONS configurable via a
system property.
URL: https://github.com/apache/lucene/pull/12306
--
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
jpountz commented on PR #11903:
URL: https://github.com/apache/lucene/pull/11903#issuecomment-1790680106
I confirmed that there is still a speedup:
```
TaskQPS baseline StdDevQPS
my_modified_version StdDevPct diff p-value
mikemccand closed pull request #12279: Add Setter for vector Encoding in
FieldType
URL: https://github.com/apache/lucene/pull/12279
--
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 comme
mikemccand commented on PR #12279:
URL: https://github.com/apache/lucene/pull/12279#issuecomment-1790681242
Thanks @naveentatikonda!
It seems strange to set vector dimensions to 0, and it looks like
`FieldType.setVectorAttribute` can otherwise be used to set the
`vectorEncoding`. I'
mikemccand commented on PR #11867:
URL: https://github.com/apache/lucene/pull/11867#issuecomment-1790686276
I love this idea of a "high scale" KNN monster test! It can catch overflow
exceptions that we otherwise miss, and @rmuir hit a spooky exception that might
be just such an example? @
jpountz merged PR #11903:
URL: https://github.com/apache/lucene/pull/11903
--
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 PR #11903:
URL: https://github.com/apache/lucene/pull/11903#issuecomment-1790706658
@mikemccand You will need to regold before the next nightly run.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
jprinet commented on PR #414:
URL: https://github.com/apache/lucene/pull/414#issuecomment-1790708355
As far as I remember, all the relevant changes were integrated in:
I think this one has been superseded by
https://github.com/apache/lucene/pull/421
See the Jira issue for more cont
mikemccand commented on PR #12412:
URL: https://github.com/apache/lucene/pull/12412#issuecomment-1790763295
Now that we are removing patching for the doc block encoding maybe
vectorizing decode of these blocks is more palatable?
--
This is an automated message from the Apache Git Service.
msokolov closed pull request #12373: require that float vector components are
smaller than 1E17 to prevent overflowing to Infinity
URL: https://github.com/apache/lucene/pull/12373
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
msokolov commented on PR #12373:
URL: https://github.com/apache/lucene/pull/12373#issuecomment-1790767056
it's not clear that we need this limit and it seems somewhat complicated to
maintain. I'm closing since we haven't seen any activity in quite a while and
there's no consensus to impose
dungba88 commented on code in PR #12738:
URL: https://github.com/apache/lucene/pull/12738#discussion_r1380008658
##
lucene/core/src/java/org/apache/lucene/util/fst/NodeHash.java:
##
@@ -328,7 +298,100 @@ private void rehash(long lastNodeAddress) throws
IOException {
}
mikemccand commented on code in PR #12433:
URL: https://github.com/apache/lucene/pull/12433#discussion_r1380176295
##
lucene/analysis/common/src/java/org/apache/lucene/analysis/synonym/word2vec/Word2VecSynonymFilter.java:
##
@@ -62,14 +65,16 @@ public Word2VecSynonymFilter(
clayburn commented on PR #414:
URL: https://github.com/apache/lucene/pull/414#issuecomment-1790798362
I agree with @jprinet that the PR should probably be closed just due to it's
age. Many of the changes here deal with caching, with the Lucene project
explicitly opts out of by default. If t
fsparv commented on PR #12301:
URL: https://github.com/apache/lucene/pull/12301#issuecomment-1790800773
Hmm, yeah I think I got busy and forgot about this. Will need to review
again. Thx for the nudge.
--
This is an automated message from the Apache Git Service.
To respond to the message,
mikemccand closed issue #12389: Need to resolve the duplicate CSVUtil classes
in analyzer Nori and Kuromoji
URL: https://github.com/apache/lucene/issues/12389
--
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
mikemccand commented on PR #12287:
URL: https://github.com/apache/lucene/pull/12287#issuecomment-1790817479
Thanks @heemin32 for taking the effort to bring the fix down to Lucene, from
OpenSearch test failures. A dedicated Lucene unit test would be great. Maybe
@nknize could help evaluate
mikemccand commented on PR #12390:
URL: https://github.com/apache/lucene/pull/12390#issuecomment-1790813760
I merged to main but there are quite a few conflicts on backport to 9.x --
any chance you could open a backport PR @twosom? Thanks!
--
This is an automated message from the Apache
mikemccand commented on PR #12446:
URL: https://github.com/apache/lucene/pull/12446#issuecomment-1790831094
Rank unsafe optimizations is a neat idea! It'd give another tool for maybe
more smoothly trading cost for recall.
--
This is an automated message from the Apache Git Service.
To re
mikemccand merged PR #12390:
URL: https://github.com/apache/lucene/pull/12390
--
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.
mikemccand commented on PR #12405:
URL: https://github.com/apache/lucene/pull/12405#issuecomment-1790852894
@LuXugang @jpountz it looks like this PR went through some great discussions
/ iterations and was close towards the end, but it has accumulated some
conflicts now?
--
This is an au
mikemccand merged PR #12495:
URL: https://github.com/apache/lucene/pull/12495
--
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.
mikemccand commented on issue #10309:
URL: https://github.com/apache/lucene/issues/10309#issuecomment-1790845238
> * while testing a solution for adding `perReaderTermState` to the current
`TermQuery#equals` implementation, I found a test that I believe is not doing
anything of what it was
s1monw commented on issue #12725:
URL: https://github.com/apache/lucene/issues/12725#issuecomment-1790864645
@gf2121 do we have any numbers if it actually helps applying deletes? I
think we can assume that we make use of `seekCeil` in the common case ie. all
terms have the same field. I wou
mikemccand commented on PR #12354:
URL: https://github.com/apache/lucene/pull/12354#issuecomment-1790844104
Thank you @rafalh! Query scores depending on `HashMap` iteration order is
really awful. And thank you @stefanvodita for reviewing. @rafalh do you want
to fold in the feedback maybe
s1monw commented on issue #12725:
URL: https://github.com/apache/lucene/issues/12725#issuecomment-1790887384
@gf2121 wild idea but would it make sense to build an automaton off these
terms and intersect it? We could reuse it for multiple segments? I am not sure
how big the costs are for tha
mikemccand commented on PR #12301:
URL: https://github.com/apache/lucene/pull/12301#issuecomment-1790965114
Thanks @fsparv.
--
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.
Deepika0510 commented on PR #12345:
URL: https://github.com/apache/lucene/pull/12345#issuecomment-1790951857
What I meant to ask is that after creating the `TimeoutLeafReader` class,
how would we make sure that this wrapped class's object is used instead of any
normal `LeafReader` instance?
zacharymorn commented on PR #240:
URL: https://github.com/apache/lucene/pull/240#issuecomment-1791103958
Thanks @mikemccand for reminding me on this PR, and sorry for missing your
question earlier @javanna ! This has totally fallen out of my radar. @javanna
Looking at the codebase, it seems
mikemccand commented on PR #12345:
URL: https://github.com/apache/lucene/pull/12345#issuecomment-1791127377
Hi @Deepika0510 -- what is the problem when callers access the leaves?
Since you would subclass `FilterLeafReader` (which subclasses `LeafReader`) it
should be fine to existing code?
lukas-vlcek opened a new pull request, #12750:
URL: https://github.com/apache/lucene/pull/12750
… PathHierarchy tokenizer
### Description
This PR is expected to fail. It demonstrates issue with
`BaseTokenStreamTestCase.assertAnalyzesTo()` method in connection to
`PathHierarchy
asfgit merged PR #12747:
URL: https://github.com/apache/lucene/pull/12747
--
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
uschindler commented on PR #12737:
URL: https://github.com/apache/lucene/pull/12737#issuecomment-1791239215
I will try to work on the proposed PR tomorrow (or maybe later this
evening). Sorry, I am very busy :-(
--
This is an automated message from the Apache Git Service.
To respond to th
JarvisCraft commented on PR #12295:
URL: https://github.com/apache/lucene/pull/12295#issuecomment-1791256716
@mikemccand, thanks for the comments!
I've undone the changes to `equals()` methods and applied the fix to the
remaining fixable occurrences of the pattern.
--
This is an au
msokolov merged PR #11833:
URL: https://github.com/apache/lucene/pull/11833
--
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.ap
iverase commented on PR #12295:
URL: https://github.com/apache/lucene/pull/12295#issuecomment-1791268281
can we keep explicit `== false` checks instead of less readable `!`?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
Deepika0510 commented on PR #12345:
URL: https://github.com/apache/lucene/pull/12345#issuecomment-1790965164
Came across `SoftDeletesDirectoryReaderWrapper` where we have wrap
[method](https://github.com/apache/lucene/blob/2d50c345fea3d1a64090d6d0cffef6b70d482a9f/lucene/core/src/java/org/apa
JarvisCraft commented on PR #12295:
URL: https://github.com/apache/lucene/pull/12295#issuecomment-1791269608
> can we keep explicit `== false` checks instead of less readable `!`?
No, since javac only recognizes `(!(EXPR instanceof TYPE NAME))`
--
This is an automated message from t
rmuir commented on PR #12737:
URL: https://github.com/apache/lucene/pull/12737#issuecomment-1791341717
No worries, I just wanted to merge in the benchmarking fixes so we can rely
upon the results.
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
dweiss commented on issue #12654:
URL: https://github.com/apache/lucene/issues/12654#issuecomment-1791471920
Well, this test is almost never "fast" for me... the conditions passed in
Failure.eval are frequently called, but rarely hit the right call stack - this
is particularly problematic w
gus-asf commented on PR #12301:
URL: https://github.com/apache/lucene/pull/12301#issuecomment-1791479559
> Thanks @fsparv.
heh whoops wrong browser instance ;)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
dweiss commented on issue #12654:
URL: https://github.com/apache/lucene/issues/12654#issuecomment-1791500183
You can reproduce this problem from the IDE as well:
```
-ea -Dtests.seed=4A059D04FCC8873 -Dtests.nightly=true -Dtests.multiplier=1
-Dtests.verbose=true
```
The last messa
dweiss commented on issue #12654:
URL: https://github.com/apache/lucene/issues/12654#issuecomment-1791510733
This may be a legitimate bug somewhere. Maybe @mikemccand or @s1monw will
know what the expected state here should be.
--
This is an automated message from the Apache Git Service.
dweiss commented on issue #12654:
URL: https://github.com/apache/lucene/issues/12654#issuecomment-1791521460
In fact, I think it's this block in IW:
```
// close all the closeables we can (but important is readerPool
and writeLock to prevent
// leaks)
nknize commented on code in PR #12688:
URL: https://github.com/apache/lucene/pull/12688#discussion_r1380840508
##
lucene/sandbox/src/java/org/apache/lucene/sandbox/codecs/lucene90/randomaccess/bitpacking/BitPacker.java:
##
Review Comment:
Looks like this is only used by tes
MarcusSorealheis commented on PR #940:
URL: https://github.com/apache/lucene/pull/940#issuecomment-1791704616
I can reawaken it and get it to closure. I need to carve out time on Sunday
unless someone else picks it up.
--
This is an automated message from the Apache Git Service.
To respon
goankur closed pull request #282: Lucene-10070
URL: https://github.com/apache/lucene/pull/282
--
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-uns
goankur commented on PR #282:
URL: https://github.com/apache/lucene/pull/282#issuecomment-1791762753
> @goankur this can be closed out now right since you opened a separate PR
for this change?
Yep this is correct. I am closing this PR.
--
This is an automated message from the Apach
easyice commented on PR #12748:
URL: https://github.com/apache/lucene/pull/12748#issuecomment-1791978766
I ran this with wikimedium10m and wikimediumall, There was no significant
performance improvement or regression that was found. The total size of tip has
a slight reduced:
|
jpountz commented on PR #12719:
URL: https://github.com/apache/lucene/pull/12719#issuecomment-1792001309
Interestingly, it seems to also help with facets:
http://people.apache.org/~mikemccand/lucenebench/AndHighHighDayTaxoFacets.html.
--
This is an automated message from the Apache Git Se
jpountz commented on PR #12719:
URL: https://github.com/apache/lucene/pull/12719#issuecomment-1792000689
This yielded a good speedup on [nightly
benchmarks](http://people.apache.org/~mikemccand/lucenebench/CountAndHighHigh.html).
I pushed an annotation.
--
This is an automated message f
jpountz merged PR #1052:
URL: https://github.com/apache/lucene/pull/1052
--
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
jpountz closed issue #11596: Can we speed up OrdinalMap construction?
[LUCENE-10560]
URL: https://github.com/apache/lucene/issues/11596
--
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 c
mikemccand commented on PR #1052:
URL: https://github.com/apache/lucene/pull/1052#issuecomment-1792099793
> +1 I fell a bit into a trap by trying to make long shared prefixes less
adversarial. Let's do progress over perfection and start with a simple approach
and look into whether/how we ca
s1monw commented on issue #12654:
URL: https://github.com/apache/lucene/issues/12654#issuecomment-1792101597
@dweiss I agree this is the problem. We should execute that
`IOUtils.closeWhileHandlingException(readerPool, deleter, writeLock);` in a try
/ finally block. I can open a PR for that
mikemccand commented on PR #12748:
URL: https://github.com/apache/lucene/pull/12748#issuecomment-1792198963
I tested this PR using `IndexToFST` from `luceneutil`. This just tests
construction time and final FST size, on all `wikimediumall` unique terms,
allowing up to 64 MB RAM while build
mikemccand commented on PR #12748:
URL: https://github.com/apache/lucene/pull/12748#issuecomment-1792202492
I'll run `Test2BFST` too ... takes a few hours!
--
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
dweiss commented on issue #12654:
URL: https://github.com/apache/lucene/issues/12654#issuecomment-1792205263
Thanks, Simon. I'll open up a PR.
--
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
mikemccand commented on code in PR #12748:
URL: https://github.com/apache/lucene/pull/12748#discussion_r1381468008
##
lucene/core/src/java/org/apache/lucene/util/fst/FSTCompiler.java:
##
@@ -444,9 +446,15 @@ long addNode(FSTCompiler.UnCompiledNode nodeIn) throws
IOException {
dweiss opened a new pull request, #12751:
URL: https://github.com/apache/lucene/pull/12751
Fixes #12654.
--
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-m
1901 - 2000 of 22924 matches
Mail list logo
