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

2022-03-19 Thread GitBox
mdmarshmallow commented on a change in pull request #747: URL: https://github.com/apache/lucene/pull/747#discussion_r830444294 ## File path: lucene/facet/src/java/org/apache/lucene/facet/sortedset/SortedSetDocValuesFacetCounts.java ## @@ -114,7 +116,7 @@ public FacetResult get

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

2022-03-19 Thread GitBox
uschindler commented on a change in pull request #2649: URL: https://github.com/apache/lucene-solr/pull/2649#discussion_r830460575 ## File path: lucene/common-build.xml ## @@ -63,7 +63,7 @@ - + Review comment: At least there must be a `\b`

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

2022-03-19 Thread GitBox
uschindler commented on a change in pull request #2649: URL: https://github.com/apache/lucene-solr/pull/2649#discussion_r830460575 ## File path: lucene/common-build.xml ## @@ -63,7 +63,7 @@ - + Review comment: At least there must be a `\b`

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

2022-03-19 Thread GitBox
uschindler commented on a change in pull request #2649: URL: https://github.com/apache/lucene-solr/pull/2649#discussion_r830460575 ## File path: lucene/common-build.xml ## @@ -63,7 +63,7 @@ - + Review comment: At least there must be a `\b`

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

2022-03-19 Thread GitBox
uschindler commented on pull request #2649: URL: https://github.com/apache/lucene-solr/pull/2649#issuecomment-1072980556 I am not sure about the consequences for scripts, but this variant should be fine. The spec version inside jar files stays with version base so the compatibility is ensu

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

2022-03-19 Thread GitBox
uschindler edited a comment on pull request #2649: URL: https://github.com/apache/lucene-solr/pull/2649#issuecomment-1072980556 I am not sure about the consequences for scripts, but this variant should be fine. The spec version inside jar files stays with version base so the compatibility

[GitHub] [lucene] mocobeta opened a new pull request #755: Add note for --tmp-dir option in releaseWizard

2022-03-19 Thread GitBox
mocobeta opened a new pull request #755: URL: https://github.com/apache/lucene/pull/755 I encountered several times a "disk full" error when running the smoke tester. From my observation, there is a test that (intentionally) creates very large index files. It might be helpful to have a

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

2022-03-19 Thread GitBox
msokolov commented on pull request #755: URL: https://github.com/apache/lucene/pull/755#issuecomment-1073041616 Big plus one for this. I can never complete the smoke test without it and I always forget the parameter name and where it has to go on the command line. I don't think it can

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

2022-03-19 Thread GitBox
Yuti-G commented on a change in pull request #747: URL: https://github.com/apache/lucene/pull/747#discussion_r830539688 ## File path: lucene/facet/src/test/org/apache/lucene/facet/sortedset/TestSortedSetDocValuesFacets.java ## @@ -1221,6 +1326,12 @@ public void testRandomHiera

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

2022-03-19 Thread GitBox
Yuti-G commented on a change in pull request #747: URL: https://github.com/apache/lucene/pull/747#discussion_r830539760 ## File path: lucene/facet/src/java/org/apache/lucene/facet/sortedset/SortedSetDocValuesFacetCounts.java ## @@ -114,7 +116,7 @@ public FacetResult getTopChil

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

2022-03-19 Thread GitBox
Yuti-G commented on a change in pull request #747: URL: https://github.com/apache/lucene/pull/747#discussion_r830539828 ## File path: lucene/facet/src/java/org/apache/lucene/facet/sortedset/SortedSetDocValuesFacetCounts.java ## @@ -411,7 +484,109 @@ public int compare(FacetRes

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

2022-03-19 Thread GitBox
Yuti-G commented on a change in pull request #747: URL: https://github.com/apache/lucene/pull/747#discussion_r830539916 ## File path: lucene/facet/src/java/org/apache/lucene/facet/sortedset/SortedSetDocValuesFacetCounts.java ## @@ -411,7 +484,109 @@ public int compare(FacetRes

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

2022-03-19 Thread GitBox
Yuti-G commented on pull request #747: URL: https://github.com/apache/lucene/pull/747#issuecomment-1073132559 > I didn't really have a lot of real feedback, just minor nits here and there. I did want to say that I have a PR out for a benchmark that compares the faceting API performance for

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

2022-03-19 Thread GitBox
Yuti-G commented on a change in pull request #747: URL: https://github.com/apache/lucene/pull/747#discussion_r830539688 ## File path: lucene/facet/src/test/org/apache/lucene/facet/sortedset/TestSortedSetDocValuesFacets.java ## @@ -1221,6 +1326,12 @@ public void testRandomHiera

[jira] [Commented] (LUCENE-9905) Revise approach to specifying NN algorithm

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

[jira] [Commented] (LUCENE-9905) Revise approach to specifying NN algorithm

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

[jira] [Commented] (LUCENE-9905) Revise approach to specifying NN algorithm

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

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

2022-03-19 Thread GitBox
mocobeta commented on pull request #755: URL: https://github.com/apache/lucene/pull/755#issuecomment-1073174420 My `tmpfs` partition (where system `/tmp` is mounted) has 8GB of available disk space. The test itself has been there for a long time and I think I had not seen this "disk ful

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

2022-03-19 Thread GitBox
mocobeta edited a comment on pull request #755: URL: https://github.com/apache/lucene/pull/755#issuecomment-1073174420 My `tmpfs` partition (where system `/tmp` is mounted) has 8GB of available disk space. The test itself has been there for a long time and I think I had not seen this "d

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

2022-03-19 Thread Vigya Sharma (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509380#comment-17509380 ] Vigya Sharma commented on LUCENE-10448: --- {color:#172b4d}Thanks for making the cha

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

2022-03-19 Thread Vigya Sharma (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509380#comment-17509380 ] Vigya Sharma edited comment on LUCENE-10448 at 3/20/22, 6:11 AM:

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

2022-03-19 Thread GitBox
vigyasharma commented on pull request #738: URL: https://github.com/apache/lucene/pull/738#issuecomment-1073176564 > Since a release candidate is out for testing, I'll wait until that process completes before merging this. It would be safe anyway, and this is small, but just in case there

[GitHub] [lucene] vigyasharma commented on pull request #633: LUCENE-10216: Use MergeScheduler and MergePolicy to run addIndexes(CodecReader[]) merges.

2022-03-19 Thread GitBox
vigyasharma commented on pull request #633: URL: https://github.com/apache/lucene/pull/633#issuecomment-1073177695 Thanks @mikemccand . I figured that I had changed the exception signature for addIndexes() in my last revision. I was throwing a blanket MergePolicyException when addIndexes f

[jira] [Commented] (LUCENE-10216) Add concurrency to addIndexes(CodecReader…) API

2022-03-19 Thread Vigya Sharma (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509383#comment-17509383 ] Vigya Sharma commented on LUCENE-10216: --- Updated the PR to retain transactionalit

[jira] [Commented] (LUCENE-10216) Add concurrency to addIndexes(CodecReader…) API

2022-03-19 Thread Vigya Sharma (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509384#comment-17509384 ] Vigya Sharma commented on LUCENE-10216: --- While making this change, I found an exi

[jira] [Commented] (LUCENE-8580) Make segment merging parallel in SegmentMerger

2022-03-19 Thread Vigya Sharma (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-8580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17509385#comment-17509385 ] Vigya Sharma commented on LUCENE-8580: -- I started looking into this, and had some e

[GitHub] [lucene] vigyasharma commented on a change in pull request #633: LUCENE-10216: Use MergeScheduler and MergePolicy to run addIndexes(CodecReader[]) merges.

2022-03-19 Thread GitBox
vigyasharma commented on a change in pull request #633: URL: https://github.com/apache/lucene/pull/633#discussion_r830568250 ## File path: lucene/core/src/java/org/apache/lucene/index/MergePolicy.java ## @@ -813,12 +866,24 @@ protected final boolean verbose(MergeContext mergeC

[jira] [Created] (LUCENE-10476) Modify the findMerges policy for addIndexes(CodecReader[]) to create 1 segment per reader

2022-03-19 Thread Vigya Sharma (Jira)
Vigya Sharma created LUCENE-10476: - Summary: Modify the findMerges policy for addIndexes(CodecReader[]) to create 1 segment per reader Key: LUCENE-10476 URL: https://issues.apache.org/jira/browse/LUCENE-10476

[GitHub] [lucene] vigyasharma commented on a change in pull request #633: LUCENE-10216: Use MergeScheduler and MergePolicy to run addIndexes(CodecReader[]) merges.

2022-03-19 Thread GitBox
vigyasharma commented on a change in pull request #633: URL: https://github.com/apache/lucene/pull/633#discussion_r830568703 ## File path: lucene/core/src/java/org/apache/lucene/index/MergePolicy.java ## @@ -567,6 +605,21 @@ public abstract MergeSpecification findMerges(