[GitHub] [lucene-solr] dweiss commented on a change in pull request #2068: LUCENE-8982: Separate out native code to another module to allow cpp build with gradle

2020-11-09 Thread GitBox
dweiss commented on a change in pull request #2068: URL: https://github.com/apache/lucene-solr/pull/2068#discussion_r519618160 ## File path: lucene/native/build.gradle ## @@ -0,0 +1,42 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contributor

[GitHub] [lucene-solr] dweiss commented on pull request #2068: LUCENE-8982: Separate out native code to another module to allow cpp build with gradle

2020-11-09 Thread GitBox
dweiss commented on pull request #2068: URL: https://github.com/apache/lucene-solr/pull/2068#issuecomment-723842876 bq. The windows equivalentWindowsDirectory.cpp still sits in misc module and hasn't been moved over yet. Ah... I didn't realize this is the case then - sorry. Give me a

[GitHub] [lucene-solr] bruno-roustant commented on a change in pull request #2066: SOLR-14975: Optimize CoreContainer.getAllCoreNames and getLoadedCoreNames.

2020-11-09 Thread GitBox
bruno-roustant commented on a change in pull request #2066: URL: https://github.com/apache/lucene-solr/pull/2066#discussion_r519684814 ## File path: solr/core/src/java/org/apache/solr/core/SolrCores.java ## @@ -490,21 +492,19 @@ public CoreDescriptor getCoreDescriptor(String co

[jira] [Commented] (SOLR-14683) Review the metrics API to ensure consistent placeholders for missing values

2020-11-09 Thread Andrzej Bialecki (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14683?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228512#comment-17228512 ] Andrzej Bialecki commented on SOLR-14683: - {quote}Solr's JSON Response writer alr

[jira] [Comment Edited] (SOLR-14683) Review the metrics API to ensure consistent placeholders for missing values

2020-11-09 Thread Andrzej Bialecki (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14683?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228512#comment-17228512 ] Andrzej Bialecki edited comment on SOLR-14683 at 11/9/20, 10:57 AM: ---

[jira] [Comment Edited] (SOLR-14683) Review the metrics API to ensure consistent placeholders for missing values

2020-11-09 Thread Andrzej Bialecki (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14683?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228512#comment-17228512 ] Andrzej Bialecki edited comment on SOLR-14683 at 11/9/20, 10:58 AM: ---

[jira] [Commented] (SOLR-14969) Prevent creating multiple cores with the same name which leads to instabilities (race condition)

2020-11-09 Thread Andreas Hubold (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14969?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228529#comment-17228529 ] Andreas Hubold commented on SOLR-14969: --- I've tested concurrent core creation for b

[GitHub] [lucene-solr] bruno-roustant commented on pull request #2066: SOLR-14975: Optimize CoreContainer.getAllCoreNames and getLoadedCoreNames.

2020-11-09 Thread GitBox
bruno-roustant commented on pull request #2066: URL: https://github.com/apache/lucene-solr/pull/2066#issuecomment-723960636 Ok, now I think I integrated all comments. This is an automated message from the Apache Git Service.

[GitHub] [lucene-solr] ErickErickson commented on pull request #2066: SOLR-14975: Optimize CoreContainer.getAllCoreNames and getLoadedCoreNames.

2020-11-09 Thread GitBox
ErickErickson commented on pull request #2066: URL: https://github.com/apache/lucene-solr/pull/2066#issuecomment-723990436 +1 > On Nov 8, 2020, at 3:30 PM, David Smiley wrote: > > > So @dsmiley @ErickErickson do we all agree that we throw an exception in in TransientSolrC

[GitHub] [lucene-solr] dsmiley commented on a change in pull request #2066: SOLR-14975: Optimize CoreContainer.getAllCoreNames and getLoadedCoreNames.

2020-11-09 Thread GitBox
dsmiley commented on a change in pull request #2066: URL: https://github.com/apache/lucene-solr/pull/2066#discussion_r519792336 ## File path: solr/core/src/java/org/apache/solr/core/SolrCores.java ## @@ -121,8 +101,8 @@ protected void close() { // make a copy of the co

[jira] [Commented] (SOLR-14969) Prevent creating multiple cores with the same name which leads to instabilities (race condition)

2020-11-09 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14969?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228572#comment-17228572 ] ASF subversion and git services commented on SOLR-14969: Commit b

[jira] [Commented] (SOLR-14969) Prevent creating multiple cores with the same name which leads to instabilities (race condition)

2020-11-09 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14969?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228583#comment-17228583 ] ASF subversion and git services commented on SOLR-14969: Commit 9

[jira] [Commented] (SOLR-14969) Prevent creating multiple cores with the same name which leads to instabilities (race condition)

2020-11-09 Thread Andreas Hubold (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14969?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228593#comment-17228593 ] Andreas Hubold commented on SOLR-14969: --- Works, no logged ERROR messages anymore. T

[jira] [Created] (SOLR-14991) tag and remove obsolete branches

2020-11-09 Thread Erick Erickson (Jira)
Erick Erickson created SOLR-14991: - Summary: tag and remove obsolete branches Key: SOLR-14991 URL: https://issues.apache.org/jira/browse/SOLR-14991 Project: Solr Issue Type: Improvement

[GitHub] [lucene-solr] mikemccand commented on a change in pull request #2052: LUCENE-8982: Make NativeUnixDirectory pure java with FileChannel direct IO flag, and rename to DirectIODirectory

2020-11-09 Thread GitBox
mikemccand commented on a change in pull request #2052: URL: https://github.com/apache/lucene-solr/pull/2052#discussion_r519868079 ## File path: lucene/misc/src/java/org/apache/lucene/store/DirectIODirectory.java ## @@ -66,45 +66,32 @@ * * @lucene.experimental */ -public

[GitHub] [lucene-solr] mikemccand commented on a change in pull request #2022: LUCENE-9004: KNN vector search using NSW graphs

2020-11-09 Thread GitBox
mikemccand commented on a change in pull request #2022: URL: https://github.com/apache/lucene-solr/pull/2022#discussion_r519873455 ## File path: lucene/core/src/java/org/apache/lucene/index/VectorValues.java ## @@ -74,6 +74,18 @@ public BytesRef binaryValue() throws IOException

[jira] [Commented] (LUCENE-9378) Configurable compression for BinaryDocValues

2020-11-09 Thread Michael McCandless (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9378?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228642#comment-17228642 ] Michael McCandless commented on LUCENE-9378: {quote}IMO this is so definitel

[GitHub] [lucene-solr] mikemccand commented on pull request #1543: LUCENE-9378: Disable compression on binary values whose length is less than 32.

2020-11-09 Thread GitBox
mikemccand commented on pull request #1543: URL: https://github.com/apache/lucene-solr/pull/1543#issuecomment-724069358 > > But then I wonder why not just add a boolean compress option to Lucene80DocValuesFormat? This is similar to the compression Mode we pass to stored fields and term vec

[GitHub] [lucene-solr] mikemccand commented on a change in pull request #2022: LUCENE-9004: KNN vector search using NSW graphs

2020-11-09 Thread GitBox
mikemccand commented on a change in pull request #2022: URL: https://github.com/apache/lucene-solr/pull/2022#discussion_r519881970 ## File path: lucene/core/src/java/org/apache/lucene/util/hnsw/HnswGraph.java ## @@ -0,0 +1,235 @@ +/* + * Licensed to the Apache Software Foundati

[jira] [Commented] (LUCENE-9378) Configurable compression for BinaryDocValues

2020-11-09 Thread Adrien Grand (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9378?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228665#comment-17228665 ] Adrien Grand commented on LUCENE-9378: -- I'll be looking into it for 8.8. > Configu

[GitHub] [lucene-solr] msokolov merged pull request #2037: LUCENE-9583: extract separate RandomAccessVectorValues interface

2020-11-09 Thread GitBox
msokolov merged pull request #2037: URL: https://github.com/apache/lucene-solr/pull/2037 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 g

[jira] [Commented] (LUCENE-9583) How should we expose VectorValues.RandomAccess?

2020-11-09 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9583?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228677#comment-17228677 ] ASF subversion and git services commented on LUCENE-9583: - Commi

[GitHub] [lucene-solr] mikemccand commented on a change in pull request #2022: LUCENE-9004: KNN vector search using NSW graphs

2020-11-09 Thread GitBox
mikemccand commented on a change in pull request #2022: URL: https://github.com/apache/lucene-solr/pull/2022#discussion_r519895947 ## File path: lucene/core/src/java/org/apache/lucene/codecs/lucene90/Lucene90VectorReader.java ## @@ -22,39 +22,60 @@ import java.nio.FloatBuffer

[jira] [Commented] (LUCENE-9378) Configurable compression for BinaryDocValues

2020-11-09 Thread Michael McCandless (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9378?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228681#comment-17228681 ] Michael McCandless commented on LUCENE-9378: {quote}I'll be looking into it

[GitHub] [lucene-solr] asfgit closed pull request #972: SOLR-13452: Update the lucene-solr build from Ivy+Ant+Maven (shadow build) to Gradle.

2020-11-09 Thread GitBox
asfgit closed pull request #972: URL: https://github.com/apache/lucene-solr/pull/972 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

[jira] [Commented] (SOLR-14991) tag and remove obsolete branches

2020-11-09 Thread Erick Erickson (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14991?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228687#comment-17228687 ] Erick Erickson commented on SOLR-14991: --- I just did the grade branches, with the ex

[GitHub] [lucene-solr] bruno-roustant commented on pull request #2066: SOLR-14975: Optimize CoreContainer.getAllCoreNames and getLoadedCoreNames.

2020-11-09 Thread GitBox
bruno-roustant commented on pull request #2066: URL: https://github.com/apache/lucene-solr/pull/2066#issuecomment-724113057 @dsmiley I pushed a new commit to have a NO OP TransientSolrCacheFactory before SolrCores.load() is called. This is to have your opinion, whether to keep it or remo

[jira] [Updated] (SOLR-14683) Review the metrics API to ensure consistent placeholders for missing values

2020-11-09 Thread Andrzej Bialecki (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14683?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrzej Bialecki updated SOLR-14683: Attachment: SOLR-14683.patch > Review the metrics API to ensure consistent placeholders for

[jira] [Commented] (SOLR-14951) Upgrade Angular JS 1.7.9 to 1.8.0

2020-11-09 Thread Kevin Risden (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14951?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228695#comment-17228695 ] Kevin Risden commented on SOLR-14951: - Sigh I missed merging it. I just saw the revie

[jira] [Updated] (SOLR-14951) Upgrade Angular JS 1.7.9 to 1.8.0

2020-11-09 Thread Kevin Risden (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14951?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kevin Risden updated SOLR-14951: Fix Version/s: 8.8 > Upgrade Angular JS 1.7.9 to 1.8.0 > - > >

[GitHub] [lucene-solr] cpoerschke commented on a change in pull request #1571: SOLR-14560: Interleaving for Learning To Rank

2020-11-09 Thread GitBox
cpoerschke commented on a change in pull request #1571: URL: https://github.com/apache/lucene-solr/pull/1571#discussion_r519944462 ## File path: solr/solr-ref-guide/src/learning-to-rank.adoc ## @@ -247,6 +254,81 @@ The output XML will include feature values as a comma-separate

[GitHub] [lucene-solr] cpoerschke commented on a change in pull request #1571: SOLR-14560: Interleaving for Learning To Rank

2020-11-09 Thread GitBox
cpoerschke commented on a change in pull request #1571: URL: https://github.com/apache/lucene-solr/pull/1571#discussion_r519967315 ## File path: solr/contrib/ltr/src/java/org/apache/solr/ltr/interleaving/TeamDraftInterleaving.java ## @@ -0,0 +1,121 @@ +/* + * Licensed to the A

[jira] [Created] (SOLR-14992) TestPullReplicaErrorHandling.testCantConnectToPullReplica Failures

2020-11-09 Thread Tomas Eduardo Fernandez Lobbe (Jira)
Tomas Eduardo Fernandez Lobbe created SOLR-14992: Summary: TestPullReplicaErrorHandling.testCantConnectToPullReplica Failures Key: SOLR-14992 URL: https://issues.apache.org/jira/browse/SOLR-14992

[GitHub] [lucene-solr] muse-dev[bot] commented on a change in pull request #2066: SOLR-14975: Optimize CoreContainer.getAllCoreNames and getLoadedCoreNames.

2020-11-09 Thread GitBox
muse-dev[bot] commented on a change in pull request #2066: URL: https://github.com/apache/lucene-solr/pull/2066#discussion_r519994863 ## File path: solr/core/src/java/org/apache/solr/core/SolrCores.java ## @@ -62,55 +51,44 @@ // to essentially queue them up to be handled via

[jira] [Commented] (LUCENE-9322) Discussing a unified vectors format API

2020-11-09 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9322?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228742#comment-17228742 ] ASF subversion and git services commented on LUCENE-9322: - Commi

[GitHub] [lucene-solr] jpountz opened a new pull request #2069: LUCENE-9378: Make it possible to configure how to trade speed for compression on doc values.

2020-11-09 Thread GitBox
jpountz opened a new pull request #2069: URL: https://github.com/apache/lucene-solr/pull/2069 This adds a switch to `Lucene80DocValuesFormat` which allows to configure whether to prioritize retrieval speed over compression ratio or the other way around. When prioritizing retrieval spee

[jira] [Created] (LUCENE-9602) TestBackwardsCompatibility should test BEST_COMPRESSION

2020-11-09 Thread Adrien Grand (Jira)
Adrien Grand created LUCENE-9602: Summary: TestBackwardsCompatibility should test BEST_COMPRESSION Key: LUCENE-9602 URL: https://issues.apache.org/jira/browse/LUCENE-9602 Project: Lucene - Core

[GitHub] [lucene-solr] mikemccand commented on a change in pull request #2069: LUCENE-9378: Make it possible to configure how to trade speed for compression on doc values.

2020-11-09 Thread GitBox
mikemccand commented on a change in pull request #2069: URL: https://github.com/apache/lucene-solr/pull/2069#discussion_r520024634 ## File path: lucene/core/src/test/org/apache/lucene/codecs/lucene80/BaseLucene80DocValuesFormatTestCase.java ## @@ -286,7 +278,7 @@ private void

[GitHub] [lucene-solr] dsmiley commented on a change in pull request #2066: SOLR-14975: Optimize CoreContainer.getAllCoreNames and getLoadedCoreNames.

2020-11-09 Thread GitBox
dsmiley commented on a change in pull request #2066: URL: https://github.com/apache/lucene-solr/pull/2066#discussion_r520029778 ## File path: solr/core/src/java/org/apache/solr/core/SolrCores.java ## @@ -51,7 +51,7 @@ // to essentially queue them up to be handled via pending

[GitHub] [lucene-solr] mikemccand commented on a change in pull request #2069: LUCENE-9378: Make it possible to configure how to trade speed for compression on doc values.

2020-11-09 Thread GitBox
mikemccand commented on a change in pull request #2069: URL: https://github.com/apache/lucene-solr/pull/2069#discussion_r520031264 ## File path: lucene/core/src/test/org/apache/lucene/codecs/lucene80/TestBestSpeedLucene80DocValuesFormat.java ## @@ -0,0 +1,33 @@ +/* + * License

[GitHub] [lucene-solr] jpountz commented on a change in pull request #2069: LUCENE-9378: Make it possible to configure how to trade speed for compression on doc values.

2020-11-09 Thread GitBox
jpountz commented on a change in pull request #2069: URL: https://github.com/apache/lucene-solr/pull/2069#discussion_r520033652 ## File path: lucene/core/src/test/org/apache/lucene/codecs/lucene80/TestBestSpeedLucene80DocValuesFormat.java ## @@ -0,0 +1,33 @@ +/* + * Licensed t

[GitHub] [lucene-solr] jpountz commented on a change in pull request #2069: LUCENE-9378: Make it possible to configure how to trade speed for compression on doc values.

2020-11-09 Thread GitBox
jpountz commented on a change in pull request #2069: URL: https://github.com/apache/lucene-solr/pull/2069#discussion_r520034757 ## File path: lucene/core/src/test/org/apache/lucene/codecs/lucene80/BaseLucene80DocValuesFormatTestCase.java ## @@ -286,7 +278,7 @@ private void doT

[GitHub] [lucene-solr] jpountz commented on a change in pull request #2069: LUCENE-9378: Make it possible to configure how to trade speed for compression on doc values.

2020-11-09 Thread GitBox
jpountz commented on a change in pull request #2069: URL: https://github.com/apache/lucene-solr/pull/2069#discussion_r520035432 ## File path: lucene/backward-codecs/src/java/org/apache/lucene/backward_codecs/lucene87/Lucene87Codec.java ## @@ -56,6 +57,23 @@ * @lucene.experim

[GitHub] [lucene-solr] alessandrobenedetti commented on a change in pull request #1571: SOLR-14560: Interleaving for Learning To Rank

2020-11-09 Thread GitBox
alessandrobenedetti commented on a change in pull request #1571: URL: https://github.com/apache/lucene-solr/pull/1571#discussion_r520038324 ## File path: solr/contrib/ltr/src/java/org/apache/solr/ltr/response/transform/LTRFeatureLoggerTransformerFactory.java ## @@ -210,50 +216

[GitHub] [lucene-solr] bruno-roustant commented on a change in pull request #2066: SOLR-14975: Optimize CoreContainer.getAllCoreNames and getLoadedCoreNames.

2020-11-09 Thread GitBox
bruno-roustant commented on a change in pull request #2066: URL: https://github.com/apache/lucene-solr/pull/2066#discussion_r520040508 ## File path: solr/core/src/java/org/apache/solr/core/SolrCores.java ## @@ -51,7 +51,7 @@ // to essentially queue them up to be handled via

[jira] [Updated] (SOLR-14683) Review the metrics API to ensure consistent placeholders for missing values

2020-11-09 Thread Andrzej Bialecki (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14683?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrzej Bialecki updated SOLR-14683: Attachment: SOLR-14683.patch > Review the metrics API to ensure consistent placeholders for

[jira] [Commented] (SOLR-14683) Review the metrics API to ensure consistent placeholders for missing values

2020-11-09 Thread Andrzej Bialecki (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14683?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228768#comment-17228768 ] Andrzej Bialecki commented on SOLR-14683: - This patch adds configurable placehold

[jira] [Commented] (SOLR-14973) Solr 8.6 is shipping libraries that are incompatible with each other

2020-11-09 Thread Kevin Risden (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14973?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228769#comment-17228769 ] Kevin Risden commented on SOLR-14973: - FYI [~tallison] - not sure who updated Tika li

[GitHub] [lucene-solr] bruno-roustant commented on a change in pull request #2066: SOLR-14975: Optimize CoreContainer.getAllCoreNames and getLoadedCoreNames.

2020-11-09 Thread GitBox
bruno-roustant commented on a change in pull request #2066: URL: https://github.com/apache/lucene-solr/pull/2066#discussion_r520049002 ## File path: solr/core/src/java/org/apache/solr/core/SolrCores.java ## @@ -62,55 +51,44 @@ // to essentially queue them up to be handled vi

[GitHub] [lucene-solr] mikemccand commented on a change in pull request #2069: LUCENE-9378: Make it possible to configure how to trade speed for compression on doc values.

2020-11-09 Thread GitBox
mikemccand commented on a change in pull request #2069: URL: https://github.com/apache/lucene-solr/pull/2069#discussion_r520055341 ## File path: lucene/backward-codecs/src/java/org/apache/lucene/backward_codecs/lucene87/Lucene87Codec.java ## @@ -56,6 +57,23 @@ * @lucene.expe

[GitHub] [lucene-solr] alessandrobenedetti commented on a change in pull request #1571: SOLR-14560: Interleaving for Learning To Rank

2020-11-09 Thread GitBox
alessandrobenedetti commented on a change in pull request #1571: URL: https://github.com/apache/lucene-solr/pull/1571#discussion_r520055346 ## File path: solr/contrib/ltr/src/java/org/apache/solr/ltr/response/transform/LTRFeatureLoggerTransformerFactory.java ## @@ -210,50 +216

[GitHub] [lucene-solr] dsmiley commented on a change in pull request #2066: SOLR-14975: Optimize CoreContainer.getAllCoreNames and getLoadedCoreNames.

2020-11-09 Thread GitBox
dsmiley commented on a change in pull request #2066: URL: https://github.com/apache/lucene-solr/pull/2066#discussion_r520057072 ## File path: solr/core/src/java/org/apache/solr/core/SolrCores.java ## @@ -51,7 +51,7 @@ // to essentially queue them up to be handled via pending

[GitHub] [lucene-solr] dsmiley commented on a change in pull request #2066: SOLR-14975: Optimize CoreContainer.getAllCoreNames and getLoadedCoreNames.

2020-11-09 Thread GitBox
dsmiley commented on a change in pull request #2066: URL: https://github.com/apache/lucene-solr/pull/2066#discussion_r520058820 ## File path: solr/core/src/java/org/apache/solr/core/SolrCores.java ## @@ -536,7 +538,9 @@ public void queueCoreToClose(SolrCore coreToClose) { *

[jira] [Commented] (SOLR-14973) Solr 8.6 is shipping libraries that are incompatible with each other

2020-11-09 Thread Tim Allison (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14973?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228776#comment-17228776 ] Tim Allison commented on SOLR-14973: Thank you [~krisden] for the ping. > Solr 8.6

[GitHub] [lucene-solr] bruno-roustant commented on a change in pull request #2066: SOLR-14975: Optimize CoreContainer.getAllCoreNames and getLoadedCoreNames.

2020-11-09 Thread GitBox
bruno-roustant commented on a change in pull request #2066: URL: https://github.com/apache/lucene-solr/pull/2066#discussion_r520089739 ## File path: solr/core/src/java/org/apache/solr/core/SolrCores.java ## @@ -536,7 +538,9 @@ public void queueCoreToClose(SolrCore coreToClose)

[GitHub] [lucene-solr] dsmiley commented on a change in pull request #2066: SOLR-14975: Optimize CoreContainer.getAllCoreNames and getLoadedCoreNames.

2020-11-09 Thread GitBox
dsmiley commented on a change in pull request #2066: URL: https://github.com/apache/lucene-solr/pull/2066#discussion_r520092229 ## File path: solr/core/src/java/org/apache/solr/core/SolrCores.java ## @@ -536,7 +538,9 @@ public void queueCoreToClose(SolrCore coreToClose) { *

[GitHub] [lucene-solr] dweiss commented on pull request #2020: SOLR-14949: Ability to customize Solr Docker build

2020-11-09 Thread GitBox
dweiss commented on pull request #2020: URL: https://github.com/apache/lucene-solr/pull/2020#issuecomment-724260281 Hmm... sorry, missed your mention/ request somehow. Yeah - these functions are intended to read variables from multiple locations so it looks ok to me. I didn't test it (or u

[GitHub] [lucene-solr] HoustonPutman commented on pull request #2020: SOLR-14949: Ability to customize Solr Docker build

2020-11-09 Thread GitBox
HoustonPutman commented on pull request #2020: URL: https://github.com/apache/lucene-solr/pull/2020#issuecomment-724264582 Thanks for the sanity check! I've tested pretty thoroughly, and the PR test does some checks on its own too. --

[jira] [Created] (SOLR-14993) Unable to download zookeeper files of 1byte in size

2020-11-09 Thread Allen Sooredoo (Jira)
Allen Sooredoo created SOLR-14993: - Summary: Unable to download zookeeper files of 1byte in size Key: SOLR-14993 URL: https://issues.apache.org/jira/browse/SOLR-14993 Project: Solr Issue Type

[GitHub] [lucene-solr] dweiss commented on pull request #2068: LUCENE-8982: Separate out native code to another module to allow cpp build with gradle

2020-11-09 Thread GitBox
dweiss commented on pull request #2068: URL: https://github.com/apache/lucene-solr/pull/2068#issuecomment-724283052 Zach I've cleaned up the native build a bit - moved it under lucene/misc, added Windows build (it does build the native library for me). I didn't check whether it works on a

[GitHub] [lucene-solr] dweiss commented on a change in pull request #2068: LUCENE-8982: Separate out native code to another module to allow cpp build with gradle

2020-11-09 Thread GitBox
dweiss commented on a change in pull request #2068: URL: https://github.com/apache/lucene-solr/pull/2068#discussion_r520125532 ## File path: lucene/misc/native/build.gradle ## @@ -0,0 +1,69 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * contri

[GitHub] [lucene-solr] jtibshirani opened a new pull request #2070: LUCENE-9536: Correct the OrdinalMap optimization.

2020-11-09 Thread GitBox
jtibshirani opened a new pull request #2070: URL: https://github.com/apache/lucene-solr/pull/2070 Previously we only checked that the first segment's ordinal deltas were all zero. This didn't account for some rare cases where some of the segment's ordinals are filtered out, so the ordi

[GitHub] [lucene-solr] dweiss commented on pull request #2068: LUCENE-8982: Separate out native code to another module to allow cpp build with gradle

2020-11-09 Thread GitBox
dweiss commented on pull request #2068: URL: https://github.com/apache/lucene-solr/pull/2068#issuecomment-724287441 Byt. if we do have to add that explicit 'build.native' option then I'd implement it as a task (graph) exclusion rather than project exclusion. Windows users in particular may

[GitHub] [lucene-solr] jtibshirani commented on pull request #2070: LUCENE-9536: Correct the OrdinalMap optimization.

2020-11-09 Thread GitBox
jtibshirani commented on pull request #2070: URL: https://github.com/apache/lucene-solr/pull/2070#issuecomment-724288495 This should fix the failures we're seeing like `TestLucene70DocValuesFormat#testSparseSortedVariableLengthVsStoredFields` and `TestSimpleTextDocValuesFormat#testSparseSo

[jira] [Created] (SOLR-14994) Bring in Solr Operator into the Lucene project

2020-11-09 Thread Anshum Gupta (Jira)
Anshum Gupta created SOLR-14994: --- Summary: Bring in Solr Operator into the Lucene project Key: SOLR-14994 URL: https://issues.apache.org/jira/browse/SOLR-14994 Project: Solr Issue Type: Task

[jira] [Commented] (SOLR-14788) Solr: The Next Big Thing

2020-11-09 Thread Mark Robert Miller (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228852#comment-17228852 ] Mark Robert Miller commented on SOLR-14788: --- This Overseer work (release it fro

[jira] [Comment Edited] (SOLR-14788) Solr: The Next Big Thing

2020-11-09 Thread Mark Robert Miller (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228852#comment-17228852 ] Mark Robert Miller edited comment on SOLR-14788 at 11/9/20, 11:16 PM: -

[GitHub] [lucene-solr] madrob commented on a change in pull request #2067: SOLR-14987: Reuse HttpSolrClient per node vs. one per Solr core when using CloudSolrStream

2020-11-09 Thread GitBox
madrob commented on a change in pull request #2067: URL: https://github.com/apache/lucene-solr/pull/2067#discussion_r520198395 ## File path: solr/solrj/src/java/org/apache/solr/client/solrj/io/stream/CloudSolrStream.java ## @@ -334,11 +334,6 @@ private StreamComparator parseCo

[GitHub] [lucene-solr] jtibshirani opened a new pull request #2071: LUCENE-9322: Some fixes to SimpleTextVectorFormat.

2020-11-09 Thread GitBox
jtibshirani opened a new pull request #2071: URL: https://github.com/apache/lucene-solr/pull/2071 * Make sure the files are unique by renaming the term vectors extension to `tvc`. * Fix a bug where reading a vector would drop the leading digit of the first element. ---

[GitHub] [lucene-solr] jtibshirani commented on pull request #2071: LUCENE-9322: Some fixes to SimpleTextVectorFormat.

2020-11-09 Thread GitBox
jtibshirani commented on pull request #2071: URL: https://github.com/apache/lucene-solr/pull/2071#issuecomment-724386681 I found these issues while fixing the following failing test: ``` ./gradlew test --tests TestSortingCodecReader.testSortOnAddIndicesRandom -Dtests.seed=B38EBA45

[GitHub] [lucene-solr] msokolov commented on a change in pull request #2071: LUCENE-9322: Some fixes to SimpleTextVectorFormat.

2020-11-09 Thread GitBox
msokolov commented on a change in pull request #2071: URL: https://github.com/apache/lucene-solr/pull/2071#discussion_r520253512 ## File path: lucene/codecs/src/java/org/apache/lucene/codecs/simpletext/SimpleTextVectorReader.java ## @@ -245,8 +245,8 @@ private void readAllVect

[GitHub] [lucene-solr] msokolov merged pull request #2071: LUCENE-9322: Some fixes to SimpleTextVectorFormat.

2020-11-09 Thread GitBox
msokolov merged pull request #2071: URL: https://github.com/apache/lucene-solr/pull/2071 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 g

[jira] [Commented] (LUCENE-9322) Discussing a unified vectors format API

2020-11-09 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9322?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228945#comment-17228945 ] ASF subversion and git services commented on LUCENE-9322: - Commi

[jira] [Commented] (LUCENE-9322) Discussing a unified vectors format API

2020-11-09 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9322?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228951#comment-17228951 ] ASF subversion and git services commented on LUCENE-9322: - Commi

[GitHub] [lucene-solr] zacharymorn commented on a change in pull request #2052: LUCENE-8982: Make NativeUnixDirectory pure java with FileChannel direct IO flag, and rename to DirectIODirectory

2020-11-09 Thread GitBox
zacharymorn commented on a change in pull request #2052: URL: https://github.com/apache/lucene-solr/pull/2052#discussion_r520268743 ## File path: lucene/misc/src/java/org/apache/lucene/store/DirectIODirectory.java ## @@ -66,45 +66,32 @@ * * @lucene.experimental */ -public

[GitHub] [lucene-solr] zacharymorn commented on a change in pull request #2052: LUCENE-8982: Make NativeUnixDirectory pure java with FileChannel direct IO flag, and rename to DirectIODirectory

2020-11-09 Thread GitBox
zacharymorn commented on a change in pull request #2052: URL: https://github.com/apache/lucene-solr/pull/2052#discussion_r520268743 ## File path: lucene/misc/src/java/org/apache/lucene/store/DirectIODirectory.java ## @@ -66,45 +66,32 @@ * * @lucene.experimental */ -public

[GitHub] [lucene-solr] zacharymorn commented on a change in pull request #2068: LUCENE-8982: Separate out native code to another module to allow cpp build with gradle

2020-11-09 Thread GitBox
zacharymorn commented on a change in pull request #2068: URL: https://github.com/apache/lucene-solr/pull/2068#discussion_r520270563 ## File path: lucene/misc/native/build.gradle ## @@ -0,0 +1,69 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * c

[GitHub] [lucene-solr] zacharymorn commented on pull request #2068: LUCENE-8982: Separate out native code to another module to allow cpp build with gradle

2020-11-09 Thread GitBox
zacharymorn commented on pull request #2068: URL: https://github.com/apache/lucene-solr/pull/2068#issuecomment-724434703 > Zach I've cleaned up the native build a bit - moved it under lucene/misc, added Windows build (it does build the native library for me). I didn't check whether it work

[GitHub] [lucene-solr] zacharymorn commented on a change in pull request #2068: LUCENE-8982: Separate out native code to another module to allow cpp build with gradle

2020-11-09 Thread GitBox
zacharymorn commented on a change in pull request #2068: URL: https://github.com/apache/lucene-solr/pull/2068#discussion_r520273368 ## File path: lucene/misc/native/build.gradle ## @@ -0,0 +1,69 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + * c

[jira] [Commented] (LUCENE-9590) Add javadoc for Lucene86PointsFormat class

2020-11-09 Thread Lu Xugang (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9590?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17228980#comment-17228980 ] Lu Xugang commented on LUCENE-9590: --- here the link:https://www.amazingkoala.com.cn/Lu