[
https://issues.apache.org/jira/browse/LUCENE-8352?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16618834#comment-16618834
]
Adrien Grand commented on LUCENE-8352:
--
+1 Great to see that the patch is simpler than I expected.
[
https://issues.apache.org/jira/browse/LUCENE-8482?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16618121#comment-16618121
]
Adrien Grand commented on LUCENE-8482:
--
bq. New patch where we approximate using a box query which
[
https://issues.apache.org/jira/browse/LUCENE-8505?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16617770#comment-16617770
]
Adrien Grand commented on LUCENE-8505:
--
+1 You only plan to merge this to master, right? We still
[
https://issues.apache.org/jira/browse/LUCENE-8501?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16617541#comment-16617541
]
Adrien Grand commented on LUCENE-8501:
--
bq. In my case I'd be a bit worried about the loss of
[
https://issues.apache.org/jira/browse/LUCENE-8503?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16617534#comment-16617534
]
Adrien Grand commented on LUCENE-8503:
--
+1
> Simplify unwrapping Filter*Reader
>
[
https://issues.apache.org/jira/browse/LUCENE-8503?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16617289#comment-16617289
]
Adrien Grand commented on LUCENE-8503:
--
I'm a bit on the fence about adding such an API to ease
[
https://issues.apache.org/jira/browse/LUCENE-8501?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16617272#comment-16617272
]
Adrien Grand commented on LUCENE-8501:
--
Storing floats as term frequencies not only raises issues
[
https://issues.apache.org/jira/browse/LUCENE-8502?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16617212#comment-16617212
]
Adrien Grand commented on LUCENE-8502:
--
+1
> Allow access to delegate in FilterCodecReader
>
[
https://issues.apache.org/jira/browse/LUCENE-8498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16614980#comment-16614980
]
Adrien Grand commented on LUCENE-8498:
--
+1
> Deprecate/Remove LowerCaseTokenizer
>
[
https://issues.apache.org/jira/browse/LUCENE-8490?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16614918#comment-16614918
]
Adrien Grand commented on LUCENE-8490:
--
One thing that annoys me a bit with the patch is that you
[
https://issues.apache.org/jira/browse/LUCENE-8490?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16614917#comment-16614917
]
Adrien Grand commented on LUCENE-8490:
--
One thing that annoys me a bit with the patch is that you
[
https://issues.apache.org/jira/browse/LUCENE-8496?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16614910#comment-16614910
]
Adrien Grand commented on LUCENE-8496:
--
It is a pity that the patch is so large given that the
[
https://issues.apache.org/jira/browse/LUCENE-8489?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16608565#comment-16608565
]
Adrien Grand commented on LUCENE-8489:
--
bq. but users will think positively if the API supports
[
https://issues.apache.org/jira/browse/LUCENE-8482?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16607126#comment-16607126
]
Adrien Grand commented on LUCENE-8482:
--
Indeed I had misread!
> Boosting by geo distance
>
[
https://issues.apache.org/jira/browse/LUCENE-8482?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16607101#comment-16607101
]
Adrien Grand commented on LUCENE-8482:
--
This looks promising! There seems to be an issue with
[
https://issues.apache.org/jira/browse/LUCENE-7862?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16605853#comment-16605853
]
Adrien Grand commented on LUCENE-7862:
--
The improvement in QPS look indeed very significant in some
[
https://issues.apache.org/jira/browse/LUCENE-8478?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16605745#comment-16605745
]
Adrien Grand commented on LUCENE-8478:
--
bq. so not sure if this is worth pursuing further or not.
[
https://issues.apache.org/jira/browse/LUCENE-8342?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Adrien Grand reopened LUCENE-8342:
--
Reopen: was closed by mistake.
> Can we enforce more properties on fields like uniquness
>
[
https://issues.apache.org/jira/browse/LUCENE-8481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16604268#comment-16604268
]
Adrien Grand commented on LUCENE-8481:
--
This patch still looks like a net improvement to me by
[
https://issues.apache.org/jira/browse/LUCENE-8483?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16604118#comment-16604118
]
Adrien Grand commented on LUCENE-8483:
--
bq. I renamed FakeScorable to ScoreAndDoc
+1 let's also
[
https://issues.apache.org/jira/browse/LUCENE-8484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16603095#comment-16603095
]
Adrien Grand commented on LUCENE-8484:
--
I see you just made it hard coded. +1
> Allow
[
https://issues.apache.org/jira/browse/LUCENE-8484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16603079#comment-16603079
]
Adrien Grand commented on LUCENE-8484:
--
What happens when a segment gets removed, will there be a
[
https://issues.apache.org/jira/browse/LUCENE-8483?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16603002#comment-16603002
]
Adrien Grand commented on LUCENE-8483:
--
Can we somehow avoid the FakeWeight in ValuesSourceScorer?
[
https://issues.apache.org/jira/browse/LUCENE-8340?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Adrien Grand resolved LUCENE-8340.
--
Resolution: Fixed
Fix Version/s: master (8.0)
> Efficient boosting by recency
>
[
https://issues.apache.org/jira/browse/LUCENE-6247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16602960#comment-16602960
]
Adrien Grand commented on LUCENE-6247:
--
I'm decreasing the priority from blocker to major since
[
https://issues.apache.org/jira/browse/LUCENE-6247?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Adrien Grand updated LUCENE-6247:
-
Priority: Major (was: Blocker)
> artifacts are half a gigabyte
> -
Adrien Grand created LUCENE-8482:
Summary: Boosting by geo distance
Key: LUCENE-8482
URL: https://issues.apache.org/jira/browse/LUCENE-8482
Project: Lucene - Core
Issue Type: New Feature
[
https://issues.apache.org/jira/browse/LUCENE-8481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16602837#comment-16602837
]
Adrien Grand commented on LUCENE-8481:
--
OK fair enough.
> Javadocs should no longer reference
[
https://issues.apache.org/jira/browse/LUCENE-8340?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16602831#comment-16602831
]
Adrien Grand commented on LUCENE-8340:
--
Good call, the way that the leadCost was leveraged was not
[
https://issues.apache.org/jira/browse/LUCENE-6228?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16602771#comment-16602771
]
Adrien Grand commented on LUCENE-6228:
--
I think FilterScorable should not delegate
[
https://issues.apache.org/jira/browse/LUCENE-8481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16602711#comment-16602711
]
Adrien Grand commented on LUCENE-8481:
--
I saw LUCENE-8474 but was thinking that we do not only need
[
https://issues.apache.org/jira/browse/SOLR-12074?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16602451#comment-16602451
]
Adrien Grand commented on SOLR-12074:
-
bq. I think Lucene allows one field to have a Terms index &
Adrien Grand created LUCENE-8481:
Summary: Javadocs should no longer reference RAMDirectory
Key: LUCENE-8481
URL: https://issues.apache.org/jira/browse/LUCENE-8481
Project: Lucene - Core
[
https://issues.apache.org/jira/browse/LUCENE-8476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16602449#comment-16602449
]
Adrien Grand commented on LUCENE-8476:
--
The problem with calling BufferedReader#close is that it
[
https://issues.apache.org/jira/browse/LUCENE-8340?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Adrien Grand updated LUCENE-8340:
-
Summary: Efficient boosting by recency (was: Allow to boost by recency)
> Efficient boosting
[
https://issues.apache.org/jira/browse/LUCENE-8478?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16602446#comment-16602446
]
Adrien Grand commented on LUCENE-8478:
--
I'm not sure this is a net win since it requires the
[
https://issues.apache.org/jira/browse/LUCENE-8340?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16602299#comment-16602299
]
Adrien Grand commented on LUCENE-8340:
--
So I went back to this patch and did some testing. I played
[
https://issues.apache.org/jira/browse/LUCENE-8422?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16602273#comment-16602273
]
Adrien Grand commented on LUCENE-8422:
--
+1
> Add Matches iteration to interval queries
>
[
https://issues.apache.org/jira/browse/LUCENE-8477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16602195#comment-16602195
]
Adrien Grand commented on LUCENE-8477:
--
I'd be tempted to just document this behavior for now. I'm
[
https://issues.apache.org/jira/browse/LUCENE-8422?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16602180#comment-16602180
]
Adrien Grand commented on LUCENE-8422:
--
I was thinking of something like having a MatchesUtils
[
https://issues.apache.org/jira/browse/LUCENE-8422?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16602123#comment-16602123
]
Adrien Grand commented on LUCENE-8422:
--
bq. All sources return null from MatchesIterator#getQuery()
[
https://issues.apache.org/jira/browse/LUCENE-8422?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16602118#comment-16602118
]
Adrien Grand commented on LUCENE-8422:
--
+1 to the patch in general, only two minor comments:
-
[
https://issues.apache.org/jira/browse/LUCENE-6228?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16602097#comment-16602097
]
Adrien Grand commented on LUCENE-6228:
--
+1 to this patch, let's also make sure we add a note about
[
https://issues.apache.org/jira/browse/SOLR-12535?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16601678#comment-16601678
]
Adrien Grand commented on SOLR-12535:
-
+1 to remove support for indexing boosts entirely in 8.0
>
[
https://issues.apache.org/jira/browse/LUCENE-8475?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16598952#comment-16598952
]
Adrien Grand commented on LUCENE-8475:
--
+1 I'll commit next week. Thanks [~dathanasiou]!
> Remove
[
https://issues.apache.org/jira/browse/LUCENE-8472?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16598778#comment-16598778
]
Adrien Grand commented on LUCENE-8472:
--
+1
> Soft-deletes merge retention query should be
[
https://issues.apache.org/jira/browse/LUCENE-8432?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Adrien Grand resolved LUCENE-8432.
--
Resolution: Fixed
Fix Version/s: master (8.0)
7.5
Merged. Thanks
[
https://issues.apache.org/jira/browse/LUCENE-8465?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Adrien Grand resolved LUCENE-8465.
--
Resolution: Fixed
Fix Version/s: master (8.0)
> Remove legacy auto-prefix logic from
[
https://issues.apache.org/jira/browse/LUCENE-765?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Adrien Grand resolved LUCENE-765.
-
Resolution: Fixed
Fix Version/s: master (8.0)
7.5
I was still
[
https://issues.apache.org/jira/browse/LUCENE-8267?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16597132#comment-16597132
]
Adrien Grand commented on LUCENE-8267:
--
+1 to recommend defaults
> Remove memory codecs from the
[
https://issues.apache.org/jira/browse/LUCENE-8471?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596581#comment-16596581
]
Adrien Grand commented on LUCENE-8471:
--
+1 Let's maybe mention in javadocs that this is a subset of
[
https://issues.apache.org/jira/browse/LUCENE-8470?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596541#comment-16596541
]
Adrien Grand commented on LUCENE-8470:
--
Here is a patch. Instead of removing the legacy classes, I
[
https://issues.apache.org/jira/browse/LUCENE-8470?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Adrien Grand updated LUCENE-8470:
-
Attachment: LUCENE-8470.patch
> Remove Legacy*DocValues classes
>
[
https://issues.apache.org/jira/browse/LUCENE-8470?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Adrien Grand updated LUCENE-8470:
-
Attachment: (was: LUCENE-8470.patch)
> Remove Legacy*DocValues classes
>
[
https://issues.apache.org/jira/browse/LUCENE-8470?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Adrien Grand updated LUCENE-8470:
-
Attachment: LUCENE-8470.patch
> Remove Legacy*DocValues classes
>
Adrien Grand created LUCENE-8470:
Summary: Remove Legacy*DocValues classes
Key: LUCENE-8470
URL: https://issues.apache.org/jira/browse/LUCENE-8470
Project: Lucene - Core
Issue Type: Task
[
https://issues.apache.org/jira/browse/LUCENE-8469?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596332#comment-16596332
]
Adrien Grand commented on LUCENE-8469:
--
Let's also remove StringHelper#compare on master? The patch
[
https://issues.apache.org/jira/browse/LUCENE-765?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596326#comment-16596326
]
Adrien Grand commented on LUCENE-765:
-
[~sokolov] I have precommit failures with this patch which
[
https://issues.apache.org/jira/browse/LUCENE-8466?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16596122#comment-16596122
]
Adrien Grand commented on LUCENE-8466:
--
Maybe we should start thinking about releasing 7.5 because
[
https://issues.apache.org/jira/browse/LUCENE-8460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16593741#comment-16593741
]
Adrien Grand commented on LUCENE-8460:
--
Looking further at the patch, the "throws
[
https://issues.apache.org/jira/browse/LUCENE-8466?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16593652#comment-16593652
]
Adrien Grand commented on LUCENE-8466:
--
Maybe rename the test since it doesn't only test delete by
[
https://issues.apache.org/jira/browse/LUCENE-8374?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16593644#comment-16593644
]
Adrien Grand commented on LUCENE-8374:
--
Indeed I mentioned skip lists because this is the natural
Adrien Grand created LUCENE-8466:
Summary: FrozenBufferedUpdates#apply*Deletes is incorrect when
index sorting is enabled
Key: LUCENE-8466
URL: https://issues.apache.org/jira/browse/LUCENE-8466
[
https://issues.apache.org/jira/browse/LUCENE-765?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16591790#comment-16591790
]
Adrien Grand commented on LUCENE-765:
-
Unfortunately the patch doesn't apply cleanly, [~sokolov] did
[
https://issues.apache.org/jira/browse/LUCENE-8432?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16591556#comment-16591556
]
Adrien Grand commented on LUCENE-8432:
--
I updated the patch to apply on a recent master and renamed
[
https://issues.apache.org/jira/browse/LUCENE-8432?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Adrien Grand updated LUCENE-8432:
-
Attachment: LUCENE-8432.patch
> Stop calling comparator even if early termination is not
[
https://issues.apache.org/jira/browse/LUCENE-8465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16591515#comment-16591515
]
Adrien Grand commented on LUCENE-8465:
--
Good call, I'll remove them when pushing.
> Remove legacy
[
https://issues.apache.org/jira/browse/LUCENE-765?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16591505#comment-16591505
]
Adrien Grand commented on LUCENE-765:
-
+1 I'll merge. Thanks [~sokolov]!
> Index package level
[
https://issues.apache.org/jira/browse/LUCENE-8432?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16591453#comment-16591453
]
Adrien Grand commented on LUCENE-8432:
--
Got it, thanks. There have been a couple of changes to
[
https://issues.apache.org/jira/browse/LUCENE-8374?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16591452#comment-16591452
]
Adrien Grand commented on LUCENE-8374:
--
The above numbers are very impressive! Do you have a sense
[
https://issues.apache.org/jira/browse/LUCENE-8422?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16591393#comment-16591393
]
Adrien Grand commented on LUCENE-8422:
--
I see how putting the logic in IntervalIterator makes
[
https://issues.apache.org/jira/browse/LUCENE-8460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16591299#comment-16591299
]
Adrien Grand commented on LUCENE-8460:
--
bq. In fact, I think the null check is missing.
+1 Let's
[
https://issues.apache.org/jira/browse/LUCENE-765?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16591281#comment-16591281
]
Adrien Grand commented on LUCENE-765:
-
Thanks Mike, it looks great. Here are some suggestions:
- I'm
[
https://issues.apache.org/jira/browse/LUCENE-8461?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16590831#comment-16590831
]
Adrien Grand commented on LUCENE-8461:
--
Woops, thanks Steve!
> Add a Lucene80Codec
>
[
https://issues.apache.org/jira/browse/LUCENE-8465?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Adrien Grand updated LUCENE-8465:
-
Attachment: LUCENE-8465.patch
> Remove legacy auto-prefix logic from IntersectTermsEnum
>
[
https://issues.apache.org/jira/browse/LUCENE-8465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16590389#comment-16590389
]
Adrien Grand commented on LUCENE-8465:
--
Here is a patch.
> Remove legacy auto-prefix logic from
Adrien Grand created LUCENE-8465:
Summary: Remove legacy auto-prefix logic from IntersectTermsEnum
Key: LUCENE-8465
URL: https://issues.apache.org/jira/browse/LUCENE-8465
Project: Lucene - Core
[
https://issues.apache.org/jira/browse/LUCENE-8461?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Adrien Grand resolved LUCENE-8461.
--
Resolution: Fixed
Fix Version/s: master (8.0)
> Add a Lucene80Codec
>
Adrien Grand created LUCENE-8464:
Summary: Implement ConstantScoreScorer#setMinCompetitiveScore
Key: LUCENE-8464
URL: https://issues.apache.org/jira/browse/LUCENE-8464
Project: Lucene - Core
Adrien Grand created LUCENE-8463:
Summary: Early-terminate queries sorted by SortField.DOC
Key: LUCENE-8463
URL: https://issues.apache.org/jira/browse/LUCENE-8463
Project: Lucene - Core
[
https://issues.apache.org/jira/browse/LUCENE-8461?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16587683#comment-16587683
]
Adrien Grand commented on LUCENE-8461:
--
Here is a patch. Lucene70Codec moved to the backward-codecs
[
https://issues.apache.org/jira/browse/LUCENE-8461?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Adrien Grand updated LUCENE-8461:
-
Attachment: LUCENE-8461.patch
> Add a Lucene80Codec
> ---
>
>
Adrien Grand created LUCENE-8461:
Summary: Add a Lucene80Codec
Key: LUCENE-8461
URL: https://issues.apache.org/jira/browse/LUCENE-8461
Project: Lucene - Core
Issue Type: Improvement
[
https://issues.apache.org/jira/browse/LUCENE-8448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16586005#comment-16586005
]
Adrien Grand commented on LUCENE-8448:
--
We now have in impressive speedup on AndHighOrMedMed
[
https://issues.apache.org/jira/browse/LUCENE-8448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16580100#comment-16580100
]
Adrien Grand commented on LUCENE-8448:
--
+1 Maybe we need to special case minCompetitiveScore == 0
[
https://issues.apache.org/jira/browse/LUCENE-8340?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16579796#comment-16579796
]
Adrien Grand commented on LUCENE-8340:
--
bq. we need to do some changes to make boolean queries as
[
https://issues.apache.org/jira/browse/LUCENE-8448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16579774#comment-16579774
]
Adrien Grand commented on LUCENE-8448:
--
Looks great. Let's add a unit test with fake scorers to
[
https://issues.apache.org/jira/browse/LUCENE-8452?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16579481#comment-16579481
]
Adrien Grand commented on LUCENE-8452:
--
This looks like a nice dataset for benchmarking indeed!
>
[
https://issues.apache.org/jira/browse/LUCENE-8432?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578682#comment-16578682
]
Adrien Grand commented on LUCENE-8432:
--
[~khitrin] Could you add a test for these changes? I could
[
https://issues.apache.org/jira/browse/LUCENE-8452?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578650#comment-16578650
]
Adrien Grand commented on LUCENE-8452:
--
bq. What it actually concerns me is that the dimensions are
[
https://issues.apache.org/jira/browse/LUCENE-8449?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578488#comment-16578488
]
Adrien Grand commented on LUCENE-8449:
--
If you share a patch created with git-format-patch, we can
[
https://issues.apache.org/jira/browse/LUCENE-8449?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16578369#comment-16578369
]
Adrien Grand commented on LUCENE-8449:
--
You can safely ignore the automated message!
> Code
[
https://issues.apache.org/jira/browse/LUCENE-8447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16573516#comment-16573516
]
Adrien Grand commented on LUCENE-8447:
--
+1 I'd just keep the transpose() method as an impl detail
[
https://issues.apache.org/jira/browse/LUCENE-8449?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16573460#comment-16573460
]
Adrien Grand commented on LUCENE-8449:
--
Changes 2, 3 and 4 look good to me. Can you revert 1 in
[
https://issues.apache.org/jira/browse/LUCENE-8204?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16573112#comment-16573112
]
Adrien Grand commented on LUCENE-8204:
--
+1
> ReqOptSumScorer should leverage sub scorers'
[
https://issues.apache.org/jira/browse/LUCENE-8204?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16572971#comment-16572971
]
Adrien Grand commented on LUCENE-8204:
--
+1 in general, some minor comments below:
{code}
[
https://issues.apache.org/jira/browse/LUCENE-8439?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16572938#comment-16572938
]
Adrien Grand commented on LUCENE-8439:
--
FYI the patch includes some unintended changes in the solr/
[
https://issues.apache.org/jira/browse/LUCENE-8439?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16572934#comment-16572934
]
Adrien Grand commented on LUCENE-8439:
--
+1
> DisjunctionMaxScorer should leverage sub scorers'
[
https://issues.apache.org/jira/browse/LUCENE-8439?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16571970#comment-16571970
]
Adrien Grand commented on LUCENE-8439:
--
I'm curious why you introduced {{wrapTopScoresApprox}}
[
https://issues.apache.org/jira/browse/LUCENE-8204?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16571947#comment-16571947
]
Adrien Grand commented on LUCENE-8204:
--
Thanks Jim! It looks great in general, I'm just a bit
501 - 600 of 4980 matches
Mail list logo