[GitHub] [lucene-solr] iverase commented on pull request #2155: LUCENE-9641: Support for spatial relationships in LatLonPoint

2020-12-29 Thread GitBox


iverase commented on pull request #2155:
URL: https://github.com/apache/lucene-solr/pull/2155#issuecomment-751994065


   if there are no more comments, I will push this change next week.



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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[jira] [Updated] (LUCENE-9570) Review code diffs after automatic formatting and correct problems before it is applied

2020-12-29 Thread Dawid Weiss (Jira)


 [ 
https://issues.apache.org/jira/browse/LUCENE-9570?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Dawid Weiss updated LUCENE-9570:

Description: 
Review and correct all the javadocs before they're messed up by automatic 
formatting. Apply project-by-project, review diff, correct. Lots of diffs but 
it should be relatively quick.

*Reviewing diffs manually*
 * switch to branch jira/LUCENE-9570 which the PR is based on:
{code:java}
git remote add dweiss g...@github.com:dweiss/lucene-solr.git
git fetch dweiss
git checkout jira/LUCENE-9570
{code}

 * Open gradle/validation/spotless.gradle and locate the project/ package you 
wish to review. Enable it in spotless.gradle by creating a corresponding switch 
case block (refer to existing examples), for example:
{code:java}
  case ":lucene:highlighter":
target "src/**"
targetExclude "**/resources/**", "**/overview.html"
break
{code}

 * Reformat the code:
{code:java}
gradlew tidy && git diff -w > /tmp/diff.patch && git status
{code}

 * Look at what has changed (git status) and review the differences manually 
(/tmp/diff.patch). If everything looks ok, commit it directly to 
jira/LUCENE-9570 or make a PR against that branch.
{code:java}
git commit -am ":lucene:core - src/**/org/apache/lucene/document/**"
{code}

*Packages remaining* (put your name next to a module you're working on to avoid 
duplication).
 * case ":lucene:analysis:nori":
 * case ":lucene:analysis:opennlp":
 * case ":lucene:analysis:phonetic":
 * case ":lucene:analysis:smartcn":
 * case ":lucene:analysis:stempel":
 * case ":lucene:backward-codecs":
 * case ":lucene:benchmark":
 * case ":lucene:classification":
 * case ":lucene:codecs":
 * case ":lucene:expressions":
 * case ":lucene:facet": (Erick Erickson)
 * case ":lucene:grouping":
 * case ":lucene:join":
 * case ":lucene:luke":
 * case ":lucene:memory":
 * case ":lucene:misc":
 * case ":lucene:monitor":
 * case ":lucene:queryparser":
 * case ":lucene:replicator":
 * case ":lucene:sandbox":
 * case ":lucene:spatial3d":
 * case ":lucene:spatial-extras":
 * case ":lucene:suggest":
 * case ":lucene:test-framework":

  was:
Review and correct all the javadocs before they're messed up by automatic 
formatting. Apply project-by-project, review diff, correct. Lots of diffs but 
it should be relatively quick.

*Reviewing diffs manually*
 * switch to branch jira/LUCENE-9570 which the PR is based on:
{code:java}
git remote add dweiss g...@github.com:dweiss/lucene-solr.git
git fetch dweiss
git checkout jira/LUCENE-9570
{code}

 * Open gradle/validation/spotless.gradle and locate the project/ package you 
wish to review. Enable it in spotless.gradle by creating a corresponding switch 
case block (refer to existing examples), for example:
{code:java}
  case ":lucene:highlighter":
target "src/**"
targetExclude "**/resources/**", "**/overview.html"
break
{code}

 * Reformat the code:
{code:java}
gradlew tidy && git diff -w > /tmp/diff.patch && git status
{code}

 * Look at what has changed (git status) and review the differences manually 
(/tmp/diff.patch). If everything looks ok, commit it directly to 
jira/LUCENE-9570 or make a PR against that branch.
{code:java}
git commit -am ":lucene:core - src/**/org/apache/lucene/document/**"
{code}

*Packages remaining* (put your name next to a module you're working on to avoid 
duplication).
 * case ":lucene:analysis:icu":
 * case ":lucene:analysis:kuromoji":
 * case ":lucene:analysis:nori":
 * case ":lucene:analysis:opennlp":
 * case ":lucene:analysis:phonetic":
 * case ":lucene:analysis:smartcn":
 * case ":lucene:analysis:stempel":
 * case ":lucene:backward-codecs":
 * case ":lucene:benchmark":
 * case ":lucene:classification":
 * case ":lucene:codecs":
 * case ":lucene:expressions":
 * case ":lucene:facet": (Erick Erickson)
 * case ":lucene:grouping":
 * case ":lucene:join":
 * case ":lucene:luke":
 * case ":lucene:memory":
 * case ":lucene:misc":
 * case ":lucene:monitor":
 * case ":lucene:queryparser":
 * case ":lucene:replicator":
 * case ":lucene:sandbox":
 * case ":lucene:spatial3d":
 * case ":lucene:spatial-extras":
 * case ":lucene:suggest":
 * case ":lucene:test-framework":


> Review code diffs after automatic formatting and correct problems before it 
> is applied
> --
>
> Key: LUCENE-9570
> URL: https://issues.apache.org/jira/browse/LUCENE-9570
> Project: Lucene - Core
>  Issue Type: Sub-task
>Reporter: Dawid Weiss
>Assignee: Dawid Weiss
>Priority: Blocker
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Review and correct all the javadocs before they're messed up by automatic 
> formatting. Apply project-by-project, review diff, correct. Lots of diffs but 
> it should be rel

[jira] [Updated] (LUCENE-9570) Review code diffs after automatic formatting and correct problems before it is applied

2020-12-29 Thread Dawid Weiss (Jira)


 [ 
https://issues.apache.org/jira/browse/LUCENE-9570?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Dawid Weiss updated LUCENE-9570:

Description: 
Review and correct all the javadocs before they're messed up by automatic 
formatting. Apply project-by-project, review diff, correct. Lots of diffs but 
it should be relatively quick.

*Reviewing diffs manually*
 * switch to branch jira/LUCENE-9570 which the PR is based on:
{code:java}
git remote add dweiss g...@github.com:dweiss/lucene-solr.git
git fetch dweiss
git checkout jira/LUCENE-9570
{code}

 * Open gradle/validation/spotless.gradle and locate the project/ package you 
wish to review. Enable it in spotless.gradle by creating a corresponding switch 
case block (refer to existing examples), for example:
{code:java}
  case ":lucene:highlighter":
target "src/**"
targetExclude "**/resources/**", "**/overview.html"
break
{code}

 * Reformat the code:
{code:java}
gradlew tidy && git diff -w > /tmp/diff.patch && git status
{code}

 * Look at what has changed (git status) and review the differences manually 
(/tmp/diff.patch). If everything looks ok, commit it directly to 
jira/LUCENE-9570 or make a PR against that branch.
{code:java}
git commit -am ":lucene:core - src/**/org/apache/lucene/document/**"
{code}

*Packages remaining* (put your name next to a module you're working on to avoid 
duplication).
 * case ":lucene:analysis:nori":
 * case ":lucene:analysis:opennlp":
 * case ":lucene:analysis:phonetic":
 * case ":lucene:analysis:smartcn":
 * case ":lucene:analysis:stempel":
 * case ":lucene:backward-codecs":
 * case ":lucene:benchmark":
 * case ":lucene:classification":
 * case ":lucene:codecs":
 * case ":lucene:expressions":
 * case ":lucene:facet": (Erick Erickson)
 * case ":lucene:grouping":
 * case ":lucene:join":
 * case ":lucene:luke":
 * case ":lucene:misc":
 * case ":lucene:monitor":
 * case ":lucene:queryparser":
 * case ":lucene:replicator":
 * case ":lucene:sandbox":
 * case ":lucene:spatial3d":
 * case ":lucene:spatial-extras":
 * case ":lucene:suggest":
 * case ":lucene:test-framework":

  was:
Review and correct all the javadocs before they're messed up by automatic 
formatting. Apply project-by-project, review diff, correct. Lots of diffs but 
it should be relatively quick.

*Reviewing diffs manually*
 * switch to branch jira/LUCENE-9570 which the PR is based on:
{code:java}
git remote add dweiss g...@github.com:dweiss/lucene-solr.git
git fetch dweiss
git checkout jira/LUCENE-9570
{code}

 * Open gradle/validation/spotless.gradle and locate the project/ package you 
wish to review. Enable it in spotless.gradle by creating a corresponding switch 
case block (refer to existing examples), for example:
{code:java}
  case ":lucene:highlighter":
target "src/**"
targetExclude "**/resources/**", "**/overview.html"
break
{code}

 * Reformat the code:
{code:java}
gradlew tidy && git diff -w > /tmp/diff.patch && git status
{code}

 * Look at what has changed (git status) and review the differences manually 
(/tmp/diff.patch). If everything looks ok, commit it directly to 
jira/LUCENE-9570 or make a PR against that branch.
{code:java}
git commit -am ":lucene:core - src/**/org/apache/lucene/document/**"
{code}

*Packages remaining* (put your name next to a module you're working on to avoid 
duplication).
 * case ":lucene:analysis:nori":
 * case ":lucene:analysis:opennlp":
 * case ":lucene:analysis:phonetic":
 * case ":lucene:analysis:smartcn":
 * case ":lucene:analysis:stempel":
 * case ":lucene:backward-codecs":
 * case ":lucene:benchmark":
 * case ":lucene:classification":
 * case ":lucene:codecs":
 * case ":lucene:expressions":
 * case ":lucene:facet": (Erick Erickson)
 * case ":lucene:grouping":
 * case ":lucene:join":
 * case ":lucene:luke":
 * case ":lucene:memory":
 * case ":lucene:misc":
 * case ":lucene:monitor":
 * case ":lucene:queryparser":
 * case ":lucene:replicator":
 * case ":lucene:sandbox":
 * case ":lucene:spatial3d":
 * case ":lucene:spatial-extras":
 * case ":lucene:suggest":
 * case ":lucene:test-framework":


> Review code diffs after automatic formatting and correct problems before it 
> is applied
> --
>
> Key: LUCENE-9570
> URL: https://issues.apache.org/jira/browse/LUCENE-9570
> Project: Lucene - Core
>  Issue Type: Sub-task
>Reporter: Dawid Weiss
>Assignee: Dawid Weiss
>Priority: Blocker
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Review and correct all the javadocs before they're messed up by automatic 
> formatting. Apply project-by-project, review diff, correct. Lots of diffs but 
> it should be relatively quick.
> *Reviewing diffs manually*
>  * switch to branch jira/LUCENE-9570 which the PR

[jira] [Created] (LUCENE-9651) Correct javadoc for benchmarks

2020-12-29 Thread Dawid Weiss (Jira)
Dawid Weiss created LUCENE-9651:
---

 Summary: Correct javadoc for benchmarks 
 Key: LUCENE-9651
 URL: https://issues.apache.org/jira/browse/LUCENE-9651
 Project: Lucene - Core
  Issue Type: Task
Reporter: Dawid Weiss
Assignee: Dawid Weiss


Javadoc still mentions ant
{code}
Easiest way to run a benchmarks is using the predefined ant task:
{code}

It'd be good to correct it to use gradle (or direct java -jar... calls).



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[jira] [Updated] (LUCENE-9570) Review code diffs after automatic formatting and correct problems before it is applied

2020-12-29 Thread Dawid Weiss (Jira)


 [ 
https://issues.apache.org/jira/browse/LUCENE-9570?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Dawid Weiss updated LUCENE-9570:

Description: 
Review and correct all the javadocs before they're messed up by automatic 
formatting. Apply project-by-project, review diff, correct. Lots of diffs but 
it should be relatively quick.

*Reviewing diffs manually*
 * switch to branch jira/LUCENE-9570 which the PR is based on:
{code:java}
git remote add dweiss g...@github.com:dweiss/lucene-solr.git
git fetch dweiss
git checkout jira/LUCENE-9570
{code}

 * Open gradle/validation/spotless.gradle and locate the project/ package you 
wish to review. Enable it in spotless.gradle by creating a corresponding switch 
case block (refer to existing examples), for example:
{code:java}
  case ":lucene:highlighter":
target "src/**"
targetExclude "**/resources/**", "**/overview.html"
break
{code}

 * Reformat the code:
{code:java}
gradlew tidy && git diff -w > /tmp/diff.patch && git status
{code}

 * Look at what has changed (git status) and review the differences manually 
(/tmp/diff.patch). If everything looks ok, commit it directly to 
jira/LUCENE-9570 or make a PR against that branch.
{code:java}
git commit -am ":lucene:core - src/**/org/apache/lucene/document/**"
{code}

*Packages remaining* (put your name next to a module you're working on to avoid 
duplication).
 * case ":lucene:analysis:nori":
 * case ":lucene:analysis:opennlp":
 * case ":lucene:analysis:phonetic":
 * case ":lucene:analysis:smartcn":
 * case ":lucene:analysis:stempel":
 * case ":lucene:backward-codecs":
 * case ":lucene:classification":
 * case ":lucene:codecs":
 * case ":lucene:expressions":
 * case ":lucene:facet": (Erick Erickson)
 * case ":lucene:grouping":
 * case ":lucene:join":
 * case ":lucene:luke":
 * case ":lucene:misc":
 * case ":lucene:monitor":
 * case ":lucene:queryparser":
 * case ":lucene:replicator":
 * case ":lucene:sandbox":
 * case ":lucene:spatial3d":
 * case ":lucene:spatial-extras":
 * case ":lucene:suggest":
 * case ":lucene:test-framework":

  was:
Review and correct all the javadocs before they're messed up by automatic 
formatting. Apply project-by-project, review diff, correct. Lots of diffs but 
it should be relatively quick.

*Reviewing diffs manually*
 * switch to branch jira/LUCENE-9570 which the PR is based on:
{code:java}
git remote add dweiss g...@github.com:dweiss/lucene-solr.git
git fetch dweiss
git checkout jira/LUCENE-9570
{code}

 * Open gradle/validation/spotless.gradle and locate the project/ package you 
wish to review. Enable it in spotless.gradle by creating a corresponding switch 
case block (refer to existing examples), for example:
{code:java}
  case ":lucene:highlighter":
target "src/**"
targetExclude "**/resources/**", "**/overview.html"
break
{code}

 * Reformat the code:
{code:java}
gradlew tidy && git diff -w > /tmp/diff.patch && git status
{code}

 * Look at what has changed (git status) and review the differences manually 
(/tmp/diff.patch). If everything looks ok, commit it directly to 
jira/LUCENE-9570 or make a PR against that branch.
{code:java}
git commit -am ":lucene:core - src/**/org/apache/lucene/document/**"
{code}

*Packages remaining* (put your name next to a module you're working on to avoid 
duplication).
 * case ":lucene:analysis:nori":
 * case ":lucene:analysis:opennlp":
 * case ":lucene:analysis:phonetic":
 * case ":lucene:analysis:smartcn":
 * case ":lucene:analysis:stempel":
 * case ":lucene:backward-codecs":
 * case ":lucene:benchmark":
 * case ":lucene:classification":
 * case ":lucene:codecs":
 * case ":lucene:expressions":
 * case ":lucene:facet": (Erick Erickson)
 * case ":lucene:grouping":
 * case ":lucene:join":
 * case ":lucene:luke":
 * case ":lucene:misc":
 * case ":lucene:monitor":
 * case ":lucene:queryparser":
 * case ":lucene:replicator":
 * case ":lucene:sandbox":
 * case ":lucene:spatial3d":
 * case ":lucene:spatial-extras":
 * case ":lucene:suggest":
 * case ":lucene:test-framework":


> Review code diffs after automatic formatting and correct problems before it 
> is applied
> --
>
> Key: LUCENE-9570
> URL: https://issues.apache.org/jira/browse/LUCENE-9570
> Project: Lucene - Core
>  Issue Type: Sub-task
>Reporter: Dawid Weiss
>Assignee: Dawid Weiss
>Priority: Blocker
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Review and correct all the javadocs before they're messed up by automatic 
> formatting. Apply project-by-project, review diff, correct. Lots of diffs but 
> it should be relatively quick.
> *Reviewing diffs manually*
>  * switch to branch jira/LUCENE-9570 which the PR is based on:
> {code:java}
> git remote add dweiss g..

[GitHub] [lucene-solr] trdillon commented on a change in pull request #2169: SOLR-14723: Remove the class attribute for the caches in the _default/example configsets

2020-12-29 Thread GitBox


trdillon commented on a change in pull request #2169:
URL: https://github.com/apache/lucene-solr/pull/2169#discussion_r549674546



##
File path: solr/core/src/test-files/solr/collection1/conf/solrconfig-caching.xml
##
@@ -21,19 +21,16 @@
   
   
 

[GitHub] [lucene-solr] trdillon commented on pull request #2169: SOLR-14723: Remove the class attribute for the caches in the _default/example configsets

2020-12-29 Thread GitBox


trdillon commented on pull request #2169:
URL: https://github.com/apache/lucene-solr/pull/2169#issuecomment-752048769


   > 
   > 
   > For `TestSolr4Spatial2`, I think you just need to add [this 
line](https://github.com/apache/lucene-solr/blob/master/solr/core/src/test/org/apache/solr/search/TestSolr4Spatial2.java#L323)
   
   Should I make changes to this class? I was working off `branch_8x` but seems 
those changes, along with some others, were made in `master` but not 
`branch_8x`.



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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[jira] [Commented] (LUCENE-9650) Errorprone on master/gradle no longer works with JDK-16

2020-12-29 Thread Uwe Schindler (Jira)


[ 
https://issues.apache.org/jira/browse/LUCENE-9650?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17255972#comment-17255972
 ] 

Uwe Schindler commented on LUCENE-9650:
---

That's what I did, see the log dump a posted on the GitHub PR (I forgot to post 
here). Jenkins runs Gradle with Java 11, and just replaces runtime JVM.

> Errorprone on master/gradle no longer works with JDK-16
> ---
>
> Key: LUCENE-9650
> URL: https://issues.apache.org/jira/browse/LUCENE-9650
> Project: Lucene - Core
>  Issue Type: Bug
>  Components: general/build
>Affects Versions: master (9.0)
>Reporter: Uwe Schindler
>Assignee: Dawid Weiss
>Priority: Major
> Fix For: master (9.0)
>
>  Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> JDK-16 no longer allows access to internal classes of any module by default. 
> It looks like errorprone tries to access some internals from the Java 
> compiler. This now fails with Exception. You have to fully open the module or 
> pass `--illegal-access=allow`.
> We have 3 options:
> - install an update of errorprone
> - disable error-prone if we detect a runtimeJdk with version >=16
> - run javac as a separate forked task (i think we do already) and pass 
> `--illegal-access=allow` or open the internal compile module.
> Error message:
> {noformat}
> > Task :solr:solr-ref-guide:compileJava FAILED
> Exception in thread "main" java.lang.IllegalAccessError: class 
> com.google.errorprone.ErrorProneJavacPlugin (in unnamed module @0x887af79) 
> cannot access class com.sun.tools.javac.api.BasicJavacTask (in module 
> jdk.compiler) because module jdk.compiler does not export 
> com.sun.tools.javac.api to unnamed module @0x887af79
>   at 
> com.google.errorprone.ErrorProneJavacPlugin.init(ErrorProneJavacPlugin.java:38)
>   at 
> jdk.compiler/com.sun.tools.javac.api.BasicJavacTask.initPlugin(BasicJavacTask.java:254)
>   at 
> jdk.compiler/com.sun.tools.javac.api.BasicJavacTask.initPlugins(BasicJavacTask.java:228)
>   at jdk.compiler/com.sun.tools.javac.main.Main.compile(Main.java:292)
>   at jdk.compiler/com.sun.tools.javac.main.Main.compile(Main.java:176)
>   at jdk.compiler/com.sun.tools.javac.Main.compile(Main.java:64)
>   at jdk.compiler/com.sun.tools.javac.Main.main(Main.java:50)
> {noformat}
> Last failed build: 
> https://jenkins.thetaphi.de/job/Lucene-Solr-master-Linux/29129/



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[jira] [Commented] (LUCENE-9650) Errorprone on master/gradle no longer works with JDK-16

2020-12-29 Thread Uwe Schindler (Jira)


[ 
https://issues.apache.org/jira/browse/LUCENE-9650?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17255973#comment-17255973
 ] 

Uwe Schindler commented on LUCENE-9650:
---

My testing:

{noformat}
> gradlew compileJava -Druntime.java.home=/path/to/jdk-16
To honour the JVM settings for this build a new JVM will be forked. Please 
consider using the daemon: 
https://docs.gradle.org/6.6.1/userguide/gradle_daemon.html.
Daemon will be stopped at the end of the build stopping after processing

> Configure project :
WARNING: errorprone disabled (won't work with JDK 16)

> Task :altJvmWarning
NOTE: Alternative java toolchain will be used for compilation and tests:
  Project will use Java 16 from: C:\Program Files\java\jdk-16
  Gradle runs with Java 11 from: C:\Program Files\Java\jdk-11.0.8


> Task :solr:solrj:compileJava
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
{noformat}

Without this patch it fails as on Jenkins.

> Errorprone on master/gradle no longer works with JDK-16
> ---
>
> Key: LUCENE-9650
> URL: https://issues.apache.org/jira/browse/LUCENE-9650
> Project: Lucene - Core
>  Issue Type: Bug
>  Components: general/build
>Affects Versions: master (9.0)
>Reporter: Uwe Schindler
>Assignee: Dawid Weiss
>Priority: Major
> Fix For: master (9.0)
>
>  Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> JDK-16 no longer allows access to internal classes of any module by default. 
> It looks like errorprone tries to access some internals from the Java 
> compiler. This now fails with Exception. You have to fully open the module or 
> pass `--illegal-access=allow`.
> We have 3 options:
> - install an update of errorprone
> - disable error-prone if we detect a runtimeJdk with version >=16
> - run javac as a separate forked task (i think we do already) and pass 
> `--illegal-access=allow` or open the internal compile module.
> Error message:
> {noformat}
> > Task :solr:solr-ref-guide:compileJava FAILED
> Exception in thread "main" java.lang.IllegalAccessError: class 
> com.google.errorprone.ErrorProneJavacPlugin (in unnamed module @0x887af79) 
> cannot access class com.sun.tools.javac.api.BasicJavacTask (in module 
> jdk.compiler) because module jdk.compiler does not export 
> com.sun.tools.javac.api to unnamed module @0x887af79
>   at 
> com.google.errorprone.ErrorProneJavacPlugin.init(ErrorProneJavacPlugin.java:38)
>   at 
> jdk.compiler/com.sun.tools.javac.api.BasicJavacTask.initPlugin(BasicJavacTask.java:254)
>   at 
> jdk.compiler/com.sun.tools.javac.api.BasicJavacTask.initPlugins(BasicJavacTask.java:228)
>   at jdk.compiler/com.sun.tools.javac.main.Main.compile(Main.java:292)
>   at jdk.compiler/com.sun.tools.javac.main.Main.compile(Main.java:176)
>   at jdk.compiler/com.sun.tools.javac.Main.compile(Main.java:64)
>   at jdk.compiler/com.sun.tools.javac.Main.main(Main.java:50)
> {noformat}
> Last failed build: 
> https://jenkins.thetaphi.de/job/Lucene-Solr-master-Linux/29129/



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[jira] [Commented] (SOLR-14608) Faster sorting for the /export handler

2020-12-29 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/SOLR-14608?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17256020#comment-17256020
 ] 

ASF subversion and git services commented on SOLR-14608:


Commit cae61336f86295014a1c373774bc56c5b9f21670 in lucene-solr's branch 
refs/heads/jira/SOLR-14608-export from Joel Bernstein
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=cae6133 ]

SOLR-14608: Fix nanoTime to millis calculation and more code cleanup


> Faster sorting for the /export handler
> --
>
> Key: SOLR-14608
> URL: https://issues.apache.org/jira/browse/SOLR-14608
> Project: Solr
>  Issue Type: New Feature
>Reporter: Joel Bernstein
>Assignee: Joel Bernstein
>Priority: Major
>
> The largest cost of the export handler is the sorting. This ticket will 
> implement an improved algorithm for sorting that should greatly increase 
> overall throughput for the export handler.
> *The current algorithm is as follows:*
> Collect a bitset of matching docs. Iterate over that bitset and materialize 
> the top level oridinals for the sort fields in the document and add them to 
> priority queue of size 3. Then export the top 3 docs, turn off the 
> bits in the bit set and iterate again until all docs are sorted and sent. 
> There are two performance bottlenecks with this approach:
> 1) Materializing the top level ordinals adds a huge amount of overhead to the 
> sorting process.
> 2) The size of priority queue, 30,000, adds significant overhead to sorting 
> operations.
> *The new algorithm:*
> Has a top level *merge sort iterator* that wraps segment level iterators that 
> perform segment level priority queue sorts.
> *Segment level:*
> The segment level docset will be iterated and the segment level ordinals for 
> the sort fields will be materialized and added to a segment level priority 
> queue. As the segment level iterator pops docs from the priority queue the 
> top level ordinals for the sort fields are materialized. Because the top 
> level ordinals are materialized AFTER the sort, they only need to be looked 
> up when the segment level ordinal changes. This takes advantage of the sort 
> to limit the lookups into the top level ordinal structures. This also 
> eliminates redundant lookups of top level ordinals that occur during the 
> multiple passes over the matching docset.
> The segment level priority queues can be kept smaller than 30,000 to improve 
> performance of the sorting operations because the overall batch size will 
> still be 30,000 or greater when all the segment priority queue sizes are 
> added up. This allows for batch sizes much larger then 30,000 without using a 
> single large priority queue. The increased batch size means fewer iterations 
> over the matching docset and the decreased priority queue size means faster 
> sorting operations.
> *Top level:*
> A top level iterator does a merge sort over the segment level iterators by 
> comparing the top level ordinals materialized when the segment level docs are 
> popped from the segment level priority queues. This requires no extra memory 
> and will be very performant.
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[jira] [Assigned] (SOLR-15058) Solr can not get correct hostname when hostname contain '_'

2020-12-29 Thread Timothy Potter (Jira)


 [ 
https://issues.apache.org/jira/browse/SOLR-15058?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Timothy Potter reassigned SOLR-15058:
-

Assignee: Timothy Potter

> Solr can not get correct hostname when hostname contain '_'
> ---
>
> Key: SOLR-15058
> URL: https://issues.apache.org/jira/browse/SOLR-15058
> Project: Solr
>  Issue Type: Bug
>  Security Level: Public(Default Security Level. Issues are Public) 
>Reporter: Su Sasa
>Assignee: Timothy Potter
>Priority: Minor
> Attachments: 
> 0001-SOLR-15058-get-the-correct-hostname-when-hostname-co.patch
>
>  Time Spent: 40m
>  Remaining Estimate: 0h
>
> Utils.getBaseUrlForNodeName split the nodename by the first '_'
> In ZK, Solr add '_solr' to the end of the hostname
> so there should split by the last '_'



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[jira] [Commented] (SOLR-15058) Solr can not get correct hostname when hostname contain '_'

2020-12-29 Thread Timothy Potter (Jira)


[ 
https://issues.apache.org/jira/browse/SOLR-15058?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17256071#comment-17256071
 ] 

Timothy Potter commented on SOLR-15058:
---

Let's get this fixed for 8.8 ... I'll work to get it committed as I've been in 
this code recently as well.

> Solr can not get correct hostname when hostname contain '_'
> ---
>
> Key: SOLR-15058
> URL: https://issues.apache.org/jira/browse/SOLR-15058
> Project: Solr
>  Issue Type: Bug
>  Security Level: Public(Default Security Level. Issues are Public) 
>Reporter: Su Sasa
>Assignee: Timothy Potter
>Priority: Minor
> Attachments: 
> 0001-SOLR-15058-get-the-correct-hostname-when-hostname-co.patch
>
>  Time Spent: 40m
>  Remaining Estimate: 0h
>
> Utils.getBaseUrlForNodeName split the nodename by the first '_'
> In ZK, Solr add '_solr' to the end of the hostname
> so there should split by the last '_'



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[jira] [Commented] (SOLR-14608) Faster sorting for the /export handler

2020-12-29 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/SOLR-14608?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17256081#comment-17256081
 ] 

ASF subversion and git services commented on SOLR-14608:


Commit d7e81e8197a4b9e06c8d387d34e941a1365ad163 in lucene-solr's branch 
refs/heads/jira/SOLR-14608-export from Joel Bernstein
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=d7e81e8 ]

SOLR-14608: Tone down debug logging


> Faster sorting for the /export handler
> --
>
> Key: SOLR-14608
> URL: https://issues.apache.org/jira/browse/SOLR-14608
> Project: Solr
>  Issue Type: New Feature
>Reporter: Joel Bernstein
>Assignee: Joel Bernstein
>Priority: Major
>
> The largest cost of the export handler is the sorting. This ticket will 
> implement an improved algorithm for sorting that should greatly increase 
> overall throughput for the export handler.
> *The current algorithm is as follows:*
> Collect a bitset of matching docs. Iterate over that bitset and materialize 
> the top level oridinals for the sort fields in the document and add them to 
> priority queue of size 3. Then export the top 3 docs, turn off the 
> bits in the bit set and iterate again until all docs are sorted and sent. 
> There are two performance bottlenecks with this approach:
> 1) Materializing the top level ordinals adds a huge amount of overhead to the 
> sorting process.
> 2) The size of priority queue, 30,000, adds significant overhead to sorting 
> operations.
> *The new algorithm:*
> Has a top level *merge sort iterator* that wraps segment level iterators that 
> perform segment level priority queue sorts.
> *Segment level:*
> The segment level docset will be iterated and the segment level ordinals for 
> the sort fields will be materialized and added to a segment level priority 
> queue. As the segment level iterator pops docs from the priority queue the 
> top level ordinals for the sort fields are materialized. Because the top 
> level ordinals are materialized AFTER the sort, they only need to be looked 
> up when the segment level ordinal changes. This takes advantage of the sort 
> to limit the lookups into the top level ordinal structures. This also 
> eliminates redundant lookups of top level ordinals that occur during the 
> multiple passes over the matching docset.
> The segment level priority queues can be kept smaller than 30,000 to improve 
> performance of the sorting operations because the overall batch size will 
> still be 30,000 or greater when all the segment priority queue sizes are 
> added up. This allows for batch sizes much larger then 30,000 without using a 
> single large priority queue. The increased batch size means fewer iterations 
> over the matching docset and the decreased priority queue size means faster 
> sorting operations.
> *Top level:*
> A top level iterator does a merge sort over the segment level iterators by 
> comparing the top level ordinals materialized when the segment level docs are 
> popped from the segment level priority queues. This requires no extra memory 
> and will be very performant.
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[GitHub] [lucene-solr] thelabdude commented on a change in pull request #2164: SOLR-15058: get the correct hostname when hostname contain '_'

2020-12-29 Thread GitBox


thelabdude commented on a change in pull request #2164:
URL: https://github.com/apache/lucene-solr/pull/2164#discussion_r549791687



##
File path: solr/solrj/src/test/org/apache/solr/common/cloud/UrlSchemeTest.java
##
@@ -35,12 +35,12 @@ public void testApplyUrlScheme() {
 String liveNode1 = "192.168.1.1:8983_solr";
 String liveNode2 = "127.0.0.1:8983_solr";
 String liveNode3 = "127.0.0.1_";
-String liveNode4 = "127.0.0.1:61631_l_%2Fig";
+String liveNode4 = "127.0.0.1:61631_lx%2Fig";

Review comment:
   I think a good solution is to find the first underscore after the colon. 
If you look at `generateNodeName` in ZkController then you'll see the colon is 
always added to the generated `nodeName`. This may require some tests to update 
the test node name as there are some tests that use an improper node name that 
doesn't adhere to how `ZkController#generateNodeName` creates actual node names 
at runtime, this seems like a bad test impl. to me and should be corrected.





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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[GitHub] [lucene-solr] thelabdude commented on a change in pull request #2164: SOLR-15058: get the correct hostname when hostname contain '_'

2020-12-29 Thread GitBox


thelabdude commented on a change in pull request #2164:
URL: https://github.com/apache/lucene-solr/pull/2164#discussion_r549791973



##
File path: solr/solrj/src/test/org/apache/solr/common/cloud/UrlSchemeTest.java
##
@@ -35,12 +35,12 @@ public void testApplyUrlScheme() {
 String liveNode1 = "192.168.1.1:8983_solr";
 String liveNode2 = "127.0.0.1:8983_solr";
 String liveNode3 = "127.0.0.1_";
-String liveNode4 = "127.0.0.1:61631_l_%2Fig";
+String liveNode4 = "127.0.0.1:61631_lx%2Fig";

Review comment:
   fwiw ~ you can't look at the last underscore because nodeName encoding 
changes `/` to `_` in the context as you can't create znodes with `/` in them 
for obvious reasons ;-)





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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[GitHub] [lucene-solr] nknize commented on pull request #2155: LUCENE-9641: Support for spatial relationships in LatLonPoint

2020-12-29 Thread GitBox


nknize commented on pull request #2155:
URL: https://github.com/apache/lucene-solr/pull/2155#issuecomment-752197102


   With the holidays and folks out on vacation would you mind holding off until 
the end of next week? I plan to have a look at the other two changes by the end 
of this week and I think it would be fair to make sure @rmuir has enough time 
to have a look as well.



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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[GitHub] [lucene-solr] thelabdude commented on a change in pull request #2164: SOLR-15058: get the correct hostname when hostname contain '_'

2020-12-29 Thread GitBox


thelabdude commented on a change in pull request #2164:
URL: https://github.com/apache/lucene-solr/pull/2164#discussion_r549807523



##
File path: solr/solrj/src/test/org/apache/solr/common/cloud/UrlSchemeTest.java
##
@@ -35,12 +35,12 @@ public void testApplyUrlScheme() {
 String liveNode1 = "192.168.1.1:8983_solr";
 String liveNode2 = "127.0.0.1:8983_solr";
 String liveNode3 = "127.0.0.1_";
-String liveNode4 = "127.0.0.1:61631_l_%2Fig";
+String liveNode4 = "127.0.0.1:61631_lx%2Fig";

Review comment:
   Adding the following check to Utils:
   ```
   diff --git a/solr/solrj/src/java/org/apache/solr/common/util/Utils.java 
b/solr/solrj/src/java/org/apache/solr/common/util/Utils.java
   index 86bef9177c8..13e732d4ed7 100644
   --- a/solr/solrj/src/java/org/apache/solr/common/util/Utils.java
   +++ b/solr/solrj/src/java/org/apache/solr/common/util/Utils.java
   @@ -759,7 +759,12 @@ public class Utils {
return getBaseUrlForNodeName(nodeName, urlScheme, false);
  }
  public static String getBaseUrlForNodeName(final String nodeName, String 
urlScheme,  boolean isV2) {
   -final int _offset = nodeName.indexOf("_");
   +final int colonAt = nodeName.indexOf(':');
   +if (colonAt == -1) {
   +  throw new IllegalArgumentException("nodeName does not contain 
expected ':' separator: " + nodeName);
   +}
   +
   +final int _offset = nodeName.indexOf("_", colonAt);
if (_offset < 0) {
  throw new IllegalArgumentException("nodeName does not contain 
expected '_' separator: " + nodeName);
}
   ```
   
   Results in the following test failures:
   ```
   1669 tests completed, 3 failed, 20 skipped
   
   4619 tests completed, 26 failed, 210 skipped
   
   ERROR: The following test(s) have failed:
 - org.apache.solr.common.cloud.UrlSchemeTest.testApplyUrlScheme 
(:solr:solrj)
   Test output: 
/Users/tjp/dev/oss/lucene-solr/master/solr/solrj/build/test-results/test/outputs/OUTPUT-org.apache.solr.common.cloud.UrlSchemeTest.txt
   Reproduce with: gradlew :solr:solrj:test --tests 
"org.apache.solr.common.cloud.UrlSchemeTest.testApplyUrlScheme" -Ptests.jvms=12 
-Ptests.jvmargs=-XX:TieredStopAtLevel=1 -Ptests.seed=14FE50F5891679FC 
-Ptests.file.encoding=US-ASCII
   
 - 
org.apache.solr.client.solrj.routing.ShufflingReplicaListTransformerTest.testTransformReplicas
 (:solr:solrj)
   Test output: 
/Users/tjp/dev/oss/lucene-solr/master/solr/solrj/build/test-results/test/outputs/OUTPUT-org.apache.solr.client.solrj.routing.ShufflingReplicaListTransformerTest.txt
   Reproduce with: gradlew :solr:solrj:test --tests 
"org.apache.solr.client.solrj.routing.ShufflingReplicaListTransformerTest.testTransformReplicas"
 -Ptests.jvms=12 -Ptests.jvmargs=-XX:TieredStopAtLevel=1 
-Ptests.seed=14FE50F5891679FC -Ptests.file.encoding=US-ASCII
   
 - 
org.apache.solr.client.solrj.routing.ReplicaListTransformerTest.testTransform 
(:solr:solrj)
   Test output: 
/Users/tjp/dev/oss/lucene-solr/master/solr/solrj/build/test-results/test/outputs/OUTPUT-org.apache.solr.client.solrj.routing.ReplicaListTransformerTest.txt
   Reproduce with: gradlew :solr:solrj:test --tests 
"org.apache.solr.client.solrj.routing.ReplicaListTransformerTest.testTransform" 
-Ptests.jvms=12 -Ptests.jvmargs=-XX:TieredStopAtLevel=1 
-Ptests.seed=14FE50F5891679FC -Ptests.file.encoding=US-ASCII
   
 - 
org.apache.solr.handler.component.CloudReplicaSourceTest.testShardsParam_DownReplica
 (:solr:core)
   Test output: 
/Users/tjp/dev/oss/lucene-solr/master/solr/core/build/test-results/test/outputs/OUTPUT-org.apache.solr.handler.component.CloudReplicaSourceTest.txt
   Reproduce with: gradlew :solr:core:test --tests 
"org.apache.solr.handler.component.CloudReplicaSourceTest.testShardsParam_DownReplica"
 -Ptests.jvms=12 -Ptests.jvmargs=-XX:TieredStopAtLevel=1 
-Ptests.seed=14FE50F5891679FC -Ptests.file.encoding=US-ASCII
   
 - 
org.apache.solr.handler.component.CloudReplicaSourceTest.testSimple_OnlyNrt 
(:solr:core)
   Test output: 
/Users/tjp/dev/oss/lucene-solr/master/solr/core/build/test-results/test/outputs/OUTPUT-org.apache.solr.handler.component.CloudReplicaSourceTest.txt
   Reproduce with: gradlew :solr:core:test --tests 
"org.apache.solr.handler.component.CloudReplicaSourceTest.testSimple_OnlyNrt" 
-Ptests.jvms=12 -Ptests.jvmargs=-XX:TieredStopAtLevel=1 
-Ptests.seed=14FE50F5891679FC -Ptests.file.encoding=US-ASCII
   
 - 
org.apache.solr.handler.component.CloudReplicaSourceTest.testShardsParam_DeadNode
 (:solr:core)
   Test output: 
/Users/tjp/dev/oss/lucene-solr/master/solr/core/build/test-results/test/outputs/OUTPUT-org.apache.solr.handler.component.CloudReplicaSourceTest.txt
   Reproduce with: gradlew :solr:core:test --tests 
"org.apache.solr.handler.component.CloudReplicaSourceTest.testShardsParam_DeadNode"
 -Ptests.jvms=12 -Ptests.jvmargs=-XX:TieredStopAtLevel=1 
-Ptests.seed=14FE50F5891679FC -Ptests.file.enco

[jira] [Resolved] (LUCENE-9622) provide gradle option to disable error-prone checker

2020-12-29 Thread Robert Muir (Jira)


 [ 
https://issues.apache.org/jira/browse/LUCENE-9622?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Robert Muir resolved LUCENE-9622.
-
Resolution: Duplicate

Fixed by LUCENE-9650

> provide gradle option to disable error-prone checker
> 
>
> Key: LUCENE-9622
> URL: https://issues.apache.org/jira/browse/LUCENE-9622
> Project: Lucene - Core
>  Issue Type: Improvement
>Reporter: Robert Muir
>Priority: Major
>
> Trying to just run tests with the latest jdk16-ea, I can't do it because of 
> bugs in "error-prone":
> {noformat}
> > Task :lucene:core:compileJava
> /home/rmuir/workspace/lucene-solr/lucene/core/src/java/org/apache/lucene/index/IndexWriter.java:3486:
>  error: An unhandled exception was thrown by the Error Prone static analysis 
> plugin.
> MergePolicy.MergeSpecification pointInTimeMerges = 
> updatePendingMerges(new OneMergeWrappingMergePolicy(config.getMergePolicy(), 
> toWrap ->
>   ^
>  Please report this at https://github.com/google/error-prone/issues/new 
> and include the following:
>   
>  error-prone version: 2.4.0
>  BugPattern: ParameterName
>  Stack Trace:
>  java.lang.NoSuchFieldError: reader
> at 
> com.google.errorprone.util.ErrorProneTokens$CommentSavingTokenizer.processComment(ErrorProneTokens.java:85)
> at 
> jdk.compiler/com.sun.tools.javac.parser.JavaTokenizer.readToken(JavaTokenizer.java:919)
> at 
> jdk.compiler/com.sun.tools.javac.parser.Scanner.nextToken(Scanner.java:115)
> at 
> com.google.errorprone.util.ErrorProneTokens.getTokens(ErrorProneTokens.java:57)
> at 
> com.google.errorprone.util.ErrorProneTokens.getTokens(ErrorProneTokens.java:74)
> at 
> com.google.errorprone.bugpatterns.ParameterName.checkArguments(ParameterName.java:97)
> at 
> com.google.errorprone.bugpatterns.ParameterName.matchMethodInvocation(ParameterName.java:66)
> at 
> com.google.errorprone.scanner.ErrorProneScanner.processMatchers(ErrorProneScanner.java:451)
> {noformat}
> I really just want to run the tests, so for now I just commented it out 
> locally. Let's provide an option as it seems it doesn't necessarily keep up 
> with the JDK? Not sure what is going on with this thing.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[GitHub] [lucene-solr] tflobbe commented on pull request #2169: SOLR-14723: Remove the class attribute for the caches in the _default/example configsets

2020-12-29 Thread GitBox


tflobbe commented on pull request #2169:
URL: https://github.com/apache/lucene-solr/pull/2169#issuecomment-752336456


   > Should I make changes to this class?
   
   Looks to me that this change will be needed given the changes in this PR, 
right? this test is failing for me in your branch



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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[GitHub] [lucene-solr] tflobbe commented on a change in pull request #2152: SOLR-14034: remove deprecated min_rf references

2020-12-29 Thread GitBox


tflobbe commented on a change in pull request #2152:
URL: https://github.com/apache/lucene-solr/pull/2152#discussion_r549948394



##
File path: solr/core/src/test/org/apache/solr/cloud/HttpPartitionTest.java
##
@@ -548,9 +548,6 @@ protected int sendDoc(int docId, Integer minRf, SolrClient 
solrClient, String co
 doc.addField("a_t", "hello" + docId);
 
 UpdateRequest up = new UpdateRequest();
-if (minRf != null) {
-  up.setParam(UpdateRequest.MIN_REPFACT, String.valueOf(minRf));
-}

Review comment:
   > It appears that the minRf argument in sendDocsWithRetry is indeed 
unused already but it might be clearer to tidy that up in a separate follow-up 
pull request rather than mix it in here at this time.
   
   I'm wondering if it would be better for this method to actually start using 
this parameter, and retry in the case of `minRf` not achieved? This is not 
something that was being done in this method, but may help improve some of the 
tests using it. I don't think such a change should be done in this PR, but 
maybe lets not remove the parameter yet then. WDYT?





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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[GitHub] [lucene-solr] iverase commented on pull request #2155: LUCENE-9641: Support for spatial relationships in LatLonPoint

2020-12-29 Thread GitBox


iverase commented on pull request #2155:
URL: https://github.com/apache/lucene-solr/pull/2155#issuecomment-752362243


   Sure, just hope it can make it to the next release.



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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[jira] [Commented] (SOLR-14723) Rremove the class atribute for the caches in the _default/example configsets

2020-12-29 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/SOLR-14723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17256360#comment-17256360
 ] 

ASF subversion and git services commented on SOLR-14723:


Commit 196632d538a26b0c92a67941ad7c47e2f9cd2e1d in lucene-solr's branch 
refs/heads/branch_8x from Tomas Eduardo Fernandez Lobbe
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=196632d ]

SOLR-14723: Remove the class attribute for the caches in the _default/example 
configsets

This is an 8.x change only. Closes #2169

Squashed commit of the following:

commit c638cbcfdb58b57cde8c4be641a7847b8a37422a
Merge: b13011e97ae a22abe8ae0d
Author: Tomas Fernandez Lobbe 
Date:   Tue Dec 29 23:33:38 2020 -0800

Merge branch 'jira/SOLR-14723' of https://github.com/trdillon/lucene-solr 
into trdillon-jira/SOLR-14723

commit a22abe8ae0ddd5934dd8f74f4a48599e188ffc73
Author: Tim Dillon 
Date:   Wed Dec 30 15:34:34 2020 +0900

Add SOLR-14723 to CHANGES.txt

commit 655bba5db63118c3055c30d331d1d082e0dd6050
Author: Tim Dillon 
Date:   Wed Dec 30 15:02:16 2020 +0900

Fix TestSolr4Spatial2 test

commit 2d8544fc455de8b596c6fbf3ecd2feb2ca6a84fc
Author: Tim Dillon 
Date:   Tue Dec 29 20:40:03 2020 +0900

Update doc to state default cache class

commit 3c5ee62d8fe87da080595822b46a3390510dea41
Author: Tim Dillon 
Date:   Tue Dec 29 19:51:57 2020 +0900

Replaced the class attribute in solrconfig-caching

commit 2341eef4a63696068b010d8da51f2fa818790fc0
Author: Tim Dillon 
Date:   Sun Dec 27 15:52:07 2020 +0900

Remove REQUIRE_CLASS from SolrCache plugin

commit 627995aba1a29dc4deaf726c0a49599224ea2de5
Author: Tim Dillon 
Date:   Sat Dec 26 11:14:03 2020 +0900

Remove cast to FastLRUCache in TestJsonRequest

commit 11319c119570c861d62f09fc6b482a6993dd38ff
Author: Tim Dillon 
Date:   Sat Dec 19 16:53:07 2020 +0900

Removed class attribute from caches


> Rremove the class atribute for the caches in the _default/example configsets
> 
>
> Key: SOLR-14723
> URL: https://issues.apache.org/jira/browse/SOLR-14723
> Project: Solr
>  Issue Type: Improvement
>Reporter: Tomas Eduardo Fernandez Lobbe
>Priority: Trivial
>  Labels: newdev
>  Time Spent: 1h
>  Remaining Estimate: 0h
>
> In 8.x branch, the default and example configsets include LRUCache and 
> FastLRUCache as the implementation for the various caches.
> {code:xml}
>   size="512"
>  initialSize="512"
>  autowarmCount="0"/>
> {code}
> Those implementations are deprecated. We should remove the attribute 
> completely and let the default take over (as it's in master), this should 
> make upgrading configsets easier:



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[jira] [Commented] (SOLR-14723) Rremove the class atribute for the caches in the _default/example configsets

2020-12-29 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/SOLR-14723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17256361#comment-17256361
 ] 

ASF subversion and git services commented on SOLR-14723:


Commit 196632d538a26b0c92a67941ad7c47e2f9cd2e1d in lucene-solr's branch 
refs/heads/branch_8x from Tomas Eduardo Fernandez Lobbe
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=196632d ]

SOLR-14723: Remove the class attribute for the caches in the _default/example 
configsets

This is an 8.x change only. Closes #2169

Squashed commit of the following:

commit c638cbcfdb58b57cde8c4be641a7847b8a37422a
Merge: b13011e97ae a22abe8ae0d
Author: Tomas Fernandez Lobbe 
Date:   Tue Dec 29 23:33:38 2020 -0800

Merge branch 'jira/SOLR-14723' of https://github.com/trdillon/lucene-solr 
into trdillon-jira/SOLR-14723

commit a22abe8ae0ddd5934dd8f74f4a48599e188ffc73
Author: Tim Dillon 
Date:   Wed Dec 30 15:34:34 2020 +0900

Add SOLR-14723 to CHANGES.txt

commit 655bba5db63118c3055c30d331d1d082e0dd6050
Author: Tim Dillon 
Date:   Wed Dec 30 15:02:16 2020 +0900

Fix TestSolr4Spatial2 test

commit 2d8544fc455de8b596c6fbf3ecd2feb2ca6a84fc
Author: Tim Dillon 
Date:   Tue Dec 29 20:40:03 2020 +0900

Update doc to state default cache class

commit 3c5ee62d8fe87da080595822b46a3390510dea41
Author: Tim Dillon 
Date:   Tue Dec 29 19:51:57 2020 +0900

Replaced the class attribute in solrconfig-caching

commit 2341eef4a63696068b010d8da51f2fa818790fc0
Author: Tim Dillon 
Date:   Sun Dec 27 15:52:07 2020 +0900

Remove REQUIRE_CLASS from SolrCache plugin

commit 627995aba1a29dc4deaf726c0a49599224ea2de5
Author: Tim Dillon 
Date:   Sat Dec 26 11:14:03 2020 +0900

Remove cast to FastLRUCache in TestJsonRequest

commit 11319c119570c861d62f09fc6b482a6993dd38ff
Author: Tim Dillon 
Date:   Sat Dec 19 16:53:07 2020 +0900

Removed class attribute from caches


> Rremove the class atribute for the caches in the _default/example configsets
> 
>
> Key: SOLR-14723
> URL: https://issues.apache.org/jira/browse/SOLR-14723
> Project: Solr
>  Issue Type: Improvement
>Reporter: Tomas Eduardo Fernandez Lobbe
>Priority: Trivial
>  Labels: newdev
>  Time Spent: 1h
>  Remaining Estimate: 0h
>
> In 8.x branch, the default and example configsets include LRUCache and 
> FastLRUCache as the implementation for the various caches.
> {code:xml}
>   size="512"
>  initialSize="512"
>  autowarmCount="0"/>
> {code}
> Those implementations are deprecated. We should remove the attribute 
> completely and let the default take over (as it's in master), this should 
> make upgrading configsets easier:



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[jira] [Commented] (SOLR-14723) Rremove the class atribute for the caches in the _default/example configsets

2020-12-29 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/SOLR-14723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17256358#comment-17256358
 ] 

ASF subversion and git services commented on SOLR-14723:


Commit 196632d538a26b0c92a67941ad7c47e2f9cd2e1d in lucene-solr's branch 
refs/heads/branch_8x from Tomas Eduardo Fernandez Lobbe
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=196632d ]

SOLR-14723: Remove the class attribute for the caches in the _default/example 
configsets

This is an 8.x change only. Closes #2169

Squashed commit of the following:

commit c638cbcfdb58b57cde8c4be641a7847b8a37422a
Merge: b13011e97ae a22abe8ae0d
Author: Tomas Fernandez Lobbe 
Date:   Tue Dec 29 23:33:38 2020 -0800

Merge branch 'jira/SOLR-14723' of https://github.com/trdillon/lucene-solr 
into trdillon-jira/SOLR-14723

commit a22abe8ae0ddd5934dd8f74f4a48599e188ffc73
Author: Tim Dillon 
Date:   Wed Dec 30 15:34:34 2020 +0900

Add SOLR-14723 to CHANGES.txt

commit 655bba5db63118c3055c30d331d1d082e0dd6050
Author: Tim Dillon 
Date:   Wed Dec 30 15:02:16 2020 +0900

Fix TestSolr4Spatial2 test

commit 2d8544fc455de8b596c6fbf3ecd2feb2ca6a84fc
Author: Tim Dillon 
Date:   Tue Dec 29 20:40:03 2020 +0900

Update doc to state default cache class

commit 3c5ee62d8fe87da080595822b46a3390510dea41
Author: Tim Dillon 
Date:   Tue Dec 29 19:51:57 2020 +0900

Replaced the class attribute in solrconfig-caching

commit 2341eef4a63696068b010d8da51f2fa818790fc0
Author: Tim Dillon 
Date:   Sun Dec 27 15:52:07 2020 +0900

Remove REQUIRE_CLASS from SolrCache plugin

commit 627995aba1a29dc4deaf726c0a49599224ea2de5
Author: Tim Dillon 
Date:   Sat Dec 26 11:14:03 2020 +0900

Remove cast to FastLRUCache in TestJsonRequest

commit 11319c119570c861d62f09fc6b482a6993dd38ff
Author: Tim Dillon 
Date:   Sat Dec 19 16:53:07 2020 +0900

Removed class attribute from caches


> Rremove the class atribute for the caches in the _default/example configsets
> 
>
> Key: SOLR-14723
> URL: https://issues.apache.org/jira/browse/SOLR-14723
> Project: Solr
>  Issue Type: Improvement
>Reporter: Tomas Eduardo Fernandez Lobbe
>Priority: Trivial
>  Labels: newdev
>  Time Spent: 1h
>  Remaining Estimate: 0h
>
> In 8.x branch, the default and example configsets include LRUCache and 
> FastLRUCache as the implementation for the various caches.
> {code:xml}
>   size="512"
>  initialSize="512"
>  autowarmCount="0"/>
> {code}
> Those implementations are deprecated. We should remove the attribute 
> completely and let the default take over (as it's in master), this should 
> make upgrading configsets easier:



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[jira] [Commented] (SOLR-14723) Rremove the class atribute for the caches in the _default/example configsets

2020-12-29 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/SOLR-14723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17256359#comment-17256359
 ] 

ASF subversion and git services commented on SOLR-14723:


Commit 196632d538a26b0c92a67941ad7c47e2f9cd2e1d in lucene-solr's branch 
refs/heads/branch_8x from Tomas Eduardo Fernandez Lobbe
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=196632d ]

SOLR-14723: Remove the class attribute for the caches in the _default/example 
configsets

This is an 8.x change only. Closes #2169

Squashed commit of the following:

commit c638cbcfdb58b57cde8c4be641a7847b8a37422a
Merge: b13011e97ae a22abe8ae0d
Author: Tomas Fernandez Lobbe 
Date:   Tue Dec 29 23:33:38 2020 -0800

Merge branch 'jira/SOLR-14723' of https://github.com/trdillon/lucene-solr 
into trdillon-jira/SOLR-14723

commit a22abe8ae0ddd5934dd8f74f4a48599e188ffc73
Author: Tim Dillon 
Date:   Wed Dec 30 15:34:34 2020 +0900

Add SOLR-14723 to CHANGES.txt

commit 655bba5db63118c3055c30d331d1d082e0dd6050
Author: Tim Dillon 
Date:   Wed Dec 30 15:02:16 2020 +0900

Fix TestSolr4Spatial2 test

commit 2d8544fc455de8b596c6fbf3ecd2feb2ca6a84fc
Author: Tim Dillon 
Date:   Tue Dec 29 20:40:03 2020 +0900

Update doc to state default cache class

commit 3c5ee62d8fe87da080595822b46a3390510dea41
Author: Tim Dillon 
Date:   Tue Dec 29 19:51:57 2020 +0900

Replaced the class attribute in solrconfig-caching

commit 2341eef4a63696068b010d8da51f2fa818790fc0
Author: Tim Dillon 
Date:   Sun Dec 27 15:52:07 2020 +0900

Remove REQUIRE_CLASS from SolrCache plugin

commit 627995aba1a29dc4deaf726c0a49599224ea2de5
Author: Tim Dillon 
Date:   Sat Dec 26 11:14:03 2020 +0900

Remove cast to FastLRUCache in TestJsonRequest

commit 11319c119570c861d62f09fc6b482a6993dd38ff
Author: Tim Dillon 
Date:   Sat Dec 19 16:53:07 2020 +0900

Removed class attribute from caches


> Rremove the class atribute for the caches in the _default/example configsets
> 
>
> Key: SOLR-14723
> URL: https://issues.apache.org/jira/browse/SOLR-14723
> Project: Solr
>  Issue Type: Improvement
>Reporter: Tomas Eduardo Fernandez Lobbe
>Priority: Trivial
>  Labels: newdev
>  Time Spent: 1h
>  Remaining Estimate: 0h
>
> In 8.x branch, the default and example configsets include LRUCache and 
> FastLRUCache as the implementation for the various caches.
> {code:xml}
>   size="512"
>  initialSize="512"
>  autowarmCount="0"/>
> {code}
> Those implementations are deprecated. We should remove the attribute 
> completely and let the default take over (as it's in master), this should 
> make upgrading configsets easier:



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[GitHub] [lucene-solr] tflobbe commented on pull request #2169: SOLR-14723: Remove the class attribute for the caches in the _default/example configsets

2020-12-29 Thread GitBox


tflobbe commented on pull request #2169:
URL: https://github.com/apache/lucene-solr/pull/2169#issuecomment-752364628


   Merged to 8.x. Not needed in master



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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[GitHub] [lucene-solr] tflobbe closed pull request #2169: SOLR-14723: Remove the class attribute for the caches in the _default/example configsets

2020-12-29 Thread GitBox


tflobbe closed pull request #2169:
URL: https://github.com/apache/lucene-solr/pull/2169


   



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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[jira] [Issue Comment Deleted] (SOLR-14723) Rremove the class atribute for the caches in the _default/example configsets

2020-12-29 Thread Tomas Eduardo Fernandez Lobbe (Jira)


 [ 
https://issues.apache.org/jira/browse/SOLR-14723?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tomas Eduardo Fernandez Lobbe updated SOLR-14723:
-
Comment: was deleted

(was: Commit 196632d538a26b0c92a67941ad7c47e2f9cd2e1d in lucene-solr's branch 
refs/heads/branch_8x from Tomas Eduardo Fernandez Lobbe
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=196632d ]

SOLR-14723: Remove the class attribute for the caches in the _default/example 
configsets

This is an 8.x change only. Closes #2169

Squashed commit of the following:

commit c638cbcfdb58b57cde8c4be641a7847b8a37422a
Merge: b13011e97ae a22abe8ae0d
Author: Tomas Fernandez Lobbe 
Date:   Tue Dec 29 23:33:38 2020 -0800

Merge branch 'jira/SOLR-14723' of https://github.com/trdillon/lucene-solr 
into trdillon-jira/SOLR-14723

commit a22abe8ae0ddd5934dd8f74f4a48599e188ffc73
Author: Tim Dillon 
Date:   Wed Dec 30 15:34:34 2020 +0900

Add SOLR-14723 to CHANGES.txt

commit 655bba5db63118c3055c30d331d1d082e0dd6050
Author: Tim Dillon 
Date:   Wed Dec 30 15:02:16 2020 +0900

Fix TestSolr4Spatial2 test

commit 2d8544fc455de8b596c6fbf3ecd2feb2ca6a84fc
Author: Tim Dillon 
Date:   Tue Dec 29 20:40:03 2020 +0900

Update doc to state default cache class

commit 3c5ee62d8fe87da080595822b46a3390510dea41
Author: Tim Dillon 
Date:   Tue Dec 29 19:51:57 2020 +0900

Replaced the class attribute in solrconfig-caching

commit 2341eef4a63696068b010d8da51f2fa818790fc0
Author: Tim Dillon 
Date:   Sun Dec 27 15:52:07 2020 +0900

Remove REQUIRE_CLASS from SolrCache plugin

commit 627995aba1a29dc4deaf726c0a49599224ea2de5
Author: Tim Dillon 
Date:   Sat Dec 26 11:14:03 2020 +0900

Remove cast to FastLRUCache in TestJsonRequest

commit 11319c119570c861d62f09fc6b482a6993dd38ff
Author: Tim Dillon 
Date:   Sat Dec 19 16:53:07 2020 +0900

Removed class attribute from caches
)

> Rremove the class atribute for the caches in the _default/example configsets
> 
>
> Key: SOLR-14723
> URL: https://issues.apache.org/jira/browse/SOLR-14723
> Project: Solr
>  Issue Type: Improvement
>Reporter: Tomas Eduardo Fernandez Lobbe
>Priority: Trivial
>  Labels: newdev
>  Time Spent: 1h 20m
>  Remaining Estimate: 0h
>
> In 8.x branch, the default and example configsets include LRUCache and 
> FastLRUCache as the implementation for the various caches.
> {code:xml}
>   size="512"
>  initialSize="512"
>  autowarmCount="0"/>
> {code}
> Those implementations are deprecated. We should remove the attribute 
> completely and let the default take over (as it's in master), this should 
> make upgrading configsets easier:



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[jira] [Issue Comment Deleted] (SOLR-14723) Rremove the class atribute for the caches in the _default/example configsets

2020-12-29 Thread Tomas Eduardo Fernandez Lobbe (Jira)


 [ 
https://issues.apache.org/jira/browse/SOLR-14723?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tomas Eduardo Fernandez Lobbe updated SOLR-14723:
-
Comment: was deleted

(was: Commit 196632d538a26b0c92a67941ad7c47e2f9cd2e1d in lucene-solr's branch 
refs/heads/branch_8x from Tomas Eduardo Fernandez Lobbe
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=196632d ]

SOLR-14723: Remove the class attribute for the caches in the _default/example 
configsets

This is an 8.x change only. Closes #2169

Squashed commit of the following:

commit c638cbcfdb58b57cde8c4be641a7847b8a37422a
Merge: b13011e97ae a22abe8ae0d
Author: Tomas Fernandez Lobbe 
Date:   Tue Dec 29 23:33:38 2020 -0800

Merge branch 'jira/SOLR-14723' of https://github.com/trdillon/lucene-solr 
into trdillon-jira/SOLR-14723

commit a22abe8ae0ddd5934dd8f74f4a48599e188ffc73
Author: Tim Dillon 
Date:   Wed Dec 30 15:34:34 2020 +0900

Add SOLR-14723 to CHANGES.txt

commit 655bba5db63118c3055c30d331d1d082e0dd6050
Author: Tim Dillon 
Date:   Wed Dec 30 15:02:16 2020 +0900

Fix TestSolr4Spatial2 test

commit 2d8544fc455de8b596c6fbf3ecd2feb2ca6a84fc
Author: Tim Dillon 
Date:   Tue Dec 29 20:40:03 2020 +0900

Update doc to state default cache class

commit 3c5ee62d8fe87da080595822b46a3390510dea41
Author: Tim Dillon 
Date:   Tue Dec 29 19:51:57 2020 +0900

Replaced the class attribute in solrconfig-caching

commit 2341eef4a63696068b010d8da51f2fa818790fc0
Author: Tim Dillon 
Date:   Sun Dec 27 15:52:07 2020 +0900

Remove REQUIRE_CLASS from SolrCache plugin

commit 627995aba1a29dc4deaf726c0a49599224ea2de5
Author: Tim Dillon 
Date:   Sat Dec 26 11:14:03 2020 +0900

Remove cast to FastLRUCache in TestJsonRequest

commit 11319c119570c861d62f09fc6b482a6993dd38ff
Author: Tim Dillon 
Date:   Sat Dec 19 16:53:07 2020 +0900

Removed class attribute from caches
)

> Rremove the class atribute for the caches in the _default/example configsets
> 
>
> Key: SOLR-14723
> URL: https://issues.apache.org/jira/browse/SOLR-14723
> Project: Solr
>  Issue Type: Improvement
>Reporter: Tomas Eduardo Fernandez Lobbe
>Priority: Trivial
>  Labels: newdev
>  Time Spent: 1h 20m
>  Remaining Estimate: 0h
>
> In 8.x branch, the default and example configsets include LRUCache and 
> FastLRUCache as the implementation for the various caches.
> {code:xml}
>   size="512"
>  initialSize="512"
>  autowarmCount="0"/>
> {code}
> Those implementations are deprecated. We should remove the attribute 
> completely and let the default take over (as it's in master), this should 
> make upgrading configsets easier:



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[jira] [Issue Comment Deleted] (SOLR-14723) Rremove the class atribute for the caches in the _default/example configsets

2020-12-29 Thread Tomas Eduardo Fernandez Lobbe (Jira)


 [ 
https://issues.apache.org/jira/browse/SOLR-14723?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tomas Eduardo Fernandez Lobbe updated SOLR-14723:
-
Comment: was deleted

(was: Commit 196632d538a26b0c92a67941ad7c47e2f9cd2e1d in lucene-solr's branch 
refs/heads/branch_8x from Tomas Eduardo Fernandez Lobbe
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=196632d ]

SOLR-14723: Remove the class attribute for the caches in the _default/example 
configsets

This is an 8.x change only. Closes #2169

Squashed commit of the following:

commit c638cbcfdb58b57cde8c4be641a7847b8a37422a
Merge: b13011e97ae a22abe8ae0d
Author: Tomas Fernandez Lobbe 
Date:   Tue Dec 29 23:33:38 2020 -0800

Merge branch 'jira/SOLR-14723' of https://github.com/trdillon/lucene-solr 
into trdillon-jira/SOLR-14723

commit a22abe8ae0ddd5934dd8f74f4a48599e188ffc73
Author: Tim Dillon 
Date:   Wed Dec 30 15:34:34 2020 +0900

Add SOLR-14723 to CHANGES.txt

commit 655bba5db63118c3055c30d331d1d082e0dd6050
Author: Tim Dillon 
Date:   Wed Dec 30 15:02:16 2020 +0900

Fix TestSolr4Spatial2 test

commit 2d8544fc455de8b596c6fbf3ecd2feb2ca6a84fc
Author: Tim Dillon 
Date:   Tue Dec 29 20:40:03 2020 +0900

Update doc to state default cache class

commit 3c5ee62d8fe87da080595822b46a3390510dea41
Author: Tim Dillon 
Date:   Tue Dec 29 19:51:57 2020 +0900

Replaced the class attribute in solrconfig-caching

commit 2341eef4a63696068b010d8da51f2fa818790fc0
Author: Tim Dillon 
Date:   Sun Dec 27 15:52:07 2020 +0900

Remove REQUIRE_CLASS from SolrCache plugin

commit 627995aba1a29dc4deaf726c0a49599224ea2de5
Author: Tim Dillon 
Date:   Sat Dec 26 11:14:03 2020 +0900

Remove cast to FastLRUCache in TestJsonRequest

commit 11319c119570c861d62f09fc6b482a6993dd38ff
Author: Tim Dillon 
Date:   Sat Dec 19 16:53:07 2020 +0900

Removed class attribute from caches
)

> Rremove the class atribute for the caches in the _default/example configsets
> 
>
> Key: SOLR-14723
> URL: https://issues.apache.org/jira/browse/SOLR-14723
> Project: Solr
>  Issue Type: Improvement
>Reporter: Tomas Eduardo Fernandez Lobbe
>Priority: Trivial
>  Labels: newdev
>  Time Spent: 1h 20m
>  Remaining Estimate: 0h
>
> In 8.x branch, the default and example configsets include LRUCache and 
> FastLRUCache as the implementation for the various caches.
> {code:xml}
>   size="512"
>  initialSize="512"
>  autowarmCount="0"/>
> {code}
> Those implementations are deprecated. We should remove the attribute 
> completely and let the default take over (as it's in master), this should 
> make upgrading configsets easier:



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[jira] [Commented] (SOLR-14723) Rremove the class atribute for the caches in the _default/example configsets

2020-12-29 Thread Tomas Eduardo Fernandez Lobbe (Jira)


[ 
https://issues.apache.org/jira/browse/SOLR-14723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17256367#comment-17256367
 ] 

Tomas Eduardo Fernandez Lobbe commented on SOLR-14723:
--

Thanks [~trdillon], merged the PR. And thanks [~shshnk28] for the initial 
contribution.

> Rremove the class atribute for the caches in the _default/example configsets
> 
>
> Key: SOLR-14723
> URL: https://issues.apache.org/jira/browse/SOLR-14723
> Project: Solr
>  Issue Type: Improvement
>Reporter: Tomas Eduardo Fernandez Lobbe
>Priority: Trivial
>  Labels: newdev
>  Time Spent: 1h 20m
>  Remaining Estimate: 0h
>
> In 8.x branch, the default and example configsets include LRUCache and 
> FastLRUCache as the implementation for the various caches.
> {code:xml}
>   size="512"
>  initialSize="512"
>  autowarmCount="0"/>
> {code}
> Those implementations are deprecated. We should remove the attribute 
> completely and let the default take over (as it's in master), this should 
> make upgrading configsets easier:



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org



[jira] [Resolved] (SOLR-14723) Rremove the class atribute for the caches in the _default/example configsets

2020-12-29 Thread Tomas Eduardo Fernandez Lobbe (Jira)


 [ 
https://issues.apache.org/jira/browse/SOLR-14723?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tomas Eduardo Fernandez Lobbe resolved SOLR-14723.
--
Fix Version/s: 8.8
   Resolution: Fixed

> Rremove the class atribute for the caches in the _default/example configsets
> 
>
> Key: SOLR-14723
> URL: https://issues.apache.org/jira/browse/SOLR-14723
> Project: Solr
>  Issue Type: Improvement
>Reporter: Tomas Eduardo Fernandez Lobbe
>Priority: Trivial
>  Labels: newdev
> Fix For: 8.8
>
>  Time Spent: 1h 20m
>  Remaining Estimate: 0h
>
> In 8.x branch, the default and example configsets include LRUCache and 
> FastLRUCache as the implementation for the various caches.
> {code:xml}
>   size="512"
>  initialSize="512"
>  autowarmCount="0"/>
> {code}
> Those implementations are deprecated. We should remove the attribute 
> completely and let the default take over (as it's in master), this should 
> make upgrading configsets easier:



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org