LuXugang commented on a change in pull request #736:
URL: https://github.com/apache/lucene/pull/736#discussion_r826515036
##
File path:
lucene/sandbox/src/java/org/apache/lucene/sandbox/search/IndexSortSortedNumericDocValuesRangeQuery.java
##
@@ -198,16 +198,22 @@ public boole
[
https://issues.apache.org/jira/browse/LUCENE-10467?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Yuting Gan updated LUCENE-10467:
Description:
Currently, when calling getAllDims(0) in SortedSetDocValuesFacetCounts, it
would th
[
https://issues.apache.org/jira/browse/LUCENE-10467?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Yuting Gan updated LUCENE-10467:
Summary: Throws IllegalArgumentException for getAllDims if topN <= 0 (was:
Throws IllegalArgumen
Yuting Gan created LUCENE-10467:
---
Summary: Throws IllegalArgumentException for getAllDims(0)
Key: LUCENE-10467
URL: https://issues.apache.org/jira/browse/LUCENE-10467
Project: Lucene - Core
Iss
[
https://issues.apache.org/jira/browse/LUCENE-10467?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Yuting Gan updated LUCENE-10467:
Description:
Currently, when calling getAllDims(0) in SortedSetDocValuesFacetCounts, it
would th
[
https://issues.apache.org/jira/browse/LUCENE-10325?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506640#comment-17506640
]
Yuting Gan edited comment on LUCENE-10325 at 3/15/22, 12:25 AM:
-
[
https://issues.apache.org/jira/browse/LUCENE-10325?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506640#comment-17506640
]
Yuting Gan edited comment on LUCENE-10325 at 3/15/22, 12:22 AM:
-
[
https://issues.apache.org/jira/browse/LUCENE-10325?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506640#comment-17506640
]
Yuting Gan commented on LUCENE-10325:
-
Thanks [~gsmiller] for creating issue!
I pr
Yuti-G commented on a change in pull request #747:
URL: https://github.com/apache/lucene/pull/747#discussion_r826461610
##
File path:
lucene/facet/src/test/org/apache/lucene/facet/sortedset/TestSortedSetDocValuesFacets.java
##
@@ -104,6 +104,65 @@ public void testBasic() throw
Yuti-G commented on a change in pull request #747:
URL: https://github.com/apache/lucene/pull/747#discussion_r82646
##
File path:
lucene/facet/src/test/org/apache/lucene/facet/sortedset/TestSortedSetDocValuesFacets.java
##
@@ -104,6 +104,65 @@ public void testBasic() throw
Yuti-G commented on a change in pull request #747:
URL: https://github.com/apache/lucene/pull/747#discussion_r826461610
##
File path:
lucene/facet/src/test/org/apache/lucene/facet/sortedset/TestSortedSetDocValuesFacets.java
##
@@ -104,6 +104,65 @@ public void testBasic() throw
Yuti-G commented on a change in pull request #747:
URL: https://github.com/apache/lucene/pull/747#discussion_r826456758
##
File path:
lucene/facet/src/test/org/apache/lucene/facet/sortedset/TestSortedSetDocValuesFacets.java
##
@@ -104,6 +104,65 @@ public void testBasic() throw
Yuti-G commented on a change in pull request #747:
URL: https://github.com/apache/lucene/pull/747#discussion_r826456302
##
File path:
lucene/facet/src/test/org/apache/lucene/facet/range/TestRangeFacetCounts.java
##
@@ -243,6 +243,24 @@ public void testLongGetAllDims() throws E
Yuti-G commented on a change in pull request #747:
URL: https://github.com/apache/lucene/pull/747#discussion_r826455347
##
File path: lucene/facet/src/java/org/apache/lucene/facet/Facets.java
##
@@ -48,4 +48,13 @@ public abstract FacetResult getTopChildren(int topN, String
dim
Yuti-G edited a comment on pull request #747:
URL: https://github.com/apache/lucene/pull/747#issuecomment-1067392546
> Thanks for picking this up! I've looked at everything except for your
overridden implementation in SSDV faceting, but since I may run out of time to
look at that today, I'
Yuti-G commented on pull request #747:
URL: https://github.com/apache/lucene/pull/747#issuecomment-1067392546
> Thanks for picking this up! I've looked at everything except for your
overridden implementation in SSDV faceting, but since I may run out of time to
look at that today, I'll go a
gsmiller commented on a change in pull request #747:
URL: https://github.com/apache/lucene/pull/747#discussion_r826440044
##
File path: lucene/facet/src/java/org/apache/lucene/facet/Facets.java
##
@@ -48,4 +48,13 @@ public abstract FacetResult getTopChildren(int topN, String
d
[
https://issues.apache.org/jira/browse/LUCENE-10466?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506548#comment-17506548
]
Andriy Redko commented on LUCENE-10466:
---
Thanks a lot, [~julietibs] , working on
[
https://issues.apache.org/jira/browse/LUCENE-10466?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Andriy Redko updated LUCENE-10466:
--
Description:
We have run into this issue while migrating to OpenSearch and making changes to
[
https://issues.apache.org/jira/browse/LUCENE-10458?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506532#comment-17506532
]
ASF subversion and git services commented on LUCENE-10458:
--
Co
[
https://issues.apache.org/jira/browse/LUCENE-10385?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506531#comment-17506531
]
ASF subversion and git services commented on LUCENE-10385:
--
Co
jtibshirani merged pull request #745:
URL: https://github.com/apache/lucene/pull/745
--
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-unsubs
[
https://issues.apache.org/jira/browse/LUCENE-10466?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506530#comment-17506530
]
Julie Tibshirani edited comment on LUCENE-10466 at 3/14/22, 8:40 PM:
[
https://issues.apache.org/jira/browse/LUCENE-10466?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506530#comment-17506530
]
Julie Tibshirani edited comment on LUCENE-10466 at 3/14/22, 8:40 PM:
[
https://issues.apache.org/jira/browse/LUCENE-10466?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506530#comment-17506530
]
Julie Tibshirani commented on LUCENE-10466:
---
Thank you [~reta] for reporting
[
https://issues.apache.org/jira/browse/LUCENE-10460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506528#comment-17506528
]
Julie Tibshirani commented on LUCENE-10460:
---
It indeed seems okay to use a si
Yuti-G opened a new pull request #747:
URL: https://github.com/apache/lucene/pull/747
# Description
This change
* adds a new API - getTopDims in Facets to support users specify the number
of dims and children they want to get, and returns only these dims and children.
* override g
[
https://issues.apache.org/jira/browse/LUCENE-10461?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506488#comment-17506488
]
Dawid Weiss commented on LUCENE-10461:
--
Just to be clear - I did take a look at th
[
https://issues.apache.org/jira/browse/LUCENE-10466?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Andriy Redko updated LUCENE-10466:
--
Priority: Minor (was: Major)
> IndexSortSortedNumericDocValuesRangeQuery unconditionally ass
mocobeta commented on a change in pull request #746:
URL: https://github.com/apache/lucene/pull/746#discussion_r826294904
##
File path: lucene/distribution/src/binary-release/bin/luke.cmd
##
@@ -18,21 +18,17 @@
SETLOCAL
SET MODULES=%~dp0..
-IF DEFINED LAUNCH_CMD GOTO testin
[
https://issues.apache.org/jira/browse/LUCENE-10466?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Andriy Redko updated LUCENE-10466:
--
Description:
We have run into this issue while migrating to OpenSearch and making changes to
[
https://issues.apache.org/jira/browse/LUCENE-10466?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506464#comment-17506464
]
Andriy Redko commented on LUCENE-10466:
---
[~jpountz] does the issue make sense to
Andriy Redko created LUCENE-10466:
-
Summary: IndexSortSortedNumericDocValuesRangeQuery unconditionally
assumes the usage of the LONG-encoded SortField
Key: LUCENE-10466
URL: https://issues.apache.org/jira/browse/L
[
https://issues.apache.org/jira/browse/LUCENE-10461?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506458#comment-17506458
]
Dawid Weiss commented on LUCENE-10461:
--
Hey, Tomoko. I don't think we should go in
mocobeta commented on a change in pull request #746:
URL: https://github.com/apache/lucene/pull/746#discussion_r826266136
##
File path:
lucene/distribution.tests/src/test/org/apache/lucene/distribution/TestScripts.java
##
@@ -125,13 +134,29 @@ protected void execute(
mocobeta commented on a change in pull request #746:
URL: https://github.com/apache/lucene/pull/746#discussion_r826259906
##
File path:
lucene/distribution.tests/src/test/org/apache/lucene/distribution/TestScripts.java
##
@@ -112,6 +120,7 @@ protected void execute(
Laun
[
https://issues.apache.org/jira/browse/LUCENE-10461?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506403#comment-17506403
]
Tomoko Uchida commented on LUCENE-10461:
I am fine with keeping javaw support.
[
https://issues.apache.org/jira/browse/LUCENE-10427?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506402#comment-17506402
]
Adrien Grand commented on LUCENE-10427:
---
Thanks I understand better now. With the
Muler created LUCENE-10465:
--
Summary: Unable to find antlr4.runtime
Key: LUCENE-10465
URL: https://issues.apache.org/jira/browse/LUCENE-10465
Project: Lucene - Core
Issue Type: Bug
Compone
Christine Poerschke created LUCENE-10464:
Summary: unnecessary for-loop in
WeightedSpanTermExtractor.extractWeightedSpanTerms
Key: LUCENE-10464
URL: https://issues.apache.org/jira/browse/LUCENE-10464
[
https://issues.apache.org/jira/browse/LUCENE-10464?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506392#comment-17506392
]
Christine Poerschke commented on LUCENE-10464:
--
https://github.com/apache/
mocobeta opened a new pull request #746:
URL: https://github.com/apache/lucene/pull/746
This tests if Luke process successfully starts on "java" on Mac/Linux or
"start javaw" on Windows.
TestScripts now checks if the expected message is contained in a log file
instead of the forked proc
[
https://issues.apache.org/jira/browse/LUCENE-10441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506376#comment-17506376
]
Peixin Li commented on LUCENE-10441:
How many tokens should causing issue? and is t
[
https://issues.apache.org/jira/browse/LUCENE-10458?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506341#comment-17506341
]
Adrien Grand commented on LUCENE-10458:
---
Thanks for catching this, [~lucacavanna]
gsmiller commented on pull request #718:
URL: https://github.com/apache/lucene/pull/718#issuecomment-1066998156
Even though I ran benchmarks on the backport version of this change (#719),
I figured it would be good to run benchmarks here as well. Below compares this
patch against `main` us
javanna opened a new pull request #745:
URL: https://github.com/apache/lucene/pull/745
Based on discussion happening in
https://issues.apache.org/jira/browse/LUCENE-10458 , I am reverting
LUCENE-10385 (#635) in the 8.1 branch.
I left some test improvements that are still valid but r
gsmiller commented on a change in pull request #718:
URL: https://github.com/apache/lucene/pull/718#discussion_r826085655
##
File path:
lucene/facet/src/java/org/apache/lucene/facet/taxonomy/IntTaxonomyFacets.java
##
@@ -173,17 +185,17 @@ public FacetResult getTopChildren(int
gsmiller commented on a change in pull request #718:
URL: https://github.com/apache/lucene/pull/718#discussion_r826082690
##
File path:
lucene/facet/src/java/org/apache/lucene/facet/taxonomy/FloatTaxonomyFacets.java
##
@@ -130,16 +140,16 @@ public FacetResult getTopChildren(in
jpountz commented on a change in pull request #736:
URL: https://github.com/apache/lucene/pull/736#discussion_r825981973
##
File path:
lucene/sandbox/src/java/org/apache/lucene/sandbox/search/IndexSortSortedNumericDocValuesRangeQuery.java
##
@@ -198,16 +198,22 @@ public boolea
jpountz commented on pull request #737:
URL: https://github.com/apache/lucene/pull/737#issuecomment-1066789801
This change makes sense to me.
--
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
mocobeta commented on pull request #740:
URL: https://github.com/apache/lucene/pull/740#issuecomment-1066672493
We could have a common `DictionaryBuilder` class in analyzers-common but it
brings too complex class hierarchy to me. I'd postpone refactoring
XXXDictionaryBuilder until we come
[
https://issues.apache.org/jira/browse/LUCENE-10448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17506184#comment-17506184
]
Adrien Grand commented on LUCENE-10448:
---
Like [~vigyas] my understanding is that
romseygeek commented on a change in pull request #679:
URL: https://github.com/apache/lucene/pull/679#discussion_r825818251
##
File path: lucene/monitor/src/java/org/apache/lucene/monitor/Monitor.java
##
@@ -125,14 +105,16 @@ public Monitor(Analyzer analyzer, Presearcher
prese
mogui commented on pull request #679:
URL: https://github.com/apache/lucene/pull/679#issuecomment-1066549257
@romseygeek fixed !
--
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 comm
54 matches
Mail list logo
