[GitHub] [lucene-solr] dsmiley commented on a change in pull request #2306: SOLR-15121: Move XSLT (tr param) to scripting contrib

2021-02-05 Thread GitBox
dsmiley commented on a change in pull request #2306: URL: https://github.com/apache/lucene-solr/pull/2306#discussion_r571376095 ## File path: solr/contrib/scripting/src/java/org/apache/solr/scripting/util/xslt/XSLTLoader.java ## @@ -0,0 +1,534 @@ +/* + * Licensed to the

[GitHub] [lucene-solr] dsmiley commented on a change in pull request #2306: SOLR-15121: Move XSLT (tr param) to scripting contrib

2021-02-05 Thread GitBox
dsmiley commented on a change in pull request #2306: URL: https://github.com/apache/lucene-solr/pull/2306#discussion_r571375991 ## File path: solr/contrib/scripting/src/java/org/apache/solr/scripting/util/xslt/XSLTLoader.java ## @@ -0,0 +1,534 @@ +/* + * Licensed to the

[GitHub] [lucene-solr] dsmiley commented on a change in pull request #2306: SOLR-15121: Move XSLT (tr param) to scripting contrib

2021-02-05 Thread GitBox
dsmiley commented on a change in pull request #2306: URL: https://github.com/apache/lucene-solr/pull/2306#discussion_r571368100 ## File path: solr/contrib/scripting/src/java/org/apache/solr/scripting/response/package-info.java ## @@ -0,0 +1,21 @@ +/* + * Licensed to the

[jira] [Commented] (SOLR-7913) Add stream.body support to MLT QParser

2021-02-05 Thread Isabelle Giguere (Jira)
[ https://issues.apache.org/jira/browse/SOLR-7913?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17280064#comment-17280064 ] Isabelle Giguere commented on SOLR-7913: Patch on release tag lucene-solr 8.7.0:

[jira] [Commented] (SOLR-15133) Document how to eliminate Failed to reserve shared memory warning

2021-02-05 Thread Shawn Heisey (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15133?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17280063#comment-17280063 ] Shawn Heisey commented on SOLR-15133: - I wish I understood the benefits and disadvantages of huge

[jira] [Created] (SOLR-15140) Avoid insuffcient key size when creating keyPair

2021-02-05 Thread Ying Zhang (Jira)
Ying Zhang created SOLR-15140: - Summary: Avoid insuffcient key size when creating keyPair Key: SOLR-15140 URL: https://issues.apache.org/jira/browse/SOLR-15140 Project: Solr Issue Type:

[jira] [Updated] (SOLR-7913) Add stream.body support to MLT QParser

2021-02-05 Thread Isabelle Giguere (Jira)
[ https://issues.apache.org/jira/browse/SOLR-7913?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Isabelle Giguere updated SOLR-7913: --- Attachment: SOLR-7913_tag_lucene-solr-8.7.0.patch > Add stream.body support to MLT QParser >

[jira] [Commented] (SOLR-15133) Document how to eliminate Failed to reserve shared memory warning

2021-02-05 Thread Mike Drob (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15133?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17280033#comment-17280033 ] Mike Drob commented on SOLR-15133: -- The origin seems to be from [~elyograg]'s wiki page, where he

[GitHub] [lucene-solr] muse-dev[bot] commented on a change in pull request #2306: SOLR-15121: Move XSLT (tr param) to scripting contrib

2021-02-05 Thread GitBox
muse-dev[bot] commented on a change in pull request #2306: URL: https://github.com/apache/lucene-solr/pull/2306#discussion_r571294563 ## File path: solr/contrib/scripting/src/java/org/apache/solr/scripting/util/xslt/XSLTLoader.java ## @@ -0,0 +1,534 @@ +/* + * Licensed to the

[GitHub] [lucene-solr-sandbox] anshumg commented on pull request #2: Setup gradle for sandbox root

2021-02-05 Thread GitBox
anshumg commented on pull request #2: URL: https://github.com/apache/lucene-solr-sandbox/pull/2#issuecomment-774324833 Thank you @madrob This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [lucene-solr-sandbox] anshumg merged pull request #2: Setup gradle for sandbox root

2021-02-05 Thread GitBox
anshumg merged pull request #2: URL: https://github.com/apache/lucene-solr-sandbox/pull/2 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

[jira] [Commented] (SOLR-15127) All-In-One Dockerfile for building local images as well as reproducible release builds directly from (remote) git tags

2021-02-05 Thread Houston Putman (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15127?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17280023#comment-17280023 ] Houston Putman commented on SOLR-15127: --- I think having a conditional RUN in the Dockerfile to

[jira] [Commented] (SOLR-15121) Move XSLT (tr param) to scripting contrib

2021-02-05 Thread David Eric Pugh (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15121?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17280014#comment-17280014 ] David Eric Pugh commented on SOLR-15121: First cut is DONE!   The one thing I am not happy with

[GitHub] [lucene-solr-sandbox] anshumg commented on pull request #2: Setup gradle for sandbox root

2021-02-05 Thread GitBox
anshumg commented on pull request #2: URL: https://github.com/apache/lucene-solr-sandbox/pull/2#issuecomment-774300094 I've also updated the Gradle wrappers to bring in the fixes from lucene-solr. This is an automated

[GitHub] [lucene-solr-sandbox] madrob commented on pull request #2: Setup gradle for sandbox root

2021-02-05 Thread GitBox
madrob commented on pull request #2: URL: https://github.com/apache/lucene-solr-sandbox/pull/2#issuecomment-774289125 On second thought, since we don't know what/how we're going to be releasing from here, let's not worry about it until we need to.

[GitHub] [lucene-solr] madrob commented on pull request #2309: SOLR-14301: Remove external commons-codec usage in gradle validateJarChecksums

2021-02-05 Thread GitBox
madrob commented on pull request #2309: URL: https://github.com/apache/lucene-solr/pull/2309#issuecomment-774283250 I _believe_ this was done with an external dependency because there's an issue with how checksums were handled on Windows, but I don't remember all the details and can't

[GitHub] [lucene-solr-sandbox] anshumg commented on pull request #2: Setup gradle for sandbox root

2021-02-05 Thread GitBox
anshumg commented on pull request #2: URL: https://github.com/apache/lucene-solr-sandbox/pull/2#issuecomment-774281078 @madrob - let me just copy them over right now in that case? This is an automated message from the Apache

[GitHub] [lucene-solr-sandbox] madrob commented on pull request #2: Setup gradle for sandbox root

2021-02-05 Thread GitBox
madrob commented on pull request #2: URL: https://github.com/apache/lucene-solr-sandbox/pull/2#issuecomment-774278374 It's fine to check in the jar for now, but we should instead copy the modified gradlew scripts that we have on lucene-solr. There's a few extra workarounds in there -

[jira] [Resolved] (SOLR-15123) bin/solr tools all have different styles of describing command opitons

2021-02-05 Thread David Eric Pugh (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15123?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] David Eric Pugh resolved SOLR-15123. Fix Version/s: 8.9 Resolution: Fixed > bin/solr tools all have different styles of

[jira] [Commented] (SOLR-15123) bin/solr tools all have different styles of describing command opitons

2021-02-05 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17279971#comment-17279971 ] ASF subversion and git services commented on SOLR-15123: Commit

[GitHub] [lucene-solr] epugh merged pull request #2275: SOLR-15123: Make all Tool option descriptions follow the same general pattern.

2021-02-05 Thread GitBox
epugh merged pull request #2275: URL: https://github.com/apache/lucene-solr/pull/2275 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

[GitHub] [lucene-solr] jtibshirani commented on pull request #2310: LUCENE-9705: Create Lucene90PostingsFormat

2021-02-05 Thread GitBox
jtibshirani commented on pull request #2310: URL: https://github.com/apache/lucene-solr/pull/2310#issuecomment-774254300 I also plan to copy over the blocktree dependencies like `BlockTreeTermsWriter`. Any thoughts on naming or package structure for these classes? In

[GitHub] [lucene-solr-sandbox] anshumg opened a new pull request #2: Setup gradle for sandbox root

2021-02-05 Thread GitBox
anshumg opened a new pull request #2: URL: https://github.com/apache/lucene-solr-sandbox/pull/2 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-solr] jtibshirani opened a new pull request #2310: LUCENE-9705: Create Lucene90PostingsFormat

2021-02-05 Thread GitBox
jtibshirani opened a new pull request #2310: URL: https://github.com/apache/lucene-solr/pull/2310 For now this is just a copy of Lucene90PostingsFormat. The existing Lucene84PostingsFormat was moved to backwards-codecs, along with its utility classes.

[GitHub] [lucene-solr-sandbox] anshumg merged pull request #1: Create .asf.yaml

2021-02-05 Thread GitBox
anshumg merged pull request #1: URL: https://github.com/apache/lucene-solr-sandbox/pull/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 above to

[GitHub] [lucene-solr-operator] HoustonPutman closed pull request #207: docs: removing step in local tutorial for collection creation

2021-02-05 Thread GitBox
HoustonPutman closed pull request #207: URL: https://github.com/apache/lucene-solr-operator/pull/207 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [lucene-solr-operator] HoustonPutman commented on pull request #207: docs: removing step in local tutorial for collection creation

2021-02-05 Thread GitBox
HoustonPutman commented on pull request #207: URL: https://github.com/apache/lucene-solr-operator/pull/207#issuecomment-774221538 Instead, I changed the instructions to use the Collections API to create the collection. That way users have a better understanding of how to interact with

[jira] [Updated] (SOLR-15132) Add window paramater to the nodes Streaming Expression

2021-02-05 Thread Joel Bernstein (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15132?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Joel Bernstein updated SOLR-15132: -- Description: The *nodes* Streaming Expression performs a breadth first graph traversal. This

[jira] [Updated] (SOLR-15132) Add window paramater to the nodes Streaming Expression

2021-02-05 Thread Joel Bernstein (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15132?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Joel Bernstein updated SOLR-15132: -- Description: The *nodes* Streaming Expression performs a breadth first graph traversal. This

[jira] [Updated] (SOLR-15132) Add window paramater to the nodes Streaming Expression

2021-02-05 Thread Joel Bernstein (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15132?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Joel Bernstein updated SOLR-15132: -- Description: The *nodes* Streaming Expression performs a breadth first graph traversal. This

[jira] [Updated] (SOLR-15132) Add window paramater to the nodes Streaming Expression

2021-02-05 Thread Joel Bernstein (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15132?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Joel Bernstein updated SOLR-15132: -- Description: The *nodes* Streaming Expression performs a breadth first graph traversal. This

[jira] [Updated] (SOLR-15132) Add window paramater to the nodes Streaming Expression

2021-02-05 Thread Joel Bernstein (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15132?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Joel Bernstein updated SOLR-15132: -- Description: The *nodes* Streaming Expression performs a breadth first graph traversal. This

[jira] [Updated] (SOLR-15132) Add window paramater to the nodes Streaming Expression

2021-02-05 Thread Joel Bernstein (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15132?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Joel Bernstein updated SOLR-15132: -- Description: The *nodes* Streaming Expression performs a breadth first graph traversal. This

[jira] [Updated] (SOLR-15132) Add window paramater to the nodes Streaming Expression

2021-02-05 Thread Joel Bernstein (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15132?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Joel Bernstein updated SOLR-15132: -- Description: The *nodes* Streaming Expression performs a breadth first graph traversal. This

[jira] [Updated] (SOLR-15132) Add window paramater to the nodes Streaming Expression

2021-02-05 Thread Joel Bernstein (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15132?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Joel Bernstein updated SOLR-15132: -- Description: The *nodes* Streaming Expression performs a breadth first graph traversal. This

[jira] [Updated] (SOLR-15132) Add window paramater to the nodes Streaming Expression

2021-02-05 Thread Joel Bernstein (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15132?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Joel Bernstein updated SOLR-15132: -- Description: The *nodes* Streaming Expression performs a breadth first graph traversal. This

[GitHub] [lucene-solr] epugh commented on a change in pull request #2275: SOLR-15123: Make all Tool option descriptions follow the same general pattern.

2021-02-05 Thread GitBox
epugh commented on a change in pull request #2275: URL: https://github.com/apache/lucene-solr/pull/2275#discussion_r571018503 ## File path: solr/core/src/java/org/apache/solr/util/PackageTool.java ## @@ -261,44 +261,44 @@ protected void runImpl(CommandLine cli) throws

[GitHub] [lucene-solr] bruno-roustant commented on pull request #2302: LUCENE-9663: Adding compression to terms dict from SortedSet/Sorted D…

2021-02-05 Thread GitBox
bruno-roustant commented on pull request #2302: URL: https://github.com/apache/lucene-solr/pull/2302#issuecomment-773991718 @msokolov I confirm it is backward compatible, not thanks to TestDocValuesCompression#testMergeWithUncompressedSegment() but thanks to

[jira] [Created] (LUCENE-9735) Hunspell: speed up flag checks by avoiding allocations

2021-02-05 Thread Peter Gromov (Jira)
Peter Gromov created LUCENE-9735: Summary: Hunspell: speed up flag checks by avoiding allocations Key: LUCENE-9735 URL: https://issues.apache.org/jira/browse/LUCENE-9735 Project: Lucene - Core

[GitHub] [lucene-solr] asalamon74 opened a new pull request #2309: SOLR-14301: Remove external commons-codec usage in gradle validateJarChecksums

2021-02-05 Thread GitBox
asalamon74 opened a new pull request #2309: URL: https://github.com/apache/lucene-solr/pull/2309 …Checksums # Description Right now gradle calculates SHA-1 checksums using an external commons-codec library. We can calculate SHA-1 using Java 8 classes, no need for

[jira] [Comment Edited] (SOLR-15139) Recovering forever after upgrade to 8.8.0: Timeout waiting for collection state

2021-02-05 Thread Henrik (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15139?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17279628#comment-17279628 ] Henrik edited comment on SOLR-15139 at 2/5/21, 11:04 AM: - It seems my timeout

[jira] [Comment Edited] (SOLR-15139) Recovering forever after upgrade to 8.8.0: Timeout waiting for collection state

2021-02-05 Thread Henrik (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15139?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17279628#comment-17279628 ] Henrik edited comment on SOLR-15139 at 2/5/21, 11:03 AM: - It seems I had too

[jira] [Resolved] (SOLR-15139) Recovering forever after upgrade to 8.8.0: Timeout waiting for collection state

2021-02-05 Thread Henrik (Jira)
[ https://issues.apache.org/jira/browse/SOLR-15139?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Henrik resolved SOLR-15139. --- Resolution: Workaround It seems I had too agressive settings for my timeouts. Things started working when I

[GitHub] [lucene-solr] dweiss merged pull request #2301: LUCENE-9732: Hunspell: support dictionary entries starting with slash

2021-02-05 Thread GitBox
dweiss merged pull request #2301: URL: https://github.com/apache/lucene-solr/pull/2301 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

[GitHub] [lucene-solr] noblepaul commented on pull request #2281: SOLR-15118: Convert /v2/collections APIs to annotations

2021-02-05 Thread GitBox
noblepaul commented on pull request #2281: URL: https://github.com/apache/lucene-solr/pull/2281#issuecomment-773232908 Thanks for picking this up @gerlowskija Please let me know if you need any help This is an

[GitHub] [lucene-solr] jaisonbi commented on a change in pull request #2302: LUCENE-9663: Adding compression to terms dict from SortedSet/Sorted D…

2021-02-05 Thread GitBox
jaisonbi commented on a change in pull request #2302: URL: https://github.com/apache/lucene-solr/pull/2302#discussion_r570670271 ## File path: lucene/core/src/java/org/apache/lucene/codecs/lucene80/Lucene80DocValuesConsumer.java ## @@ -43,11 +43,7 @@ import

[GitHub] [lucene-solr] jaisonbi edited a comment on pull request #2213: LUCENE-9663: Adding compression to terms dict from SortedSet/Sorted DocValues

2021-02-05 Thread GitBox
jaisonbi edited a comment on pull request #2213: URL: https://github.com/apache/lucene-solr/pull/2213#issuecomment-772993254 Thanks for the detailed suggestion:) @bruno-roustant Will add a **new custom codec** to support compression or un-compression mode. I name it as

[jira] [Resolved] (LUCENE-9732) Hunspell: support dictionary entries starting with slash

2021-02-05 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9732?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dawid Weiss resolved LUCENE-9732. - Fix Version/s: master (9.0) Resolution: Fixed > Hunspell: support dictionary entries

[GitHub] [lucene-solr] donnerpeter closed pull request #2283: LUCENE-9719: Resource files aren't deleted from build directory after…

2021-02-05 Thread GitBox
donnerpeter closed pull request #2283: URL: https://github.com/apache/lucene-solr/pull/2283 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

[GitHub] [lucene-solr] gerlowskija edited a comment on pull request #2281: SOLR-15118: Convert /v2/collections APIs to annotations

2021-02-05 Thread GitBox
gerlowskija edited a comment on pull request #2281: URL: https://github.com/apache/lucene-solr/pull/2281#issuecomment-773515236 Hey, thanks for offering @noblepaul . I do have one thing I was puzzling over: How is SolrJ's `getV2Request`/`V2RequestSupport` feature intended to work

[GitHub] [lucene-solr] gerlowskija commented on a change in pull request #2281: SOLR-15118: Convert /v2/collections APIs to annotations

2021-02-05 Thread GitBox
gerlowskija commented on a change in pull request #2281: URL: https://github.com/apache/lucene-solr/pull/2281#discussion_r570282351 ## File path: solr/core/src/java/org/apache/solr/handler/CollectionsAPI.java ## @@ -50,11 +81,144 @@ public void getCollections(SolrQueryRequest

[GitHub] [lucene-solr] murblanc commented on pull request #2285: SOLR-14928: introduce distributed cluster state updates

2021-02-05 Thread GitBox
murblanc commented on pull request #2285: URL: https://github.com/apache/lucene-solr/pull/2285#issuecomment-773661420 > Is not possible to make this a per collection feature? like the per-replica state feauture? Given Zookeeper content is identical for Overseer based and distributed

[GitHub] [lucene-solr] dweiss merged pull request #2300: LUCENE-9729: Hunspell: support CHECKCOMPOUNDREP flags

2021-02-05 Thread GitBox
dweiss merged pull request #2300: URL: https://github.com/apache/lucene-solr/pull/2300 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

[GitHub] [lucene-solr] epugh commented on pull request #2306: SOLR-15121: Move XSLT (tr param) to scripting contrib

2021-02-05 Thread GitBox
epugh commented on pull request #2306: URL: https://github.com/apache/lucene-solr/pull/2306#issuecomment-773656218 I still have a broken test, and need to update docs. This is an automated message from the Apache Git

[jira] [Commented] (LUCENE-9732) Hunspell: support dictionary entries starting with slash

2021-02-05 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9732?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17279594#comment-17279594 ] ASF subversion and git services commented on LUCENE-9732: - Commit

[GitHub] [lucene-solr] msokolov commented on pull request #2303: LUCENE-9715: fix int overflow in Lucene90VectorReader

2021-02-05 Thread GitBox
msokolov commented on pull request #2303: URL: https://github.com/apache/lucene-solr/pull/2303#issuecomment-773529165 I like Lucene's randomized tests too! But they didn't find this one. Since it requires a 2GB index to detect, it only turned up in the context of Anton's version of

[GitHub] [lucene-solr] dweiss commented on pull request #2283: LUCENE-9719: Resource files aren't deleted from build directory after…

2021-02-05 Thread GitBox
dweiss commented on pull request #2283: URL: https://github.com/apache/lucene-solr/pull/2283#issuecomment-773300099 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [lucene-solr] jpountz commented on a change in pull request #2293: LUCENE-9725: Allow BM25FQuery to use other similarities.

2021-02-05 Thread GitBox
jpountz commented on a change in pull request #2293: URL: https://github.com/apache/lucene-solr/pull/2293#discussion_r570044511 ## File path: lucene/sandbox/src/java/org/apache/lucene/sandbox/search/CombinedFieldQuery.java ## @@ -64,39 +62,35 @@ * A {@link Query} that

[GitHub] [lucene-solr] msokolov merged pull request #2303: LUCENE-9715: fix int overflow in Lucene90VectorReader

2021-02-05 Thread GitBox
msokolov merged pull request #2303: URL: https://github.com/apache/lucene-solr/pull/2303 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

[GitHub] [lucene-solr] donnerpeter commented on a change in pull request #2283: LUCENE-9719: Resource files aren't deleted from build directory after…

2021-02-05 Thread GitBox
donnerpeter commented on a change in pull request #2283: URL: https://github.com/apache/lucene-solr/pull/2283#discussion_r570083701 ## File path: gradle/ant-compat/folder-layout.gradle ## @@ -25,11 +25,15 @@ allprojects { test.resources.srcDirs = ['src/test-files']

[GitHub] [lucene-solr] gerlowskija commented on pull request #2281: SOLR-15118: Convert /v2/collections APIs to annotations

2021-02-05 Thread GitBox
gerlowskija commented on pull request #2281: URL: https://github.com/apache/lucene-solr/pull/2281#issuecomment-773515236 Hey, thanks for offering @noblepaul . I do have one thing I was puzzling over: How is SolrJ's `getV2Request`/`V2RequestSupport` feature intended to work for

[GitHub] [lucene-solr] gerlowskija merged pull request #2250: SOLR-13608: Incremental backup file format

2021-02-05 Thread GitBox
gerlowskija merged pull request #2250: URL: https://github.com/apache/lucene-solr/pull/2250 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

[GitHub] [lucene-solr] jaisonbi closed pull request #2213: LUCENE-9663: Adding compression to terms dict from SortedSet/Sorted DocValues

2021-02-05 Thread GitBox
jaisonbi closed pull request #2213: URL: https://github.com/apache/lucene-solr/pull/2213 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

[GitHub] [lucene-solr] dweiss commented on pull request #2277: LUCENE-9716: Hunspell: support flag usage before its format is even specified

2021-02-05 Thread GitBox
dweiss commented on pull request #2277: URL: https://github.com/apache/lucene-solr/pull/2277#issuecomment-773476961 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [lucene-solr] donnerpeter commented on a change in pull request #2301: LUCENE-9732: Hunspell: support dictionary entries starting with slash

2021-02-05 Thread GitBox
donnerpeter commented on a change in pull request #2301: URL: https://github.com/apache/lucene-solr/pull/2301#discussion_r570807970 ## File path: lucene/analysis/common/src/java/org/apache/lucene/analysis/hunspell/WordCase.java ## @@ -53,14 +54,19 @@ static WordCase

[GitHub] [lucene-solr] epugh commented on pull request #2304: SOLR-15133: Document how to eliminate Failed to reserve shared memory warning

2021-02-05 Thread GitBox
epugh commented on pull request #2304: URL: https://github.com/apache/lucene-solr/pull/2304#issuecomment-773428402 @ctargett I'd love a quick review of the copy editing I did... This is an automated message from the

[GitHub] [lucene-solr] jpountz commented on a change in pull request #2268: LUCENE-9705: Move Lucene50CompoundFormat to Lucene90CompoundFormat

2021-02-05 Thread GitBox
jpountz commented on a change in pull request #2268: URL: https://github.com/apache/lucene-solr/pull/2268#discussion_r570105057 ## File path: lucene/backward-codecs/src/test/org/apache/lucene/backward_codecs/lucene50/TestLucene50CompoundFormat.java ## @@ -0,0 +1,30 @@ +/* + *

[GitHub] [lucene-solr] donnerpeter commented on pull request #2283: LUCENE-9719: Resource files aren't deleted from build directory after…

2021-02-05 Thread GitBox
donnerpeter commented on pull request #2283: URL: https://github.com/apache/lucene-solr/pull/2283#issuecomment-773304025 Thanks for the explanation! This is an automated message from the Apache Git Service. To respond to the

[GitHub] [lucene-solr] HoustonPutman commented on a change in pull request #2275: SOLR-15123: Make all Tool option descriptions follow the same general pattern.

2021-02-05 Thread GitBox
HoustonPutman commented on a change in pull request #2275: URL: https://github.com/apache/lucene-solr/pull/2275#discussion_r570372810 ## File path: solr/core/src/java/org/apache/solr/util/PackageTool.java ## @@ -261,44 +261,44 @@ protected void runImpl(CommandLine cli) throws

[GitHub] [lucene-solr] donnerpeter commented on a change in pull request #2300: LUCENE-9729: Hunspell: support CHECKCOMPOUNDREP flags

2021-02-05 Thread GitBox
donnerpeter commented on a change in pull request #2300: URL: https://github.com/apache/lucene-solr/pull/2300#discussion_r570086999 ## File path: lucene/analysis/common/src/java/org/apache/lucene/analysis/hunspell/CheckCompoundPattern.java ## @@ -38,36 +38,33 @@ }

[GitHub] [lucene-solr] epugh commented on pull request #2250: SOLR-13608: Incremental backup file format

2021-02-05 Thread GitBox
epugh commented on pull request #2250: URL: https://github.com/apache/lucene-solr/pull/2250#issuecomment-773319812 I looked for some deprecated tags in the source to convey that the old approach is deprecated, and while there is some text, I don't see any of the `@deprecated` tags. The

[GitHub] [lucene-solr] asalamon74 commented on pull request #2252: SOLR-15111: Use JDK8 Base64 instead of own implementation

2021-02-05 Thread GitBox
asalamon74 commented on pull request #2252: URL: https://github.com/apache/lucene-solr/pull/2252#issuecomment-773249710 We also used a different way for String conversion, I modified the lines. This is an automated message

[GitHub] [lucene-solr] donnerpeter commented on pull request #2296: LUCENE-9728: Hunspell: add a performance test

2021-02-05 Thread GitBox
donnerpeter commented on pull request #2296: URL: https://github.com/apache/lucene-solr/pull/2296#issuecomment-773887234 `Dictionary` changes are intended, because the test parses the corpus based on the affix file `WORDCHARS` directive, which is parsed together with the other directives

[GitHub] [lucene-solr] jpountz commented on a change in pull request #2274: LUCENE-9705: Create Lucene90LiveDocsFormat

2021-02-05 Thread GitBox
jpountz commented on a change in pull request #2274: URL: https://github.com/apache/lucene-solr/pull/2274#discussion_r570062574 ## File path: lucene/backward-codecs/src/java/org/apache/lucene/backward_codecs/lucene50/Lucene50LiveDocsFormat.java ## @@ -0,0 +1,170 @@ +/* + *

[GitHub] [lucene-solr] dweiss commented on a change in pull request #2301: LUCENE-9732: Hunspell: support dictionary entries starting with slash

2021-02-05 Thread GitBox
dweiss commented on a change in pull request #2301: URL: https://github.com/apache/lucene-solr/pull/2301#discussion_r570805734 ## File path: lucene/analysis/common/src/java/org/apache/lucene/analysis/hunspell/WordCase.java ## @@ -53,14 +54,19 @@ static WordCase

[GitHub] [lucene-solr] dweiss commented on pull request #2277: LUCENE-9716: Hunspell: support flag usage before its format is even specified

2021-02-05 Thread GitBox
dweiss commented on pull request #2277: URL: https://github.com/apache/lucene-solr/pull/2277#issuecomment-773880886 I'll commit this patch in and create a separate issue for improving it later. This is an automated

[GitHub] [lucene-solr] jtibshirani merged pull request #2274: LUCENE-9705: Create Lucene90LiveDocsFormat

2021-02-05 Thread GitBox
jtibshirani merged pull request #2274: URL: https://github.com/apache/lucene-solr/pull/2274 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

[GitHub] [lucene-solr] mikemccand commented on pull request #2303: LUCENE-9715: fix int overflow in Lucene90VectorReader

2021-02-05 Thread GitBox
mikemccand commented on pull request #2303: URL: https://github.com/apache/lucene-solr/pull/2303#issuecomment-773533648 > I like Lucene's randomized tests too! But they didn't find this one. Since it requires a 2GB index to detect, it only turned up in the context of Anton's version of

[GitHub] [lucene-solr] noblepaul commented on pull request #2285: SOLR-14928: introduce distributed cluster state updates

2021-02-05 Thread GitBox
noblepaul commented on pull request #2285: URL: https://github.com/apache/lucene-solr/pull/2285#issuecomment-773236842 Is not possible to make this a per collection feature? like the per-replica state feauture? This is an

[GitHub] [lucene-solr] dweiss commented on pull request #2296: LUCENE-9728: Hunspell: add a performance test

2021-02-05 Thread GitBox
dweiss commented on pull request #2296: URL: https://github.com/apache/lucene-solr/pull/2296#issuecomment-773883740 Should this be rebased? Seems to contain unrelated changes not just the test file. A JMH test would probably strip some of the boilerplate but don't know if it's worth

[GitHub] [lucene-solr] jaisonbi commented on pull request #2213: LUCENE-9663: Adding compression to terms dict from SortedSet/Sorted DocValues

2021-02-05 Thread GitBox
jaisonbi commented on pull request #2213: URL: https://github.com/apache/lucene-solr/pull/2213#issuecomment-773179495 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [lucene-solr] jtibshirani commented on a change in pull request #2268: LUCENE-9705: Move Lucene50CompoundFormat to Lucene90CompoundFormat

2021-02-05 Thread GitBox
jtibshirani commented on a change in pull request #2268: URL: https://github.com/apache/lucene-solr/pull/2268#discussion_r570454164 ## File path: lucene/backward-codecs/src/java/org/apache/lucene/backward_codecs/lucene50/Lucene50CompoundFormat.java ## @@ -0,0 +1,88 @@ +/* + *

[GitHub] [lucene-solr-operator] HoustonPutman commented on a change in pull request #209: Basic TLS Support

2021-02-05 Thread GitBox
HoustonPutman commented on a change in pull request #209: URL: https://github.com/apache/lucene-solr-operator/pull/209#discussion_r570530780 ## File path: controllers/solrcloud_controller.go ## @@ -381,7 +436,7 @@ func (r *SolrCloudReconciler) Reconcile(req ctrl.Request)

[GitHub] [lucene-solr] janhoy merged pull request #2279: Split the publish_maven step in two TODOs

2021-02-05 Thread GitBox
janhoy merged pull request #2279: URL: https://github.com/apache/lucene-solr/pull/2279 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

[GitHub] [lucene-solr] ctargett commented on pull request #2304: SOLR-15133: Document how to eliminate Failed to reserve shared memory warning

2021-02-05 Thread GitBox
ctargett commented on pull request #2304: URL: https://github.com/apache/lucene-solr/pull/2304#issuecomment-773450876 Sorry, don't have time to review today I don't think - go ahead if Houston thinks it's good to merge.

[GitHub] [lucene-solr] jtibshirani commented on a change in pull request #2293: LUCENE-9725: Allow BM25FQuery to use other similarities.

2021-02-05 Thread GitBox
jtibshirani commented on a change in pull request #2293: URL: https://github.com/apache/lucene-solr/pull/2293#discussion_r570517621 ## File path: lucene/sandbox/src/java/org/apache/lucene/sandbox/search/CombinedFieldQuery.java ## @@ -64,39 +62,35 @@ * A {@link Query} that

[GitHub] [lucene-solr] mayya-sharipova commented on a change in pull request #2256: LUCENE-9507 Custom order for leaves in IndexReader and IndexWriter

2021-02-05 Thread GitBox
mayya-sharipova commented on a change in pull request #2256: URL: https://github.com/apache/lucene-solr/pull/2256#discussion_r570454676 ## File path: lucene/core/src/java/org/apache/lucene/index/StandardDirectoryReader.java ## @@ -39,33 +40,44 @@ final IndexWriter

[GitHub] [lucene-solr] jaisonbi edited a comment on pull request #2302: LUCENE-9663: Adding compression to terms dict from SortedSet/Sorted D…

2021-02-05 Thread GitBox
jaisonbi edited a comment on pull request #2302: URL: https://github.com/apache/lucene-solr/pull/2302#issuecomment-77375 > It looks to me as if when compression is not enabled, the index format is unchanged, so this should be backwards-compatible. Is that correct? Correct.

[GitHub] [lucene-solr] bruno-roustant commented on pull request #2213: LUCENE-9663: Adding compression to terms dict from SortedSet/Sorted DocValues

2021-02-05 Thread GitBox
bruno-roustant commented on pull request #2213: URL: https://github.com/apache/lucene-solr/pull/2213#issuecomment-773199705 @jaisonbi I didn't want to close this PR. I wanted to merge the first part related to LZ4 compression, and let you push a PR on another jira issue with the remaining

[GitHub] [lucene-solr] noblepaul commented on a change in pull request #2281: SOLR-15118: Convert /v2/collections APIs to annotations

2021-02-05 Thread GitBox
noblepaul commented on a change in pull request #2281: URL: https://github.com/apache/lucene-solr/pull/2281#discussion_r570150592 ## File path: solr/core/src/java/org/apache/solr/handler/CollectionsAPI.java ## @@ -36,7 +58,16 @@ */ public class CollectionsAPI { - private

[GitHub] [lucene-solr] dweiss merged pull request #2296: LUCENE-9728: Hunspell: add a performance test

2021-02-05 Thread GitBox
dweiss merged pull request #2296: URL: https://github.com/apache/lucene-solr/pull/2296 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

[jira] [Created] (LUCENE-9734) Hunspell: support suggestions based on "ph" morphological data

2021-02-05 Thread Peter Gromov (Jira)
Peter Gromov created LUCENE-9734: Summary: Hunspell: support suggestions based on "ph" morphological data Key: LUCENE-9734 URL: https://issues.apache.org/jira/browse/LUCENE-9734 Project: Lucene -

[GitHub] [lucene-solr] epugh commented on a change in pull request #2275: SOLR-15123: Make all Tool option descriptions follow the same general pattern.

2021-02-05 Thread GitBox
epugh commented on a change in pull request #2275: URL: https://github.com/apache/lucene-solr/pull/2275#discussion_r570365526 ## File path: solr/core/src/java/org/apache/solr/util/ExportTool.java ## @@ -216,32 +216,32 @@ void end() throws IOException {

[GitHub] [lucene-solr] jtibshirani commented on pull request #2293: LUCENE-9725: Allow BM25FQuery to use other similarities.

2021-02-05 Thread GitBox
jtibshirani commented on pull request #2293: URL: https://github.com/apache/lucene-solr/pull/2293#issuecomment-773581613 Thanks @jpountz and @jimczi for reviewing. This is an automated message from the Apache Git Service. To

[GitHub] [lucene-solr] sigram commented on a change in pull request #2281: SOLR-15118: Convert /v2/collections APIs to annotations

2021-02-05 Thread GitBox
sigram commented on a change in pull request #2281: URL: https://github.com/apache/lucene-solr/pull/2281#discussion_r570133109 ## File path: solr/core/src/java/org/apache/solr/handler/CollectionsAPI.java ## @@ -50,11 +81,144 @@ public void getCollections(SolrQueryRequest req,

[GitHub] [lucene-solr] jaisonbi commented on pull request #2302: LUCENE-9663: Adding compression to terms dict from SortedSet/Sorted D…

2021-02-05 Thread GitBox
jaisonbi commented on pull request #2302: URL: https://github.com/apache/lucene-solr/pull/2302#issuecomment-773326313 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [lucene-solr] jtibshirani merged pull request #2293: LUCENE-9725: Allow BM25FQuery to use other similarities.

2021-02-05 Thread GitBox
jtibshirani merged pull request #2293: URL: https://github.com/apache/lucene-solr/pull/2293 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

[GitHub] [lucene-solr] epugh merged pull request #2304: SOLR-15133: Document how to eliminate Failed to reserve shared memory warning

2021-02-05 Thread GitBox
epugh merged pull request #2304: URL: https://github.com/apache/lucene-solr/pull/2304 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

[GitHub] [lucene-solr] dweiss merged pull request #2295: LUCENE-9726: Hunspell: speed up spellchecking by stopping at a single…

2021-02-05 Thread GitBox
dweiss merged pull request #2295: URL: https://github.com/apache/lucene-solr/pull/2295 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

[GitHub] [lucene-solr] msokolov commented on a change in pull request #2302: LUCENE-9663: Adding compression to terms dict from SortedSet/Sorted D…

2021-02-05 Thread GitBox
msokolov commented on a change in pull request #2302: URL: https://github.com/apache/lucene-solr/pull/2302#discussion_r570467210 ## File path: lucene/core/src/java/org/apache/lucene/codecs/lucene80/Lucene80DocValuesConsumer.java ## @@ -43,11 +43,7 @@ import

  1   2   >