[GitHub] [lucene-solr] noblepaul edited a comment on pull request #1684: SOLR-14613: strongly typed initial proposal for placement plugin interface

2020-08-30 Thread GitBox
noblepaul edited a comment on pull request #1684: URL: https://github.com/apache/lucene-solr/pull/1684#issuecomment-683488170 >There's no other place to set default config (not hard code!) for which placement plugin to use What is wrong with hard coding? >t's easy to have a

[GitHub] [lucene-solr] rmuir commented on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
rmuir commented on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683533223 >> That check was added to make merging easier. See the related issues about it. I don't care. That is really wrong to do. If master is on java 11, then master is on

[jira] [Commented] (LUCENE-9490) gradlew checkMissingDocsDefault fails on JDK 15 or later

2020-08-30 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9490?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187352#comment-17187352 ] ASF subversion and git services commented on LUCENE-9490: - Commit

[jira] [Commented] (LUCENE-9456) Stored fields should store the number of chunks in the meta file

2020-08-30 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9456?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187351#comment-17187351 ] ASF subversion and git services commented on LUCENE-9456: - Commit

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187353#comment-17187353 ] ASF subversion and git services commented on LUCENE-9215: - Commit

[GitHub] [lucene-solr] noblepaul edited a comment on pull request #1684: SOLR-14613: strongly typed initial proposal for placement plugin interface

2020-08-30 Thread GitBox
noblepaul edited a comment on pull request #1684: URL: https://github.com/apache/lucene-solr/pull/1684#issuecomment-683488170 >There's no other place to set default config (not hard code!) for which placement plugin to use What is wrong with hard coding? >t's easy to have a

[jira] [Resolved] (LUCENE-9490) gradlew checkMissingDocsDefault fails on JDK 15 or later

2020-08-30 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9490?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Uwe Schindler resolved LUCENE-9490. --- Resolution: Fixed Should be OK now. > gradlew checkMissingDocsDefault fails on JDK 15 or

[jira] [Commented] (LUCENE-9490) gradlew checkMissingDocsDefault fails on JDK 15 or later

2020-08-30 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9490?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187348#comment-17187348 ] ASF subversion and git services commented on LUCENE-9490: - Commit

[GitHub] [lucene-solr] noblepaul edited a comment on pull request #1684: SOLR-14613: strongly typed initial proposal for placement plugin interface

2020-08-30 Thread GitBox
noblepaul edited a comment on pull request #1684: URL: https://github.com/apache/lucene-solr/pull/1684#issuecomment-683488170 >There's no other place to set default config (not hard code!) for which placement plugin to use What is wrong with hard coding? >t's easy to have a

[GitHub] [lucene-solr] noblepaul commented on pull request #1684: SOLR-14613: strongly typed initial proposal for placement plugin interface

2020-08-30 Thread GitBox
noblepaul commented on pull request #1684: URL: https://github.com/apache/lucene-solr/pull/1684#issuecomment-683488170 >There's no other place to set default config (not hard code!) for which placement plugin to use What is wrong with hard coding?

[jira] [Commented] (SOLR-6152) Pre-populating values into search parameters on the query page of solr admin

2020-08-30 Thread Jakob Furrer (Jira)
[ https://issues.apache.org/jira/browse/SOLR-6152?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187344#comment-17187344 ] Jakob Furrer commented on SOLR-6152: Can someone please change/update the affected versions of this

[jira] [Commented] (LUCENE-9490) gradlew checkMissingDocsDefault fails on JDK 15 or later

2020-08-30 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9490?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187345#comment-17187345 ] Uwe Schindler commented on LUCENE-9490: --- I know, it's the other task: {noformat} > Task

[jira] [Reopened] (LUCENE-9490) gradlew checkMissingDocsDefault fails on JDK 15 or later

2020-08-30 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9490?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Uwe Schindler reopened LUCENE-9490: --- For some strange reason, the task still fails on Policeman Jenkins:

[jira] [Commented] (LUCENE-9475) Enhance the Gradle build as necessary after removing Ant support

2020-08-30 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9475?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187342#comment-17187342 ] Uwe Schindler commented on LUCENE-9475: --- I created a pull request to change the

[jira] [Comment Edited] (LUCENE-9475) Enhance the Gradle build as necessary after removing Ant support

2020-08-30 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9475?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187342#comment-17187342 ] Uwe Schindler edited comment on LUCENE-9475 at 8/30/20, 11:45 PM: -- I

[GitHub] [lucene-solr] uschindler opened a new pull request #1806: LUCENE-9475: Switch validateSourcePatters away from Ant legacy

2020-08-30 Thread GitBox
uschindler opened a new pull request #1806: URL: https://github.com/apache/lucene-solr/pull/1806 This PR changes the legacy setp of the checkSourcePatterns task to directly execute the checker code in the running Gradle VM. Because of Ant compatibility we preserved the old code

[GitHub] [lucene-solr] uschindler commented on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
uschindler commented on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683483373 As this class is not part of any code in branch_8x, I think validateSourcePatterns should exclude it for this check. Can be quickyl implemented. I am dealing with

[GitHub] [lucene-solr] uschindler edited a comment on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
uschindler edited a comment on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683481056 > seems the build fails because of var use? that check is broken: master is java 11 That check was added to make merging easier. See the related issues

[GitHub] [lucene-solr] uschindler commented on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
uschindler commented on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683481056 > seems the build fails because of var use? that check is broken: master is java 11 That check was added to make merging easier. See the related issues about it.

[GitHub] [lucene-solr] rmuir commented on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
rmuir commented on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683477788 seems the build fails because of `var` use? that check is broken: master is java 11 This is an automated

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

2020-08-30 Thread Mark Robert Miller (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187336#comment-17187336 ] Mark Robert Miller commented on SOLR-14788: --- Test are back to a pretty great state after the

[GitHub] [lucene-solr] uschindler commented on a change in pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
uschindler commented on a change in pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#discussion_r479814878 ## File path: gradle/validation/forbidden-apis.gradle ## @@ -103,6 +103,13 @@ allprojects { prj -> ] } +// Doclet does use

[GitHub] [lucene-solr] dweiss commented on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
dweiss commented on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683464627 I did move it to a composite subproject. So it is now separate from Solr and Lucene, yet still in the source tree. We can move it back to buildSrc or anything else once it's

[GitHub] [lucene-solr] dweiss commented on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
dweiss commented on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683463331 A yet another alternative is to use a composite build to compile it locally. Then it is a separate project (top-level project) on its own and is not part of the Lucene/ Solr

[GitHub] [lucene-solr] dweiss commented on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
dweiss commented on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683462911 Making it a true project discovered two bugs the moment I tried to compile it. So there are benefits. It also makes development faster, at least in the early stage (because

[GitHub] [lucene-solr] uschindler commented on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
uschindler commented on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683454888 > If that's the case, I'd copypaste Robert's code into the render-javadocs.gradle file as another inner class (as the syntax of Java is mostly the same with Gradle).

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187311#comment-17187311 ] Robert Muir commented on LUCENE-9215: - if we can check javadocs without relying on override i think

[GitHub] [lucene-solr] uschindler edited a comment on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
uschindler edited a comment on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683452425 Hi, I still disagree. We have many classes in the build that are Groovy-based (like the javadocs task). What's the difference to that one, just because it's

[GitHub] [lucene-solr] uschindler commented on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
uschindler commented on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683452425 Hi, I still disagree. We have many classes in the build that are Groovy, based (like the javadocs task). Wht's the difference to that on, just because it's written in

[GitHub] [lucene-solr] uschindler edited a comment on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
uschindler edited a comment on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683452425 Hi, I still disagree. We have many classes in the build that are Groovy-based (like the javadocs task). Wht's the difference to that on, just because it's

[GitHub] [lucene-solr] dweiss commented on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
dweiss commented on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683447537 It's a bit of a chicken and egg problem (boostrapping gradle itself). I do think a separate project for this is beneficial here - at least it's clear what it is, has a

[GitHub] [lucene-solr] uschindler commented on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
uschindler commented on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683446509 > a318310 Made missing doclet a regular project. This also means all the checks are applied to it (forbidden APIs, etc.). I was so happy that `lucene/tools` was

[jira] [Comment Edited] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187293#comment-17187293 ] Uwe Schindler edited comment on LUCENE-9215 at 8/30/20, 5:16 PM: - The

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187293#comment-17187293 ] Uwe Schindler commented on LUCENE-9215: --- The good thing with this error is: It informs us about

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187292#comment-17187292 ] Uwe Schindler commented on LUCENE-9215: --- Hi, bq. The overriden method issue can be solved without

[jira] [Commented] (SOLR-14540) Ref Guide: Document how-to for Block Join Facets via Query DSL

2020-08-30 Thread Mikhail Khludnev (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187273#comment-17187273 ] Mikhail Khludnev commented on SOLR-14540: - https://github.com/apache/lucene-solr/pull/1805 edits

[GitHub] [lucene-solr] mkhludnev opened a new pull request #1805: SOLR-14540: example of drill sideway block join facets

2020-08-30 Thread GitBox
mkhludnev opened a new pull request #1805: URL: https://github.com/apache/lucene-solr/pull/1805 Contributing example of block join drill-seideway facets to Ref Guide This is an automated message from the Apache Git Service.

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187272#comment-17187272 ] Dawid Weiss commented on LUCENE-9215: - The overriden method issue can be solved without checking for

[jira] [Updated] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dawid Weiss updated LUCENE-9215: Attachment: overrides.patch > replace checkJavaDocs.py with doclet >

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187271#comment-17187271 ] Dawid Weiss commented on LUCENE-9215: - Done. The move has an upside in that all regular checks are

[jira] [Commented] (SOLR-14794) Pass global cloud configuration to Collection API command execution

2020-08-30 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14794?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187266#comment-17187266 ] ASF subversion and git services commented on SOLR-14794: Commit

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187270#comment-17187270 ] ASF subversion and git services commented on LUCENE-9215: - Commit

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187269#comment-17187269 ] ASF subversion and git services commented on LUCENE-9215: - Commit

[jira] [Commented] (LUCENE-9435) Clean up ant compatability remnants in Gradle build

2020-08-30 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9435?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187267#comment-17187267 ] ASF subversion and git services commented on LUCENE-9435: - Commit

[jira] [Commented] (LUCENE-9474) Change Jenkins jobs to use Gradle for trunk

2020-08-30 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9474?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187268#comment-17187268 ] ASF subversion and git services commented on LUCENE-9474: - Commit

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187262#comment-17187262 ] Dawid Weiss commented on LUCENE-9215: - bq. i already hit the solr issue, i just solved it

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187261#comment-17187261 ] Robert Muir commented on LUCENE-9215: - OK [~dweiss] I'm all done now, all yours. Thanks for the help

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187260#comment-17187260 ] ASF subversion and git services commented on LUCENE-9215: - Commit

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187258#comment-17187258 ] ASF subversion and git services commented on LUCENE-9215: - Commit

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187256#comment-17187256 ] Robert Muir commented on LUCENE-9215: - I'm merging one more time (i already hit the solr issue, i

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187253#comment-17187253 ] Dawid Weiss commented on LUCENE-9215: - One more thing - the doclet currently sits in buildSrc so any

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187252#comment-17187252 ] Dawid Weiss commented on LUCENE-9215: - I did overlook the setting in Solr's top-build file.

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187249#comment-17187249 ] Dawid Weiss commented on LUCENE-9215: - Ah, darn. Sorry. It should be trivial to merge - I literally

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187247#comment-17187247 ] Robert Muir commented on LUCENE-9215: - No worries, I will merge my changes with your work real quick

[GitHub] [lucene-solr] murblanc edited a comment on pull request #1684: SOLR-14613: strongly typed initial proposal for placement plugin interface

2020-08-30 Thread GitBox
murblanc edited a comment on pull request #1684: URL: https://github.com/apache/lucene-solr/pull/1684#issuecomment-683433584 > Here is an example of how to have strong typing for each atribute without creating so many different interfaces It's easy to have a single `PropertyValue`

[GitHub] [lucene-solr] murblanc edited a comment on pull request #1684: SOLR-14613: strongly typed initial proposal for placement plugin interface

2020-08-30 Thread GitBox
murblanc edited a comment on pull request #1684: URL: https://github.com/apache/lucene-solr/pull/1684#issuecomment-683434613 > So, I have two main problems with this PR I consider this progress  > 1. Using `solr.xml` There's no other place to set default config (not

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187242#comment-17187242 ] Dawid Weiss commented on LUCENE-9215: - [~rcmuir] moved stuff around a bit, hope you're having a

[GitHub] [lucene-solr] murblanc edited a comment on pull request #1684: SOLR-14613: strongly typed initial proposal for placement plugin interface

2020-08-30 Thread GitBox
murblanc edited a comment on pull request #1684: URL: https://github.com/apache/lucene-solr/pull/1684#issuecomment-683433584 > Here is an example of how to have strong typing for each atribute without creating so many different interfaces It's easy to have a single `PropertyValue`

[GitHub] [lucene-solr] murblanc edited a comment on pull request #1684: SOLR-14613: strongly typed initial proposal for placement plugin interface

2020-08-30 Thread GitBox
murblanc edited a comment on pull request #1684: URL: https://github.com/apache/lucene-solr/pull/1684#issuecomment-683433584 > Here is an example of how to have strong typing for each atribute without creating so many different interfaces It's easy to have a single `PropertyValue`

[GitHub] [lucene-solr] murblanc edited a comment on pull request #1684: SOLR-14613: strongly typed initial proposal for placement plugin interface

2020-08-30 Thread GitBox
murblanc edited a comment on pull request #1684: URL: https://github.com/apache/lucene-solr/pull/1684#issuecomment-683433584 > Here is an example of how to have strong typing for each atribute without creating so many different interfaces It's easy to have a single `PropertyValue`

[jira] [Commented] (LUCENE-9456) Stored fields should store the number of chunks in the meta file

2020-08-30 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9456?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187240#comment-17187240 ] ASF subversion and git services commented on LUCENE-9456: - Commit

[jira] [Commented] (LUCENE-9456) Stored fields should store the number of chunks in the meta file

2020-08-30 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9456?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187239#comment-17187239 ] ASF subversion and git services commented on LUCENE-9456: - Commit

[GitHub] [lucene-solr] dsmiley merged pull request #1803: LUCENE-9456: fix DirectUpdateHandlerTest#testPrepareCommit

2020-08-30 Thread GitBox
dsmiley merged pull request #1803: URL: https://github.com/apache/lucene-solr/pull/1803 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] murblanc commented on pull request #1684: SOLR-14613: strongly typed initial proposal for placement plugin interface

2020-08-30 Thread GitBox
murblanc commented on pull request #1684: URL: https://github.com/apache/lucene-solr/pull/1684#issuecomment-683434613 > So, I have two main problems with this PR I consider this progress  > 1. Using `solr.xml` There's no other place to set default config (not hard

[GitHub] [lucene-solr] murblanc commented on pull request #1684: SOLR-14613: strongly typed initial proposal for placement plugin interface

2020-08-30 Thread GitBox
murblanc commented on pull request #1684: URL: https://github.com/apache/lucene-solr/pull/1684#issuecomment-683433584 > Here is an example of how to have strong typing for each atribute without creating so many different interfaces It's easy to have a single `PropertyValue`

[jira] [Commented] (LUCENE-9474) Change Jenkins jobs to use Gradle for trunk

2020-08-30 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9474?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187236#comment-17187236 ] ASF subversion and git services commented on LUCENE-9474: - Commit

[jira] [Commented] (SOLR-13652) Remove update from initParams in example solrconfig files that only mention "df"

2020-08-30 Thread Alexandre Rafalovitch (Jira)
[ https://issues.apache.org/jira/browse/SOLR-13652?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187235#comment-17187235 ] Alexandre Rafalovitch commented on SOLR-13652: -- I think this is a great discussion. And

[jira] [Created] (SOLR-14798) Config API: indexConfig output includes all defaults

2020-08-30 Thread Alexandre Rafalovitch (Jira)
Alexandre Rafalovitch created SOLR-14798: Summary: Config API: indexConfig output includes all defaults Key: SOLR-14798 URL: https://issues.apache.org/jira/browse/SOLR-14798 Project: Solr

[jira] [Updated] (SOLR-14783) Remove DIH from 9.0

2020-08-30 Thread Alexandre Rafalovitch (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14783?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alexandre Rafalovitch updated SOLR-14783: - Description: Now that Data Import Handler (SOLR-14066) has been depreciated in

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187216#comment-17187216 ] ASF subversion and git services commented on LUCENE-9215: - Commit

[GitHub] [lucene-solr] noblepaul opened a new pull request #1804: SOLR-14151: Bbug fixes

2020-08-30 Thread GitBox
noblepaul opened a new pull request #1804: URL: https://github.com/apache/lucene-solr/pull/1804 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[jira] [Commented] (LUCENE-9215) replace checkJavaDocs.py with doclet

2020-08-30 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187214#comment-17187214 ] ASF subversion and git services commented on LUCENE-9215: - Commit

[GitHub] [lucene-solr] noblepaul edited a comment on pull request #1684: SOLR-14613: strongly typed initial proposal for placement plugin interface

2020-08-30 Thread GitBox
noblepaul edited a comment on pull request #1684: URL: https://github.com/apache/lucene-solr/pull/1684#issuecomment-683420028 Here is an example of how to have strong typing for each atribute without creating so many different interfaces `public interface PropertyFetcher {`

[GitHub] [lucene-solr] noblepaul commented on pull request #1684: SOLR-14613: strongly typed initial proposal for placement plugin interface

2020-08-30 Thread GitBox
noblepaul commented on pull request #1684: URL: https://github.com/apache/lucene-solr/pull/1684#issuecomment-683420148 So, I have two main problems with this PR 1. Using `solr.xml` 2. having 2 interfaces for each property

[GitHub] [lucene-solr] noblepaul commented on pull request #1684: SOLR-14613: strongly typed initial proposal for placement plugin interface

2020-08-30 Thread GitBox
noblepaul commented on pull request #1684: URL: https://github.com/apache/lucene-solr/pull/1684#issuecomment-683420028 Here is an example of how to have strong typing for each atribute without creating so many different interfaces `public interface PropertyFetcher {`

[GitHub] [lucene-solr] rmuir commented on a change in pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
rmuir commented on a change in pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#discussion_r479766765 ## File path: buildSrc/src/main/java/org/apache/lucene/gradle/MissingDoclet.java ## @@ -0,0 +1,331 @@ +/* + * Licensed to the Apache Software

[GitHub] [lucene-solr] rmuir commented on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
rmuir commented on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683413964 Thanks @dweiss @uschindler @mocobeta please, make any changes if you have the time and motiviation. I agree with the suggestions: plumbing the correct inputs sounds great to

[jira] [Commented] (LUCENE-9490) gradlew checkMissingDocsDefault fails on JDK 15 or later

2020-08-30 Thread Tomoko Uchida (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9490?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187195#comment-17187195 ] Tomoko Uchida commented on LUCENE-9490: --- bq. Robert made such great progress with doclet parser

[jira] [Updated] (LUCENE-9475) Enhance the Gradle build as necessary after removing Ant support

2020-08-30 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9475?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dawid Weiss updated LUCENE-9475: Description: Once the bulk of the Ant build system is removed, stuff will come bubbling up out

[jira] [Commented] (LUCENE-9435) Clean up ant compatability remnants in Gradle build

2020-08-30 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9435?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187194#comment-17187194 ] ASF subversion and git services commented on LUCENE-9435: - Commit

[jira] [Resolved] (LUCENE-9435) Clean up ant compatability remnants in Gradle build

2020-08-30 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9435?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dawid Weiss resolved LUCENE-9435. - Fix Version/s: master (9.0) Resolution: Fixed > Clean up ant compatability remnants in

[GitHub] [lucene-solr] dweiss merged pull request #1799: Lucene 9435

2020-08-30 Thread GitBox
dweiss merged pull request #1799: URL: https://github.com/apache/lucene-solr/pull/1799 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] uschindler commented on a change in pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
uschindler commented on a change in pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#discussion_r479761009 ## File path: buildSrc/src/main/java/org/apache/lucene/gradle/MissingDoclet.java ## @@ -0,0 +1,331 @@ +/* + * Licensed to the Apache Software

[jira] [Commented] (LUCENE-9490) gradlew checkMissingDocsDefault fails on JDK 15 or later

2020-08-30 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9490?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187192#comment-17187192 ] Dawid Weiss commented on LUCENE-9490: - bq. Yes I think the simplified regex is correct, but it seems

[GitHub] [lucene-solr] uschindler edited a comment on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
uschindler edited a comment on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683408276 I agree with Dawid. At the beginning I did not like his approach to centralize everything, but this makes maintenance easier, as you have a better overview.

[GitHub] [lucene-solr] dweiss commented on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
dweiss commented on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683410761 Just to be clear, Robert - I can correct this, let me know if you're ok with it. This is an automated

[GitHub] [lucene-solr] uschindler commented on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
uschindler commented on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683408548 I would make the settings of the linter `@Input` options of the RenderJavadocs task, prepoulated with the project variables (like Javadocs URLs).

[GitHub] [lucene-solr] uschindler commented on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
uschindler commented on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683408276 I agree with Dawid. At the beginning I did not like his approach to centralize everything, but this makes maintenance easier, as you have a better overview. The

[jira] [Commented] (SOLR-13652) Remove update from initParams in example solrconfig files that only mention "df"

2020-08-30 Thread Erick Erickson (Jira)
[ https://issues.apache.org/jira/browse/SOLR-13652?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187183#comment-17187183 ] Erick Erickson commented on SOLR-13652: --- Agreed. Either we need a copyField directive of everything

[jira] [Commented] (SOLR-14784) Reproducible failure for DirectUpdateHandlerTest

2020-08-30 Thread Erick Erickson (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14784?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187181#comment-17187181 ] Erick Erickson commented on SOLR-14784: --- Oops, didn't see you saw this I filed it as a Solr bug

[jira] [Commented] (LUCENE-9456) Stored fields should store the number of chunks in the meta file

2020-08-30 Thread Erick Erickson (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9456?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187180#comment-17187180 ] Erick Erickson commented on LUCENE-9456: [~dsmiley] see: SOLR-14784, [~jpountz]  is working on

[GitHub] [lucene-solr] dweiss commented on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
dweiss commented on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683402017 This looks really good already! My personal preference would be to keep the processing options inside the render javadoc file, rather than pushing them down to each

[jira] [Commented] (SOLR-14785) Update synonyms by API and reload collection in Solr

2020-08-30 Thread Gitterh (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14785?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187166#comment-17187166 ] Gitterh commented on SOLR-14785: Thank you. I've found the issue, it was that in application we also

[jira] [Commented] (SOLR-14794) Pass global cloud configuration to Collection API command execution

2020-08-30 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14794?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187165#comment-17187165 ] ASF subversion and git services commented on SOLR-14794: Commit

[GitHub] [lucene-solr] murblanc merged pull request #1801: SOLR-14794: pass configuration to Collection API commands

2020-08-30 Thread GitBox
murblanc merged pull request #1801: URL: https://github.com/apache/lucene-solr/pull/1801 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] mocobeta edited a comment on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
mocobeta edited a comment on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683397858 I ran the javadoc task and saw how it works. Would it be possible to pass the `javadocMissingLevel` property to the `RenderJavadocTask` as `@Input` (as old

[GitHub] [lucene-solr] mocobeta edited a comment on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
mocobeta edited a comment on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683397858 I ran the javadoc task and saw how it works. Would it be possible to pass the `javadocMissingLevel` property to the `RenderJavadocTask` as `@Input` (as old

[GitHub] [lucene-solr] mocobeta commented on pull request #1802: LUCENE-9215: replace checkJavaDocs.py with doclet

2020-08-30 Thread GitBox
mocobeta commented on pull request #1802: URL: https://github.com/apache/lucene-solr/pull/1802#issuecomment-683397858 I ran the javadoc task and saw how it works. Would it be possible to pass the `javadocMissingLevel` property to the javadoc task as `Input` (as old

[jira] [Commented] (SOLR-14784) Reproducible failure for DirectUpdateHandlerTest

2020-08-30 Thread David Smiley (Jira)
[ https://issues.apache.org/jira/browse/SOLR-14784?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187143#comment-17187143 ] David Smiley commented on SOLR-14784: - Oh geesh; I didn't see this issue until now. Today I debugged

[jira] [Commented] (SOLR-13652) Remove update from initParams in example solrconfig files that only mention "df"

2020-08-30 Thread David Smiley (Jira)
[ https://issues.apache.org/jira/browse/SOLR-13652?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17187140#comment-17187140 ] David Smiley commented on SOLR-13652: - FWIW I'm not a fan of "df" having a default unless this is

  1   2   >