[GitHub] [lucene] rmuir commented on a change in pull request #286: LUCENE-10089 Disable numeric sort optim when needed

2021-09-08 Thread GitBox
rmuir commented on a change in pull request #286: URL: https://github.com/apache/lucene/pull/286#discussion_r704816931 ## File path: lucene/core/src/java/org/apache/lucene/search/SortField.java ## @@ -606,4 +609,26 @@ public IndexSorter getIndexSorter() { return null;

[jira] [Commented] (LUCENE-10080) Use a bit set to count long-tail of singleton FacetLabels?

2021-09-08 Thread Marc D'Mello (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10080?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17412204#comment-17412204 ] Marc D'Mello commented on LUCENE-10080: --- Here is the pull request link for the ch

[GitHub] [lucene] mdmarshmallow opened a new pull request #288: LUCENE 10080 Added FixedBitSet for one counts when counting taxonomy facet labels

2021-09-08 Thread GitBox
mdmarshmallow opened a new pull request #288: URL: https://github.com/apache/lucene/pull/288 # Description This change is a possible improvement to how we do facet counting. Today, we accumulate counts in an HPPC int/int map or an `int[]` array. However, it is possible m

[GitHub] [lucene-solr] mikemccand commented on a change in pull request #2567: LUCENE-9662: CheckIndex should be concurrent (backporting)

2021-09-08 Thread GitBox
mikemccand commented on a change in pull request #2567: URL: https://github.com/apache/lucene-solr/pull/2567#discussion_r704763696 ## File path: lucene/core/src/java/org/apache/lucene/index/CheckIndex.java ## @@ -465,9 +493,22 @@ public void setChecksumsOnly(boolean v) {

[GitHub] [lucene-solr] mikemccand commented on pull request #2567: LUCENE-9662: CheckIndex should be concurrent (backporting)

2021-09-08 Thread GitBox
mikemccand commented on pull request #2567: URL: https://github.com/apache/lucene-solr/pull/2567#issuecomment-915557112 Did you also include backport for fixing `Usage: ...` output? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [lucene] mayya-sharipova commented on a change in pull request #286: LUCENE-10089 Disable numeric sort optim when needed

2021-09-08 Thread GitBox
mayya-sharipova commented on a change in pull request #286: URL: https://github.com/apache/lucene/pull/286#discussion_r704733017 ## File path: lucene/core/src/java/org/apache/lucene/search/SortField.java ## @@ -606,4 +609,26 @@ public IndexSorter getIndexSorter() { ret

[GitHub] [lucene] mayya-sharipova commented on a change in pull request #286: LUCENE-10089 Disable numeric sort optim when needed

2021-09-08 Thread GitBox
mayya-sharipova commented on a change in pull request #286: URL: https://github.com/apache/lucene/pull/286#discussion_r704732855 ## File path: lucene/core/src/java/org/apache/lucene/search/SortField.java ## @@ -130,6 +130,9 @@ // Used for 'sortMissingFirst/Last' protected

[GitHub] [lucene] mayya-sharipova commented on a change in pull request #286: LUCENE-10089 Disable numeric sort optim when needed

2021-09-08 Thread GitBox
mayya-sharipova commented on a change in pull request #286: URL: https://github.com/apache/lucene/pull/286#discussion_r704707532 ## File path: lucene/core/src/java/org/apache/lucene/search/SortField.java ## @@ -606,4 +609,26 @@ public IndexSorter getIndexSorter() { ret

[jira] [Resolved] (LUCENE-10092) TestCheckIndex failure

2021-09-08 Thread Michael McCandless (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10092?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Michael McCandless resolved LUCENE-10092. - Fix Version/s: main (9.0) Resolution: Fixed Thanks [~jpountz]. We will

[jira] [Commented] (LUCENE-10092) TestCheckIndex failure

2021-09-08 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10092?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17412117#comment-17412117 ] ASF subversion and git services commented on LUCENE-10092: -- Co

[jira] [Commented] (LUCENE-10092) TestCheckIndex failure

2021-09-08 Thread Zach Chen (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10092?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17412111#comment-17412111 ] Zach Chen commented on LUCENE-10092: Thanks Michael! I appreciate it! > TestCheckI

[jira] [Commented] (LUCENE-10092) TestCheckIndex failure

2021-09-08 Thread Michael McCandless (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10092?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17412108#comment-17412108 ] Michael McCandless commented on LUCENE-10092: - No worries [~zacharymorn] –

[jira] [Commented] (LUCENE-10092) TestCheckIndex failure

2021-09-08 Thread Zach Chen (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10092?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17412105#comment-17412105 ] Zach Chen commented on LUCENE-10092: Sorry for (another) TestCheckIndex failure! Th

[GitHub] [lucene-solr] mikemccand commented on pull request #2567: LUCENE-9662: CheckIndex should be concurrent (backporting)

2021-09-08 Thread GitBox
mikemccand commented on pull request #2567: URL: https://github.com/apache/lucene-solr/pull/2567#issuecomment-915401627 Let's remember to also backport [fix for LUCENE-10092](https://issues.apache.org/jira/browse/LUCENE-10092). -- This is an automated message from the Apache Git Service.

[GitHub] [lucene] xiaoshi2013 commented on pull request #173: Initial rewrite of MMapDirectory for JDK-16 preview (incubating) Panama APIs (>= JDK-16-ea-b32)

2021-09-08 Thread GitBox
xiaoshi2013 commented on pull request #173: URL: https://github.com/apache/lucene/pull/173#issuecomment-915366162 so cool! -- 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.

[GitHub] [lucene] msokolov commented on pull request #274: LUCENE-10082: add detail to schema inconsistency error messages

2021-09-08 Thread GitBox
msokolov commented on pull request #274: URL: https://github.com/apache/lucene/pull/274#issuecomment-915314874 Yes, looks OK. Note I also did add some overloads to avoid autoboxing, so I think mostly we didn't add any. May not have been needed. On Wed, Sep 8, 2021 at 7:59 AM Adrien

[jira] [Comment Edited] (LUCENE-10091) Fix some old errors in the benchmark module

2021-09-08 Thread xiaoshi (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10091?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17411974#comment-17411974 ] xiaoshi edited comment on LUCENE-10091 at 9/8/21, 2:41 PM: --- T

[jira] [Commented] (LUCENE-10091) Fix some old errors in the benchmark module

2021-09-08 Thread xiaoshi (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10091?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17411974#comment-17411974 ] xiaoshi commented on LUCENE-10091: -- Thanks @mikemccand , request you a review of my pr

[GitHub] [lucene-solr] xiaoshi2013 removed a comment on pull request #2566: LUCENE-10091 Fix some old errors in the benchmark module

2021-09-08 Thread GitBox
xiaoshi2013 removed a comment on pull request #2566: URL: https://github.com/apache/lucene-solr/pull/2566#issuecomment-915296398 Thanks @mikemccand , request you a review of my pr. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [lucene-solr] xiaoshi2013 commented on pull request #2566: LUCENE-10091 Fix some old errors in the benchmark module

2021-09-08 Thread GitBox
xiaoshi2013 commented on pull request #2566: URL: https://github.com/apache/lucene-solr/pull/2566#issuecomment-915296398 Thanks @mikemccand , request you a review of my pr. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [lucene] rmuir commented on a change in pull request #286: LUCENE-10089 Disable numeric sort optim when needed

2021-09-08 Thread GitBox
rmuir commented on a change in pull request #286: URL: https://github.com/apache/lucene/pull/286#discussion_r704440549 ## File path: lucene/core/src/java/org/apache/lucene/search/SortField.java ## @@ -130,6 +130,9 @@ // Used for 'sortMissingFirst/Last' protected Object mi

[jira] [Commented] (LUCENE-10088) Too many open files in TestIndexWriterMergePolicy.testStressUpdateSameDocumentWithMergeOnGetReader

2021-09-08 Thread Michael McCandless (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10088?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17411913#comment-17411913 ] Michael McCandless commented on LUCENE-10088: - OK the 8.x failure repros fo

[jira] [Commented] (LUCENE-10088) Too many open files in TestIndexWriterMergePolicy.testStressUpdateSameDocumentWithMergeOnGetReader

2021-09-08 Thread Michael McCandless (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10088?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17411911#comment-17411911 ] Michael McCandless commented on LUCENE-10088: - Yeah now that we understand

[jira] [Commented] (LUCENE-10092) TestCheckIndex failure

2021-09-08 Thread Michael McCandless (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10092?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17411909#comment-17411909 ] Michael McCandless commented on LUCENE-10092: - This seems to work: {noforma

[jira] [Commented] (LUCENE-10092) TestCheckIndex failure

2021-09-08 Thread Michael McCandless (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10092?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17411908#comment-17411908 ] Michael McCandless commented on LUCENE-10092: - I added {{-Dtests.verbose=tr

[jira] [Commented] (LUCENE-10092) TestCheckIndex failure

2021-09-08 Thread Michael McCandless (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10092?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17411907#comment-17411907 ] Michael McCandless commented on LUCENE-10092: - Repros for me ... looks like

[jira] [Updated] (LUCENE-10092) TestCheckIndex failure

2021-09-08 Thread Adrien Grand (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10092?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Adrien Grand updated LUCENE-10092: -- Description: We got the below test failure on Elastic's CI: {noformat} 10:07:08 org.apache.l

[jira] [Created] (LUCENE-10092) TestCheckIndex failure

2021-09-08 Thread Adrien Grand (Jira)
Adrien Grand created LUCENE-10092: - Summary: TestCheckIndex failure Key: LUCENE-10092 URL: https://issues.apache.org/jira/browse/LUCENE-10092 Project: Lucene - Core Issue Type: Bug

[GitHub] [lucene] jpountz commented on a change in pull request #286: LUCENE-10089 Disable numeric sort optim when needed

2021-09-08 Thread GitBox
jpountz commented on a change in pull request #286: URL: https://github.com/apache/lucene/pull/286#discussion_r704360843 ## File path: lucene/CHANGES.txt ## @@ -141,6 +141,10 @@ API Changes The Weight#count API represents a cleaner way for Query classes to optimize their co

[GitHub] [lucene] mayya-sharipova commented on pull request #286: LUCENE-10089 Disable numeric sort optim when needed

2021-09-08 Thread GitBox
mayya-sharipova commented on pull request #286: URL: https://github.com/apache/lucene/pull/286#issuecomment-915181037 @romseygeek Thanks for the review. @jpountz I've addressed your comments. Can you please continue the review. -- This is an automated message from the Apache Git S

[GitHub] [lucene] mayya-sharipova commented on a change in pull request #286: LUCENE-10089 Disable numeric sort optim when needed

2021-09-08 Thread GitBox
mayya-sharipova commented on a change in pull request #286: URL: https://github.com/apache/lucene/pull/286#discussion_r704358532 ## File path: lucene/core/src/java/org/apache/lucene/search/SortField.java ## @@ -606,4 +609,20 @@ public IndexSorter getIndexSorter() { ret

[GitHub] [lucene] mayya-sharipova commented on a change in pull request #286: LUCENE-10089 Disable numeric sort optim when needed

2021-09-08 Thread GitBox
mayya-sharipova commented on a change in pull request #286: URL: https://github.com/apache/lucene/pull/286#discussion_r704358433 ## File path: lucene/core/src/java/org/apache/lucene/search/SortField.java ## @@ -606,4 +609,20 @@ public IndexSorter getIndexSorter() { ret

[jira] [Commented] (LUCENE-10088) Too many open files in TestIndexWriterMergePolicy.testStressUpdateSameDocumentWithMergeOnGetReader

2021-09-08 Thread Adrien Grand (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10088?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17411887#comment-17411887 ] Adrien Grand commented on LUCENE-10088: --- [~simonw] I'd be interested in your take

[GitHub] [lucene] jpountz commented on pull request #274: LUCENE-10082: add detail to schema inconsistency error messages

2021-09-08 Thread GitBox
jpountz commented on pull request #274: URL: https://github.com/apache/lucene/pull/274#issuecomment-915171475 Indexing rate and GC/JIT times look stable. :+1: -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[jira] [Commented] (LUCENE-10010) Should we have a NFA Query?

2021-09-08 Thread Michael McCandless (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10010?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17411882#comment-17411882 ] Michael McCandless commented on LUCENE-10010: - Thanks for the update [~zhai

[jira] [Commented] (LUCENE-10091) Fix some old errors in the benchmark module

2021-09-08 Thread Michael McCandless (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-10091?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17411881#comment-17411881 ] Michael McCandless commented on LUCENE-10091: - Thanks [~xiaoshi_2021]!  You

[jira] [Commented] (LUCENE-9662) CheckIndex should be concurrent

2021-09-08 Thread Michael McCandless (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17411880#comment-17411880 ] Michael McCandless commented on LUCENE-9662: {quote}it seems like it's ok fo

[GitHub] [lucene] jpountz commented on a change in pull request #286: LUCENE-10089 Disable numeric sort optim when needed

2021-09-08 Thread GitBox
jpountz commented on a change in pull request #286: URL: https://github.com/apache/lucene/pull/286#discussion_r704166498 ## File path: lucene/core/src/java/org/apache/lucene/search/SortField.java ## @@ -606,4 +609,20 @@ public IndexSorter getIndexSorter() { return null