[GitHub] [lucene] mogui commented on pull request #679: Monitor Improvements LUCENE-10422

2022-03-21 Thread GitBox
mogui commented on pull request #679: URL: https://github.com/apache/lucene/pull/679#issuecomment-1073666978 @romseygeek sorry to bother you, just a reminder :D -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [lucene] jpountz merged pull request #754: LUCENE-10473: Make tests a bit faster when running nightly.

2022-03-21 Thread GitBox
jpountz merged pull request #754: URL: https://github.com/apache/lucene/pull/754 -- 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..

[jira] [Commented] (LUCENE-10473) Address slow testRandomBig runs

2022-03-21 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10473?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509718#comment-17509718 ] ASF subversion and git services commented on LUCENE-10473: -- Co

[jira] [Commented] (LUCENE-10473) Address slow testRandomBig runs

2022-03-21 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10473?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509721#comment-17509721 ] ASF subversion and git services commented on LUCENE-10473: -- Co

[jira] [Commented] (LUCENE-10473) Address slow testRandomBig runs

2022-03-21 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10473?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509720#comment-17509720 ] ASF subversion and git services commented on LUCENE-10473: -- Co

[GitHub] [lucene] jpountz commented on pull request #755: Add note for smoke tester --tmp-dir option in rc announcing

2022-03-21 Thread GitBox
jpountz commented on pull request #755: URL: https://github.com/apache/lucene/pull/755#issuecomment-1073690606 One downside of this approach is that nobody could just copy-paste the suggested command to run the smoke tester any longer, they would always have to tune the temporary path. I'd

[jira] [Commented] (LUCENE-10448) MergeRateLimiter doesn't always limit instant rate.

2022-03-21 Thread Adrien Grand (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509741#comment-17509741 ] Adrien Grand commented on LUCENE-10448: --- bq. we could handle the edge case of whe

[GitHub] [lucene] iverase opened a new pull request #756: LUCENE-10470: [Tessellator] Prevent bridges that introduce collinear edges

2022-03-21 Thread GitBox
iverase opened a new pull request #756: URL: https://github.com/apache/lucene/pull/756 Similar to LUCENE-9580, It might happen that the new edge introduced when removing a tile is collinear with edges from the resulting polygon. This situation makes this edges not eligible for filtering b

[GitHub] [lucene] jpountz commented on a change in pull request #736: LUCENE-10458: BoundedDocSetIdIterator may supply error count in Weigth#count(LeafReaderContext) when missingValue enables

2022-03-21 Thread GitBox
jpountz commented on a change in pull request #736: URL: https://github.com/apache/lucene/pull/736#discussion_r827810798 ## File path: lucene/sandbox/src/java/org/apache/lucene/sandbox/search/IndexSortSortedNumericDocValuesRangeQuery.java ## @@ -198,16 +200,23 @@ public boolea

[GitHub] [lucene] janhoy commented on pull request #755: Add note for smoke tester --tmp-dir option in rc announcing

2022-03-21 Thread GitBox
janhoy commented on pull request #755: URL: https://github.com/apache/lucene/pull/755#issuecomment-1073746350 Perhaps we could have an env var `LUCENE_SMOKE_TESTER_TMP_DIR` that the smoke tester would respect, then devs can set that once and be happy? -- This is an automated message from

[jira] [Commented] (LUCENE-10470) Unable to Tessellate polygon

2022-03-21 Thread Ignacio Vera (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10470?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509783#comment-17509783 ] Ignacio Vera commented on LUCENE-10470: --- Thanks [~yixunx] fro reporting. Indeed

[GitHub] [lucene] mocobeta commented on pull request #755: Add note for smoke tester --tmp-dir option in rc announcing

2022-03-21 Thread GitBox
mocobeta commented on pull request #755: URL: https://github.com/apache/lucene/pull/755#issuecomment-1073827624 Thanks for your feedback and suggestions. I reverted the command line to keep it as is. The option could still be useful for some situations, I added a description of it in

[GitHub] [lucene] mocobeta edited a comment on pull request #755: Add note for smoke tester --tmp-dir option in rc announcing

2022-03-21 Thread GitBox
mocobeta edited a comment on pull request #755: URL: https://github.com/apache/lucene/pull/755#issuecomment-1073827624 Thanks for your feedback and suggestions. I reverted the command line to keep it as is. The option could still be useful for some situations, I added a description o

[jira] [Created] (LUCENE-10477) SpanBoostQuery.rewrite was no-op for boost==1 factor

2022-03-21 Thread Christine Poerschke (Jira)
Christine Poerschke created LUCENE-10477: Summary: SpanBoostQuery.rewrite was no-op for boost==1 factor Key: LUCENE-10477 URL: https://issues.apache.org/jira/browse/LUCENE-10477 Project: Lucene

[jira] [Commented] (LUCENE-10477) SpanBoostQuery.rewrite was no-op for boost==1 factor

2022-03-21 Thread Adrien Grand (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509827#comment-17509827 ] Adrien Grand commented on LUCENE-10477: --- It should be ok given that callers are e

[jira] [Commented] (LUCENE-8637) WeightedSpanTermExtractor unnexessarily enforces rewrite for some SpanQueiries

2022-03-21 Thread Christine Poerschke (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-8637?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509828#comment-17509828 ] Christine Poerschke commented on LUCENE-8637: - bq. Method mustRewriteQuery(S

[GitHub] [lucene-solr] cpoerschke opened a new pull request #2650: LUCENE-10477: SpanBoostQuery.rewrite was no-op for boost==1 factor.

2022-03-21 Thread GitBox
cpoerschke opened a new pull request #2650: URL: https://github.com/apache/lucene-solr/pull/2650 https://issues.apache.org/jira/browse/LUCENE-10477 -- 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 t

[jira] [Commented] (LUCENE-10477) SpanBoostQuery.rewrite was no-op for boost==1 factor

2022-03-21 Thread Christine Poerschke (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509835#comment-17509835 ] Christine Poerschke commented on LUCENE-10477: -- For 8.0 this should be the

[jira] [Commented] (LUCENE-10477) SpanBoostQuery.rewrite was no-op for boost==1 factor

2022-03-21 Thread Adrien Grand (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509841#comment-17509841 ] Adrien Grand commented on LUCENE-10477: --- Thanks Christine, this is where the bug

[jira] [Commented] (LUCENE-10477) SpanBoostQuery.rewrite was no-op for boost==1 factor

2022-03-21 Thread Christine Poerschke (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509842#comment-17509842 ] Christine Poerschke commented on LUCENE-10477: -- bq. ... callers are expect

[GitHub] [lucene] jpountz commented on a change in pull request #737: LUCENE-10464: Reduce for-loop in WeightedSpanTermExtractor.extractWeightedSpanTerms method.

2022-03-21 Thread GitBox
jpountz commented on a change in pull request #737: URL: https://github.com/apache/lucene/pull/737#discussion_r831103798 ## File path: lucene/highlighter/src/java/org/apache/lucene/search/highlight/WeightedSpanTermExtractor.java ## @@ -309,11 +309,11 @@ protected void extractW

[GitHub] [lucene] cpoerschke commented on a change in pull request #737: LUCENE-10464: Reduce for-loop in WeightedSpanTermExtractor.extractWeightedSpanTerms method.

2022-03-21 Thread GitBox
cpoerschke commented on a change in pull request #737: URL: https://github.com/apache/lucene/pull/737#discussion_r831107050 ## File path: lucene/highlighter/src/java/org/apache/lucene/search/highlight/WeightedSpanTermExtractor.java ## @@ -309,11 +309,11 @@ protected void extra

[jira] [Updated] (LUCENE-10477) SpanBoostQuery.rewrite was incomplete for boost==1 factor

2022-03-21 Thread Christine Poerschke (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10477?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Christine Poerschke updated LUCENE-10477: - Summary: SpanBoostQuery.rewrite was incomplete for boost==1 factor (was: SpanB

[GitHub] [lucene-solr] cpoerschke commented on pull request #2650: LUCENE-10477: SpanBoostQuery.rewrite was incomplete for boost==1 factor.

2022-03-21 Thread GitBox
cpoerschke commented on pull request #2650: URL: https://github.com/apache/lucene-solr/pull/2650#issuecomment-1073947536 closing in favour of https://github.com/apache/lucene/pull/737 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [lucene-solr] cpoerschke closed pull request #2650: LUCENE-10477: SpanBoostQuery.rewrite was incomplete for boost==1 factor.

2022-03-21 Thread GitBox
cpoerschke closed pull request #2650: URL: https://github.com/apache/lucene-solr/pull/2650 -- 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-

[GitHub] [lucene] jpountz commented on a change in pull request #737: LUCENE-10464, LUCENE-10477: WeightedSpanTermExtractor.extractWeightedSpanTerms to rewrite sufficiently

2022-03-21 Thread GitBox
jpountz commented on a change in pull request #737: URL: https://github.com/apache/lucene/pull/737#discussion_r831152924 ## File path: lucene/highlighter/src/java/org/apache/lucene/search/highlight/WeightedSpanTermExtractor.java ## @@ -309,11 +309,12 @@ protected void extractW

[GitHub] [lucene] ywelsch commented on pull request #752: LUCENE-10474: Avoid throwing StackOverflowError when creating RegExp

2022-03-21 Thread GitBox
ywelsch commented on pull request #752: URL: https://github.com/apache/lucene/pull/752#issuecomment-1073949409 > I'm still -1 for the change. If you hit StackOverFlowError, really you should let the VM exit. There are no guarantees at this point. That kind of argument makes it even m

[GitHub] [lucene] romseygeek commented on pull request #679: Monitor Improvements LUCENE-10422

2022-03-21 Thread GitBox
romseygeek commented on pull request #679: URL: https://github.com/apache/lucene/pull/679#issuecomment-1073951849 One last thing, can you add a CHANGES.txt entry in the Lucene 9.2.0 New Features section? And then I think we're good to merge! Thanks for all your patience on this. -- This

[GitHub] [lucene] cpoerschke commented on a change in pull request #737: LUCENE-10464, LUCENE-10477: WeightedSpanTermExtractor.extractWeightedSpanTerms to rewrite sufficiently

2022-03-21 Thread GitBox
cpoerschke commented on a change in pull request #737: URL: https://github.com/apache/lucene/pull/737#discussion_r831158244 ## File path: lucene/highlighter/src/java/org/apache/lucene/search/highlight/WeightedSpanTermExtractor.java ## @@ -309,11 +309,12 @@ protected void extra

[GitHub] [lucene] mogui commented on pull request #679: Monitor Improvements LUCENE-10422

2022-03-21 Thread GitBox
mogui commented on pull request #679: URL: https://github.com/apache/lucene/pull/679#issuecomment-1073983473 Done! Thanks for Your patience and support @romseygeek ! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[jira] [Commented] (LUCENE-10477) SpanBoostQuery.rewrite was incomplete for boost==1 factor

2022-03-21 Thread Christine Poerschke (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509929#comment-17509929 ] Christine Poerschke commented on LUCENE-10477: -- https://github.com/apache/

[jira] [Created] (LUCENE-10478) Mark Test4GBStoredFields as @Monster (it consumes a lot of disk)

2022-03-21 Thread Tomoko Uchida (Jira)
Tomoko Uchida created LUCENE-10478: -- Summary: Mark Test4GBStoredFields as @Monster (it consumes a lot of disk) Key: LUCENE-10478 URL: https://issues.apache.org/jira/browse/LUCENE-10478 Project: Lucen

[jira] [Commented] (LUCENE-10478) Mark Test4GBStoredFields as @Monster (it consumes a lot of disk)

2022-03-21 Thread Adrien Grand (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10478?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509942#comment-17509942 ] Adrien Grand commented on LUCENE-10478: --- +1 Using that much disk space definitely

[GitHub] [lucene] romseygeek commented on a change in pull request #679: Monitor Improvements LUCENE-10422

2022-03-21 Thread GitBox
romseygeek commented on a change in pull request #679: URL: https://github.com/apache/lucene/pull/679#discussion_r831225341 ## File path: lucene/CHANGES.txt ## @@ -56,7 +56,7 @@ API Changes New Features - -(No changes) +* LUCENE-10422: Monitor Improvemen

[GitHub] [lucene] mocobeta opened a new pull request #757: LUCENE-10478: mark Test4GBStoredFields as @Monster

2022-03-21 Thread GitBox
mocobeta opened a new pull request #757: URL: https://github.com/apache/lucene/pull/757 Excludes `Test4GBStoredFields` from nightly tests. -- 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 spe

[jira] [Commented] (LUCENE-10478) Mark Test4GBStoredFields as @Monster (it consumes a lot of disk)

2022-03-21 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10478?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509949#comment-17509949 ] Uwe Schindler commented on LUCENE-10478: +1, test is also taking very long. >

[GitHub] [lucene] mocobeta commented on pull request #757: LUCENE-10478: mark Test4GBStoredFields as @Monster

2022-03-21 Thread GitBox
mocobeta commented on pull request #757: URL: https://github.com/apache/lucene/pull/757#issuecomment-1074038814 It seems the commit message accidentally mentions an unrelated github account - I'm sorry for the noise. -- This is an automated message from the Apache Git Service. To respond

[jira] [Commented] (LUCENE-10478) Mark Test4GBStoredFields as @Monster (it consumes a lot of disk)

2022-03-21 Thread Tomoko Uchida (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10478?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509954#comment-17509954 ] Tomoko Uchida commented on LUCENE-10478: I opened #757, the link to it has appe

[GitHub] [lucene] mogui commented on pull request #679: Monitor Improvements LUCENE-10422

2022-03-21 Thread GitBox
mogui commented on pull request #679: URL: https://github.com/apache/lucene/pull/679#issuecomment-1074048328 Ok! -- 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 unsubsc

[GitHub] [lucene] jpountz commented on a change in pull request #757: LUCENE-10478: mark Test4GBStoredFields as @Monster

2022-03-21 Thread GitBox
jpountz commented on a change in pull request #757: URL: https://github.com/apache/lucene/pull/757#discussion_r831260485 ## File path: lucene/core/src/test/org/apache/lucene/index/Test4GBStoredFields.java ## @@ -26,13 +26,15 @@ import org.apache.lucene.tests.codecs.compressing

[GitHub] [lucene] msokolov commented on pull request #738: LUCENE-10448: Avoid instant rate write bursts by writing bytes buffer in chunks

2022-03-21 Thread GitBox
msokolov commented on pull request #738: URL: https://github.com/apache/lucene/pull/738#issuecomment-1074085956 Yes, let's add a CHANGES entry and tag it as 9.2.0 since we'll want to backport it -- This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] [lucene] msokolov commented on pull request #752: LUCENE-10474: Avoid throwing StackOverflowError when creating RegExp

2022-03-21 Thread GitBox
msokolov commented on pull request #752: URL: https://github.com/apache/lucene/pull/752#issuecomment-1074111548 If we can find a clean way to detect imminent stack overflow and throw an exc, that would be great. Maybe a member variable on RegExp would be less intrusive than adding paramete

[GitHub] [lucene] romseygeek merged pull request #679: Monitor Improvements LUCENE-10422

2022-03-21 Thread GitBox
romseygeek merged pull request #679: URL: https://github.com/apache/lucene/pull/679 -- 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-unsubsc

[jira] [Commented] (LUCENE-10422) Monitor instantiation configurabilty improvements

2022-03-21 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10422?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17510003#comment-17510003 ] ASF subversion and git services commented on LUCENE-10422: -- Co

[GitHub] [lucene] nknize commented on a change in pull request #749: LUCENE-10466: IndexSortSortedNumericDocValuesRangeQuery unconditionally assumes the usage of the LONG-encoded SortField

2022-03-21 Thread GitBox
nknize commented on a change in pull request #749: URL: https://github.com/apache/lucene/pull/749#discussion_r831316035 ## File path: lucene/sandbox/src/test/org/apache/lucene/sandbox/search/TestIndexSortSortedNumericDocValuesRangeQueryByType.java ## @@ -0,0 +1,126 @@ +/* + *

[jira] [Commented] (LUCENE-10422) Monitor instantiation configurabilty improvements

2022-03-21 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10422?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17510009#comment-17510009 ] ASF subversion and git services commented on LUCENE-10422: -- Co

[jira] [Resolved] (LUCENE-10422) Monitor instantiation configurabilty improvements

2022-03-21 Thread Alan Woodward (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10422?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alan Woodward resolved LUCENE-10422. Fix Version/s: 9.2 Resolution: Fixed > Monitor instantiation configurabilty improv

[GitHub] [lucene] reta commented on a change in pull request #749: LUCENE-10466: IndexSortSortedNumericDocValuesRangeQuery unconditionally assumes the usage of the LONG-encoded SortField

2022-03-21 Thread GitBox
reta commented on a change in pull request #749: URL: https://github.com/apache/lucene/pull/749#discussion_r831340155 ## File path: lucene/sandbox/src/test/org/apache/lucene/sandbox/search/TestIndexSortSortedNumericDocValuesRangeQueryByType.java ## @@ -0,0 +1,126 @@ +/* + * Li

[GitHub] [lucene] gsmiller commented on a change in pull request #747: LUCENE-10325: Add getTopDims functionality to Facets

2022-03-21 Thread GitBox
gsmiller commented on a change in pull request #747: URL: https://github.com/apache/lucene/pull/747#discussion_r831477355 ## File path: lucene/facet/src/java/org/apache/lucene/facet/sortedset/SortedSetDocValuesFacetCounts.java ## @@ -414,4 +505,101 @@ public int compare(FacetR

[GitHub] [lucene-solr] anshumg commented on pull request #2649: Remove '-' between base.version and version.suffix and change common-build to allow the new format

2022-03-21 Thread GitBox
anshumg commented on pull request #2649: URL: https://github.com/apache/lucene-solr/pull/2649#issuecomment-1074345569 Thanks for your suggestion @uschindler. I like your suggestion but can't think of a clean way to make this work for multiple internal releases. I'm trying to avoid forking

[GitHub] [lucene] gsmiller commented on a change in pull request #747: LUCENE-10325: Add getTopDims functionality to Facets

2022-03-21 Thread GitBox
gsmiller commented on a change in pull request #747: URL: https://github.com/apache/lucene/pull/747#discussion_r831491304 ## File path: lucene/facet/src/java/org/apache/lucene/facet/sortedset/SortedSetDocValuesFacetCounts.java ## @@ -190,20 +254,33 @@ private FacetResult getPa

[jira] [Created] (LUCENE-10479) Benchmark documentation referes to non-existent tasks

2022-03-21 Thread Mike Drob (Jira)
Mike Drob created LUCENE-10479: -- Summary: Benchmark documentation referes to non-existent tasks Key: LUCENE-10479 URL: https://issues.apache.org/jira/browse/LUCENE-10479 Project: Lucene - Core I

[GitHub] [lucene] Yuti-G commented on a change in pull request #747: LUCENE-10325: Add getTopDims functionality to Facets

2022-03-21 Thread GitBox
Yuti-G commented on a change in pull request #747: URL: https://github.com/apache/lucene/pull/747#discussion_r831558514 ## File path: lucene/facet/src/java/org/apache/lucene/facet/sortedset/SortedSetDocValuesFacetCounts.java ## @@ -178,10 +229,23 @@ private FacetResult getPath

[GitHub] [lucene] Yuti-G commented on a change in pull request #747: LUCENE-10325: Add getTopDims functionality to Facets

2022-03-21 Thread GitBox
Yuti-G commented on a change in pull request #747: URL: https://github.com/apache/lucene/pull/747#discussion_r831558514 ## File path: lucene/facet/src/java/org/apache/lucene/facet/sortedset/SortedSetDocValuesFacetCounts.java ## @@ -178,10 +229,23 @@ private FacetResult getPath

[GitHub] [lucene] Yuti-G commented on a change in pull request #747: LUCENE-10325: Add getTopDims functionality to Facets

2022-03-21 Thread GitBox
Yuti-G commented on a change in pull request #747: URL: https://github.com/apache/lucene/pull/747#discussion_r831558514 ## File path: lucene/facet/src/java/org/apache/lucene/facet/sortedset/SortedSetDocValuesFacetCounts.java ## @@ -178,10 +229,23 @@ private FacetResult getPath

[GitHub] [lucene] Yuti-G commented on a change in pull request #747: LUCENE-10325: Add getTopDims functionality to Facets

2022-03-21 Thread GitBox
Yuti-G commented on a change in pull request #747: URL: https://github.com/apache/lucene/pull/747#discussion_r831558514 ## File path: lucene/facet/src/java/org/apache/lucene/facet/sortedset/SortedSetDocValuesFacetCounts.java ## @@ -178,10 +229,23 @@ private FacetResult getPath

[GitHub] [lucene] Yuti-G commented on a change in pull request #747: LUCENE-10325: Add getTopDims functionality to Facets

2022-03-21 Thread GitBox
Yuti-G commented on a change in pull request #747: URL: https://github.com/apache/lucene/pull/747#discussion_r831568451 ## File path: lucene/facet/src/java/org/apache/lucene/facet/sortedset/SortedSetDocValuesFacetCounts.java ## @@ -414,4 +505,101 @@ public int compare(FacetRes