mocobeta commented on code in PR #941:
URL: https://github.com/apache/lucene/pull/941#discussion_r887625886
##
buildSrc/src/main/java/org/apache/lucene/gradle/WrapperDownloader.java:
##
@@ -55,9 +64,18 @@ public static void main(String[] args) {
}
public static void chec
mocobeta commented on code in PR #941:
URL: https://github.com/apache/lucene/pull/941#discussion_r887625886
##
buildSrc/src/main/java/org/apache/lucene/gradle/WrapperDownloader.java:
##
@@ -55,9 +64,18 @@ public static void main(String[] args) {
}
public static void chec
kaivalnp commented on code in PR #932:
URL: https://github.com/apache/lucene/pull/932#discussion_r887637829
##
lucene/core/src/java/org/apache/lucene/search/KnnVectorQuery.java:
##
@@ -225,6 +225,11 @@ public BitSetIterator getIterator(int contextOrd) {
return new BitSetI
dweiss commented on code in PR #941:
URL: https://github.com/apache/lucene/pull/941#discussion_r887650993
##
buildSrc/src/main/java/org/apache/lucene/gradle/WrapperDownloader.java:
##
@@ -64,15 +57,13 @@ public static void main(String[] args) {
}
public static void check
[
https://issues.apache.org/jira/browse/LUCENE-10598?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17545309#comment-17545309
]
ASF subversion and git services commented on LUCENE-10598:
--
Co
[
https://issues.apache.org/jira/browse/LUCENE-10598?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17545308#comment-17545308
]
ASF subversion and git services commented on LUCENE-10598:
--
Co
[
https://issues.apache.org/jira/browse/LUCENE-10598?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17545313#comment-17545313
]
Adrien Grand commented on LUCENE-10598:
---
[~ChrisLu] I'm noticing that some implem
[
https://issues.apache.org/jira/browse/LUCENE-10598?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17545318#comment-17545318
]
Lu Xugang commented on LUCENE-10598:
[~jpountz] Exactly, I left some comment in
kaivalnp commented on code in PR #932:
URL: https://github.com/apache/lucene/pull/932#discussion_r887687260
##
lucene/core/src/java/org/apache/lucene/search/KnnVectorQuery.java:
##
@@ -225,6 +225,11 @@ public BitSetIterator getIterator(int contextOrd) {
return new BitSetI
mocobeta commented on PR #941:
URL: https://github.com/apache/lucene/pull/941#issuecomment-1144595521
I have clarified the pull request title for a heads-up.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
[
https://issues.apache.org/jira/browse/LUCENE-10598?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17545356#comment-17545356
]
Lu Xugang commented on LUCENE-10598:
{quote}Maybe we should enhance CheckIndex to
[
https://issues.apache.org/jira/browse/LUCENE-10598?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17545357#comment-17545357
]
Adrien Grand commented on LUCENE-10598:
---
+1
> SortedSetDocValues#docValueCount()
LuXugang opened a new pull request, #942:
URL: https://github.com/apache/lucene/pull/942
Add a count used to record docValueCount in
Lucene80DocValuesProducer#getSortedSet, similar to what SortedNumericDocValues
did.
--
This is an automated message from the Apache Git Service.
To respond
Deepika0510 commented on PR #927:
URL: https://github.com/apache/lucene/pull/927#issuecomment-1144700233
> I'm not convinced it's best to add a `timeAllowed` to `search` methods.
This is going to be invasive if we want to support timeouts on all
IndexSearcher methods?
>
> My suggesti
mocobeta commented on PR #940:
URL: https://github.com/apache/lucene/pull/940#issuecomment-1144837827
Thanks for taking look at this. The change makes sense and looks consistent
in the usage of TFIDFSimilarity to me.
I think the problem is that we have no test corpus to measure the MLT se
mocobeta commented on PR #940:
URL: https://github.com/apache/lucene/pull/940#issuecomment-1144848214
I'm using MLT somewhere else, I'll try to apply this change there and see
the result with real data if I have a chance. Meanwhile, other people who are
more confident with this change can m
mocobeta opened a new pull request, #943:
URL: https://github.com/apache/lucene/pull/943
### Description (or a Jira issue link if you have one)
This applies the same change in #941 to branch_9x. Cherry-picking from main
may not work (we don't have `checkVersion()` in 9x).
--
This i
mocobeta commented on PR #943:
URL: https://github.com/apache/lucene/pull/943#issuecomment-1144878824
--
This is an automated message from the A
rmuir commented on code in PR #943:
URL: https://github.com/apache/lucene/pull/943#discussion_r887976742
##
buildSrc/src/main/java/org/apache/lucene/gradle/WrapperDownloader.java:
##
@@ -39,20 +39,33 @@
* Has no dependencies outside of standard java libraries
*/
public clas
[
https://issues.apache.org/jira/browse/LUCENE-10599?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17545487#comment-17545487
]
Michael Sokolov commented on LUCENE-10599:
--
I don't have any deep understandin
mocobeta commented on code in PR #943:
URL: https://github.com/apache/lucene/pull/943#discussion_r887985781
##
buildSrc/src/main/java/org/apache/lucene/gradle/WrapperDownloader.java:
##
@@ -39,20 +39,33 @@
* Has no dependencies outside of standard java libraries
*/
public c
msokolov commented on code in PR #932:
URL: https://github.com/apache/lucene/pull/932#discussion_r887991586
##
lucene/core/src/java/org/apache/lucene/search/KnnVectorQuery.java:
##
@@ -225,6 +225,11 @@ public BitSetIterator getIterator(int contextOrd) {
return new BitSetI
rmuir commented on code in PR #943:
URL: https://github.com/apache/lucene/pull/943#discussion_r887992569
##
buildSrc/src/main/java/org/apache/lucene/gradle/WrapperDownloader.java:
##
@@ -39,20 +39,33 @@
* Has no dependencies outside of standard java libraries
*/
public clas
pminkov commented on PR #940:
URL: https://github.com/apache/lucene/pull/940#issuecomment-1144930676
Thanks for taking a look @mocobeta - your comment makes sense. I have a
dataset and have noticed the problem on it. I'll create a short analysis with
examples.
--
This is an automated mes
mocobeta commented on PR #941:
URL: https://github.com/apache/lucene/pull/941#issuecomment-1144940195
I made it possible to specify minimum and maximum feature versions. It is
nonsense in `main` for now but we'd need it for branch_9x as @rmuir pointed out
in https://github.com/apache/lucen
rmuir commented on PR #940:
URL: https://github.com/apache/lucene/pull/940#issuecomment-1144943399
> I think the problem is that we have no test corpus to measure the MLT
search quality, so we can't directly know if taking square roots of raw term
frequency improves the search quality. I'm
mikemccand commented on PR #633:
URL: https://github.com/apache/lucene/pull/633#issuecomment-1145016933
> > I could either wrap the runningMerges update with a synchronized
(IndexWriter.this) {}, or make runningMerges a synchronizedSet. I like the
second approach as it automatically fixes t
[
https://issues.apache.org/jira/browse/LUCENE-10236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17545527#comment-17545527
]
David Smiley commented on LUCENE-10236:
---
If this is a "improvement", then I think
msokolov closed pull request #913: Lucene 10577
URL: https://github.com/apache/lucene/pull/913
--
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-un
jtibshirani commented on code in PR #932:
URL: https://github.com/apache/lucene/pull/932#discussion_r888237781
##
lucene/core/src/java/org/apache/lucene/search/KnnVectorQuery.java:
##
@@ -225,6 +225,11 @@ public BitSetIterator getIterator(int contextOrd) {
return new BitS
jtibshirani commented on code in PR #932:
URL: https://github.com/apache/lucene/pull/932#discussion_r888237781
##
lucene/core/src/java/org/apache/lucene/search/KnnVectorQuery.java:
##
@@ -225,6 +225,11 @@ public BitSetIterator getIterator(int contextOrd) {
return new BitS
jtibshirani commented on code in PR #932:
URL: https://github.com/apache/lucene/pull/932#discussion_r888247099
##
lucene/core/src/java/org/apache/lucene/search/KnnVectorQuery.java:
##
@@ -225,6 +225,11 @@ public BitSetIterator getIterator(int contextOrd) {
return new BitS
pminkov commented on PR #940:
URL: https://github.com/apache/lucene/pull/940#issuecomment-1145379881
I created a branch with some analysis of what happens, it's
[here](https://github.com/pminkov/lucene/commit/25c5ea4c12d92b8f534d40e449509a327ab6eea9).
The code is a bit hacky, sorry.
mocobeta commented on code in PR #927:
URL: https://github.com/apache/lucene/pull/927#discussion_r888567989
##
lucene/core/src/java/org/apache/lucene/search/TimeLimitingBulkScorer.java:
##
@@ -0,0 +1,66 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or mor
mocobeta commented on code in PR #927:
URL: https://github.com/apache/lucene/pull/927#discussion_r888569668
##
lucene/core/src/java/org/apache/lucene/search/TimeLimitingBulkScorer.java:
##
@@ -0,0 +1,66 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or mor
mocobeta commented on PR #927:
URL: https://github.com/apache/lucene/pull/927#issuecomment-1145541063
> I would like to understand the performance impact of this wrapping. If you
have time, it would be great to get some results from
https://github.com/mikemccand/luceneutil benchmarking util
36 matches
Mail list logo
