[jira] [Created] (LUCENE-9857) Skip cache building if IndexOrDocValuesQuery choose the dvQuery

2021-03-22 Thread Feng Guo (Jira)
Feng Guo created LUCENE-9857: Summary: Skip cache building if IndexOrDocValuesQuery choose the dvQuery Key: LUCENE-9857 URL: https://issues.apache.org/jira/browse/LUCENE-9857 Project: Lucene - Core

[GitHub] [lucene] gf2121 opened a new pull request #29: LUCENE-9857: Skip cache building if IndexOrDocValuesQuery choose the dvQuery

2021-03-22 Thread GitBox
gf2121 opened a new pull request #29: URL: https://github.com/apache/lucene/pull/29 IndexOrDocValuesQuery can automatically use dvQueries when the cost > 8 *l eadcost, And the LRUQueryCache skips cache building when cost > 250(By default) * leadcost. There is a gap between 8 and 250, which

[GitHub] [lucene] iverase opened a new pull request #30: LUCENE-9705: Create Lucene90SegmentInfoFormat

2021-03-22 Thread GitBox
iverase opened a new pull request #30: URL: https://github.com/apache/lucene/pull/30 For now this is just a copy of Lucene86SegmentInfoFormat. The existing Lucene86SegmentInfoFormat is moved to backwards-codecs. -- This is an automated message from the Apache Git Service. To respond t

[jira] [Created] (LUCENE-9858) Remove Google Analytics from Lucene site

2021-03-22 Thread Jira
Jan Høydahl created LUCENE-9858: --- Summary: Remove Google Analytics from Lucene site Key: LUCENE-9858 URL: https://issues.apache.org/jira/browse/LUCENE-9858 Project: Lucene - Core Issue Type: Ta

[jira] [Commented] (LUCENE-9413) Add a char filter corresponding to CJKWidthFilter

2021-03-22 Thread Tomoko Uchida (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9413?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17305997#comment-17305997 ] Tomoko Uchida commented on LUCENE-9413: --- FYI: I'm planning to change the JapaneseA

[jira] [Commented] (LUCENE-9855) Reconsider codec name VectorFormat

2021-03-22 Thread Tomoko Uchida (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9855?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306063#comment-17306063 ] Tomoko Uchida commented on LUCENE-9855: --- I'm inclined to DenseVectorsFormat, to di

[GitHub] [lucene] asalamon74 opened a new pull request #31: SOLR-14024 Invalid html generated by changes2html.pl

2021-03-22 Thread GitBox
asalamon74 opened a new pull request #31: URL: https://github.com/apache/lucene/pull/31 Original lucene-solr pull request: https://github.com/apache/lucene-solr/pull/1060 # Description changes2html.pl generates invalid html # Solution Fix it to generate a

[jira] [Created] (LUCENE-9859) Failure in TestXYPointQueries

2021-03-22 Thread Ignacio Vera (Jira)
Ignacio Vera created LUCENE-9859: Summary: Failure in TestXYPointQueries Key: LUCENE-9859 URL: https://issues.apache.org/jira/browse/LUCENE-9859 Project: Lucene - Core Issue Type: Bug

[GitHub] [lucene] dweiss merged pull request #27: LUCENE-9854: Clean up utilities to download and extract test/ benchmark data sets.

2021-03-22 Thread GitBox
dweiss merged pull request #27: URL: https://github.com/apache/lucene/pull/27 -- 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 conta

[jira] [Commented] (LUCENE-9854) Clean up utilities to download and extract test/ benchmark data sets.

2021-03-22 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9854?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306128#comment-17306128 ] ASF subversion and git services commented on LUCENE-9854: - Commi

[jira] [Resolved] (LUCENE-9854) Clean up utilities to download and extract test/ benchmark data sets.

2021-03-22 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9854?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dawid Weiss resolved LUCENE-9854. - Fix Version/s: main (9.0) Resolution: Fixed > Clean up utilities to download and extract

[GitHub] [lucene] rmuir commented on a change in pull request #26: LUCENE-9853: Use CJKWidthCharFilter as the default character width normalizer in JapaneseAnalyzer

2021-03-22 Thread GitBox
rmuir commented on a change in pull request #26: URL: https://github.com/apache/lucene/pull/26#discussion_r598268124 ## File path: lucene/analysis/kuromoji/src/java/org/apache/lucene/analysis/ja/JapaneseAnalyzer.java ## @@ -39,21 +41,28 @@ private final Mode mode; privat

[GitHub] [lucene-solr] magibney commented on a change in pull request #2470: document merge-based approach to updating existing PRs

2021-03-22 Thread GitBox
magibney commented on a change in pull request #2470: URL: https://github.com/apache/lucene-solr/pull/2470#discussion_r598724131 ## File path: PRs.md ## @@ -64,3 +64,45 @@ Look at the PR and create it if it looks good. # This example's PR was at: https://github.com/apache/luc

[GitHub] [lucene] magibney commented on a change in pull request #15: LUCENE-8972: Add ICUTransformCharFilter, to support pre-tokenizer ICU text transformation

2021-03-22 Thread GitBox
magibney commented on a change in pull request #15: URL: https://github.com/apache/lucene/pull/15#discussion_r598735719 ## File path: lucene/analysis/icu/src/java/org/apache/lucene/analysis/icu/ICUTransformCharFilterFactory.java ## @@ -0,0 +1,391 @@ +/* + * Licensed to the Apa

[GitHub] [lucene] gsmiller commented on a change in pull request #25: LUCENE-9385: Add option to skip indexing facet drill down terms

2021-03-22 Thread GitBox
gsmiller commented on a change in pull request #25: URL: https://github.com/apache/lucene/pull/25#discussion_r598748091 ## File path: lucene/facet/src/java/org/apache/lucene/facet/FacetsConfig.java ## @@ -65,6 +65,38 @@ // int/float/bytes in a single indexed field: privat

[GitHub] [lucene] gsmiller commented on a change in pull request #25: LUCENE-9385: Add option to skip indexing facet drill down terms

2021-03-22 Thread GitBox
gsmiller commented on a change in pull request #25: URL: https://github.com/apache/lucene/pull/25#discussion_r598753012 ## File path: lucene/facet/src/java/org/apache/lucene/facet/FacetsConfig.java ## @@ -371,14 +426,26 @@ private void processFacetFields( }

[GitHub] [lucene] gsmiller commented on a change in pull request #25: LUCENE-9385: Add option to skip indexing facet drill down terms

2021-03-22 Thread GitBox
gsmiller commented on a change in pull request #25: URL: https://github.com/apache/lucene/pull/25#discussion_r598757471 ## File path: lucene/facet/src/java/org/apache/lucene/facet/FacetsConfig.java ## @@ -371,14 +426,26 @@ private void processFacetFields( }

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread Michael McCandless (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306234#comment-17306234 ] Michael McCandless commented on LUCENE-9856: I will cut a branch!  Let's sta

[GitHub] [lucene] mocobeta commented on a change in pull request #26: LUCENE-9853: Use CJKWidthCharFilter as the default character width normalizer in JapaneseAnalyzer

2021-03-22 Thread GitBox
mocobeta commented on a change in pull request #26: URL: https://github.com/apache/lucene/pull/26#discussion_r598772602 ## File path: lucene/analysis/kuromoji/src/java/org/apache/lucene/analysis/ja/JapaneseAnalyzer.java ## @@ -39,21 +41,28 @@ private final Mode mode; pri

[jira] [Commented] (LUCENE-8962) Can we merge small segments during refresh, for faster searching?

2021-03-22 Thread Michael McCandless (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-8962?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306249#comment-17306249 ] Michael McCandless commented on LUCENE-8962: Hi Team!  This issue was so exo

[jira] [Updated] (LUCENE-8962) Can we merge small segments during refresh, for faster searching?

2021-03-22 Thread Michael McCandless (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-8962?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Michael McCandless updated LUCENE-8962: --- Attachment: image-2021-03-22-10-36-32-201.png > Can we merge small segments during r

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306264#comment-17306264 ] ASF subversion and git services commented on LUCENE-9856: - Commi

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306265#comment-17306265 ] ASF subversion and git services commented on LUCENE-9856: - Commi

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread Michael McCandless (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306269#comment-17306269 ] Michael McCandless commented on LUCENE-9856: OK I think I succeeded!   https

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306271#comment-17306271 ] Dawid Weiss commented on LUCENE-9856: - No magic is needed as ecj runs per-module (su

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306283#comment-17306283 ] Robert Muir commented on LUCENE-9856: - [~dweiss] i don't want to disable *entire ecj

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306285#comment-17306285 ] Dawid Weiss commented on LUCENE-9856: - The pattern will be similar – apply the right

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306287#comment-17306287 ] Robert Muir commented on LUCENE-9856: - So in pseudocode something like this (similar

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306290#comment-17306290 ] Robert Muir commented on LUCENE-9856: - {quote} This aside, maybe we should return to

[jira] [Created] (LUCENE-9860) explore disabling ecj javadocs processing

2021-03-22 Thread Robert Muir (Jira)
Robert Muir created LUCENE-9860: --- Summary: explore disabling ecj javadocs processing Key: LUCENE-9860 URL: https://issues.apache.org/jira/browse/LUCENE-9860 Project: Lucene - Core Issue Type: T

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306297#comment-17306297 ] Dawid Weiss commented on LUCENE-9856: - Ok, let's leave it for later. Your snippet is

[jira] [Commented] (LUCENE-9860) explore disabling ecj javadocs processing

2021-03-22 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9860?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306298#comment-17306298 ] Dawid Weiss commented on LUCENE-9860: - +1 from me. > explore disabling ecj javadocs

[jira] [Commented] (LUCENE-9860) explore disabling ecj javadocs processing

2021-03-22 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9860?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306299#comment-17306299 ] Dawid Weiss commented on LUCENE-9860: - We can enable full processing on nightlies if

[jira] [Commented] (LUCENE-9860) explore disabling ecj javadocs processing

2021-03-22 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9860?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306302#comment-17306302 ] Robert Muir commented on LUCENE-9860: - I think the change is to simply remove {{-ena

[jira] [Commented] (LUCENE-9860) explore disabling ecj javadocs processing

2021-03-22 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9860?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306308#comment-17306308 ] Dawid Weiss commented on LUCENE-9860: - Disable this and let it bake. Those time stat

[jira] [Commented] (LUCENE-9860) explore disabling ecj javadocs processing

2021-03-22 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9860?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306312#comment-17306312 ] Robert Muir commented on LUCENE-9860: - This URL didnt work (i think it was expired)

[GitHub] [lucene] mayya-sharipova opened a new pull request #32: LUCENE-9507 Custom order for leaves in IndexReader and IndexWriter

2021-03-22 Thread GitBox
mayya-sharipova opened a new pull request #32: URL: https://github.com/apache/lucene/pull/32 1. Add an option to supply a custom leaf sorter for IndexWriter. A DirectoryReader opened from this IndexWriter will have its leaf readers sorted with the provided leaf sorter. This is useful f

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

2021-03-22 Thread GitBox
mayya-sharipova closed pull request #2256: URL: https://github.com/apache/lucene-solr/pull/2256 -- 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 ser

[jira] [Commented] (LUCENE-9860) explore disabling ecj javadocs processing

2021-03-22 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9860?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306317#comment-17306317 ] Dawid Weiss commented on LUCENE-9860: - Locally you can add -Ptask.times=true to show

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

2021-03-22 Thread GitBox
mayya-sharipova commented on pull request #2256: URL: https://github.com/apache/lucene-solr/pull/2256#issuecomment-804182320 I've moved this PR to a [new PR](https://github.com/apache/lucene/pull/32) in the new Lucene github repo. Closing this PR. -- This is an automated message from t

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

2021-03-22 Thread GitBox
mayya-sharipova commented on a change in pull request #2256: URL: https://github.com/apache/lucene-solr/pull/2256#discussion_r598854515 ## File path: lucene/core/src/test/org/apache/lucene/index/TestIndexWriterReader.java ## @@ -169,7 +176,7 @@ public void testUpdateDocument()

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

2021-03-22 Thread GitBox
mayya-sharipova commented on a change in pull request #2256: URL: https://github.com/apache/lucene-solr/pull/2256#discussion_r598855055 ## File path: lucene/core/src/java/org/apache/lucene/index/DirectoryReader.java ## @@ -56,7 +57,24 @@ * @throws IOException if there is a

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

2021-03-22 Thread GitBox
mayya-sharipova commented on a change in pull request #2256: URL: https://github.com/apache/lucene-solr/pull/2256#discussion_r598857854 ## File path: lucene/core/src/java/org/apache/lucene/index/IndexWriterConfig.java ## @@ -478,6 +479,18 @@ public IndexWriterConfig setIndexSor

[jira] [Commented] (LUCENE-9855) Reconsider codec name VectorFormat

2021-03-22 Thread Michael Sokolov (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9855?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306324#comment-17306324 ] Michael Sokolov commented on LUCENE-9855: - Thanks for picking this up, [~tomoko]

[jira] [Commented] (LUCENE-9860) explore disabling ecj javadocs processing

2021-03-22 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9860?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306404#comment-17306404 ] Robert Muir commented on LUCENE-9860: - I played around, even disabling so much javad

[jira] [Resolved] (LUCENE-9860) explore disabling ecj javadocs processing

2021-03-22 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9860?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Muir resolved LUCENE-9860. - Resolution: Won't Fix > explore disabling ecj javadocs processing >

[jira] [Created] (LUCENE-9861) speed up precommit

2021-03-22 Thread Robert Muir (Jira)
Robert Muir created LUCENE-9861: --- Summary: speed up precommit Key: LUCENE-9861 URL: https://issues.apache.org/jira/browse/LUCENE-9861 Project: Lucene - Core Issue Type: Task Reporte

[jira] [Commented] (LUCENE-9861) speed up precommit

2021-03-22 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9861?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306439#comment-17306439 ] Robert Muir commented on LUCENE-9861: - For renderJavadoc this is also 2x speedup: Be

[jira] [Updated] (LUCENE-9861) speed up precommit

2021-03-22 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9861?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Robert Muir updated LUCENE-9861: Attachment: LUCENE-9861_hack.patch Status: Open (was: Open) My current hack patch. Still

[jira] [Commented] (LUCENE-9861) speed up precommit

2021-03-22 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9861?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306462#comment-17306462 ] Robert Muir commented on LUCENE-9861: - simple bench of current hack patch before: {

[jira] [Commented] (LUCENE-9861) speed up precommit

2021-03-22 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9861?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306526#comment-17306526 ] Dawid Weiss commented on LUCENE-9861: - Whaaat?! :) I'll take a look tomorrow, Robert

[jira] [Commented] (LUCENE-9861) speed up precommit

2021-03-22 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9861?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306528#comment-17306528 ] Dawid Weiss commented on LUCENE-9861: - I did take a look at the patch... oh, man. Th

[jira] [Commented] (LUCENE-9860) explore disabling ecj javadocs processing

2021-03-22 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9860?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306531#comment-17306531 ] Dawid Weiss commented on LUCENE-9860: - Let's close this one then? > explore disabli

[jira] [Commented] (LUCENE-9860) explore disabling ecj javadocs processing

2021-03-22 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9860?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306533#comment-17306533 ] Robert Muir commented on LUCENE-9860: - yes, i closed it as won't fix > explore disa

[jira] [Commented] (LUCENE-9861) speed up precommit

2021-03-22 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9861?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306534#comment-17306534 ] Uwe Schindler commented on LUCENE-9861: --- I would really like to look also into not

[jira] [Comment Edited] (LUCENE-9861) speed up precommit

2021-03-22 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9861?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306534#comment-17306534 ] Uwe Schindler edited comment on LUCENE-9861 at 3/22/21, 8:00 PM: -

[jira] [Commented] (LUCENE-9861) speed up precommit

2021-03-22 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9861?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306536#comment-17306536 ] Robert Muir commented on LUCENE-9861: - Not forking is fine too. But forking with C2

[jira] [Commented] (LUCENE-9861) speed up precommit

2021-03-22 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9861?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306540#comment-17306540 ] Uwe Schindler commented on LUCENE-9861: --- Yes, especially also renderJavadoc should

[jira] [Commented] (LUCENE-9861) speed up precommit

2021-03-22 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9861?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306549#comment-17306549 ] Robert Muir commented on LUCENE-9861: - Also the "-times" can be removed from the pat

[jira] [Commented] (LUCENE-9861) speed up precommit

2021-03-22 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9861?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306558#comment-17306558 ] Robert Muir commented on LUCENE-9861: - {quote} But for stuff that works perfectly fi

[GitHub] [lucene] rmuir commented on pull request #31: SOLR-14024 Invalid html generated by changes2html.pl

2021-03-22 Thread GitBox
rmuir commented on pull request #31: URL: https://github.com/apache/lucene/pull/31#issuecomment-804411503 Thanks @asalamon74 ! I validated the change (though i had to tweak the comments so that the utf-8 encoding was within first 1024 bytes for https://validator.w3.org/ to accept it, anoth

[GitHub] [lucene] rmuir merged pull request #31: SOLR-14024 Invalid html generated by changes2html.pl

2021-03-22 Thread GitBox
rmuir merged pull request #31: URL: https://github.com/apache/lucene/pull/31 -- 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 contac

[GitHub] [lucene] dweiss opened a new pull request #33: LUCENE-9861: pull tuned vm options into a separate aspect.

2021-03-22 Thread GitBox
dweiss opened a new pull request #33: URL: https://github.com/apache/lucene/pull/33 -- 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

[jira] [Commented] (LUCENE-9861) speed up precommit

2021-03-22 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9861?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306599#comment-17306599 ] Dawid Weiss commented on LUCENE-9861: - I've refactored this into a somewhat more sha

[jira] [Commented] (LUCENE-9861) speed up precommit

2021-03-22 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9861?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306602#comment-17306602 ] Dawid Weiss commented on LUCENE-9861: - The problem with running javadocs in-process

[jira] [Commented] (LUCENE-9861) speed up precommit

2021-03-22 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9861?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306604#comment-17306604 ] Dawid Weiss commented on LUCENE-9861: - My results. Before: {code} Aggregate task tim

[GitHub] [lucene] gsmiller commented on a change in pull request #25: LUCENE-9385: Add option to skip indexing facet drill down terms

2021-03-22 Thread GitBox
gsmiller commented on a change in pull request #25: URL: https://github.com/apache/lucene/pull/25#discussion_r599105765 ## File path: lucene/facet/src/java/org/apache/lucene/facet/FacetsConfig.java ## @@ -408,7 +475,7 @@ private void processSSDVFacetFields( doc.add(new

[GitHub] [lucene] gsmiller commented on a change in pull request #25: LUCENE-9385: Add option to skip indexing facet drill down terms

2021-03-22 Thread GitBox
gsmiller commented on a change in pull request #25: URL: https://github.com/apache/lucene/pull/25#discussion_r599106387 ## File path: lucene/facet/src/java/org/apache/lucene/facet/FacetsConfig.java ## @@ -444,12 +511,6 @@ private void processAssocFacetFields( FacetsCon

[GitHub] [lucene] rmuir commented on a change in pull request #33: LUCENE-9861: pull tuned vm options into a separate aspect.

2021-03-22 Thread GitBox
rmuir commented on a change in pull request #33: URL: https://github.com/apache/lucene/pull/33#discussion_r599114199 ## File path: gradle/validation/ecj-lint.gradle ## @@ -74,9 +74,6 @@ allprojects { args += [ "-enableJavadoc" ] args += [ "-properties", file(

[GitHub] [lucene] rmuir commented on a change in pull request #33: LUCENE-9861: pull tuned vm options into a separate aspect.

2021-03-22 Thread GitBox
rmuir commented on a change in pull request #33: URL: https://github.com/apache/lucene/pull/33#discussion_r599114563 ## File path: gradle/validation/ecj-lint.gradle ## @@ -74,9 +74,6 @@ allprojects { args += [ "-enableJavadoc" ] args += [ "-properties", file(

[jira] [Commented] (LUCENE-9861) speed up precommit

2021-03-22 Thread Uwe Schindler (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9861?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306620#comment-17306620 ] Uwe Schindler commented on LUCENE-9861: --- Looks slower after? 🤔 > speed up precomm

[GitHub] [lucene] rmuir commented on pull request #33: LUCENE-9861: pull tuned vm options into a separate aspect.

2021-03-22 Thread GitBox
rmuir commented on pull request #33: URL: https://github.com/apache/lucene/pull/33#issuecomment-804436499 I compared times with another PR (just looking at the github check times), and it shaved a few minutes off -- This is an automated message from the Apache Git Service. To respond to

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306624#comment-17306624 ] ASF subversion and git services commented on LUCENE-9856: - Commi

[jira] [Commented] (LUCENE-9861) speed up precommit

2021-03-22 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9861?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306625#comment-17306625 ] Robert Muir commented on LUCENE-9861: - If you have 87 cores, you can afford to spend

[jira] [Commented] (LUCENE-9861) speed up precommit

2021-03-22 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9861?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306627#comment-17306627 ] Robert Muir commented on LUCENE-9861: - Another way to look at it, if you see that, y

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306649#comment-17306649 ] ASF subversion and git services commented on LUCENE-9856: - Commi

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306650#comment-17306650 ] ASF subversion and git services commented on LUCENE-9856: - Commi

[GitHub] [lucene] rmuir opened a new pull request #34: LUCENE-9856: fail precommit on unused local variables

2021-03-22 Thread GitBox
rmuir opened a new pull request #34: URL: https://github.com/apache/lucene/pull/34 See the JIRA for more details. This turns on ecj option: `ecj.javadocs.prefs:org.eclipse.jdt.core.compiler.problem.unusedLocal=error` But some modules have some generated code, and there is no w

[GitHub] [lucene] rmuir commented on pull request #34: LUCENE-9856: fail precommit on unused local variables

2021-03-22 Thread GitBox
rmuir commented on pull request #34: URL: https://github.com/apache/lucene/pull/34#issuecomment-804474683 This change is largely mechanical: simply remove the unused local variables. Try to preserve any method calls that may have side effects (e.g. throw exception) in tests. In gene

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306662#comment-17306662 ] ASF subversion and git services commented on LUCENE-9856: - Commi

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306663#comment-17306663 ] ASF subversion and git services commented on LUCENE-9856: - Commi

[GitHub] [lucene] rmuir commented on pull request #34: LUCENE-9856: fail precommit on unused local variables

2021-03-22 Thread GitBox
rmuir commented on pull request #34: URL: https://github.com/apache/lucene/pull/34#issuecomment-804487699 OK i reverted the gradle changes completely, i found a secret option so that `SuppressWarnings` can be used. This fixes the expressions module, but we need to apply `SuppressWarnings("

[GitHub] [lucene] uschindler commented on a change in pull request #33: LUCENE-9861: pull tuned vm options into a separate aspect.

2021-03-22 Thread GitBox
uschindler commented on a change in pull request #33: URL: https://github.com/apache/lucene/pull/33#discussion_r599162761 ## File path: gradle/validation/ecj-lint.gradle ## @@ -74,9 +74,6 @@ allprojects { args += [ "-enableJavadoc" ] args += [ "-properties",

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306673#comment-17306673 ] ASF subversion and git services commented on LUCENE-9856: - Commi

[GitHub] [lucene] zacharymorn commented on a change in pull request #25: LUCENE-9385: Add option to skip indexing facet drill down terms

2021-03-22 Thread GitBox
zacharymorn commented on a change in pull request #25: URL: https://github.com/apache/lucene/pull/25#discussion_r599170681 ## File path: lucene/facet/src/java/org/apache/lucene/facet/FacetsConfig.java ## @@ -371,14 +426,26 @@ private void processFacetFields( }

[GitHub] [lucene] zacharymorn commented on a change in pull request #25: LUCENE-9385: Add option to skip indexing facet drill down terms

2021-03-22 Thread GitBox
zacharymorn commented on a change in pull request #25: URL: https://github.com/apache/lucene/pull/25#discussion_r599171007 ## File path: lucene/facet/src/java/org/apache/lucene/facet/FacetsConfig.java ## @@ -371,14 +426,26 @@ private void processFacetFields( }

[GitHub] [lucene] zacharymorn commented on a change in pull request #25: LUCENE-9385: Add option to skip indexing facet drill down terms

2021-03-22 Thread GitBox
zacharymorn commented on a change in pull request #25: URL: https://github.com/apache/lucene/pull/25#discussion_r599171189 ## File path: lucene/facet/src/java/org/apache/lucene/facet/FacetsConfig.java ## @@ -408,7 +475,7 @@ private void processSSDVFacetFields( doc.add(

[GitHub] [lucene] zacharymorn commented on a change in pull request #25: LUCENE-9385: Add option to skip indexing facet drill down terms

2021-03-22 Thread GitBox
zacharymorn commented on a change in pull request #25: URL: https://github.com/apache/lucene/pull/25#discussion_r599171328 ## File path: lucene/facet/src/java/org/apache/lucene/facet/FacetsConfig.java ## @@ -444,12 +511,6 @@ private void processAssocFacetFields( Facets

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread Robert Muir (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306677#comment-17306677 ] Robert Muir commented on LUCENE-9856: - While wrestling with the PR, I found out abou

[GitHub] [lucene] gsmiller commented on a change in pull request #25: LUCENE-9385: Add option to skip indexing facet drill down terms

2021-03-22 Thread GitBox
gsmiller commented on a change in pull request #25: URL: https://github.com/apache/lucene/pull/25#discussion_r599193102 ## File path: lucene/facet/src/java/org/apache/lucene/facet/FacetsConfig.java ## @@ -371,14 +426,26 @@ private void processFacetFields( }

[GitHub] [lucene] gsmiller commented on a change in pull request #25: LUCENE-9385: Add option to skip indexing facet drill down terms

2021-03-22 Thread GitBox
gsmiller commented on a change in pull request #25: URL: https://github.com/apache/lucene/pull/25#discussion_r599195582 ## File path: lucene/facet/src/java/org/apache/lucene/facet/FacetsConfig.java ## @@ -408,7 +475,7 @@ private void processSSDVFacetFields( doc.add(new

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306709#comment-17306709 ] ASF subversion and git services commented on LUCENE-9856: - Commi

[jira] [Created] (LUCENE-9862) gradlew regenerate doesnt invoke jflexStandardTokenizerImpl

2021-03-22 Thread Robert Muir (Jira)
Robert Muir created LUCENE-9862: --- Summary: gradlew regenerate doesnt invoke jflexStandardTokenizerImpl Key: LUCENE-9862 URL: https://issues.apache.org/jira/browse/LUCENE-9862 Project: Lucene - Core

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306721#comment-17306721 ] ASF subversion and git services commented on LUCENE-9856: - Commi

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306738#comment-17306738 ] ASF subversion and git services commented on LUCENE-9856: - Commi

[jira] [Commented] (LUCENE-9856) Improve static analysis to catch dead code and other potential bugs

2021-03-22 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306761#comment-17306761 ] ASF subversion and git services commented on LUCENE-9856: - Commi

[GitHub] [lucene] rmuir commented on pull request #34: LUCENE-9856: fail precommit on unused local variables

2021-03-22 Thread GitBox
rmuir commented on pull request #34: URL: https://github.com/apache/lucene/pull/34#issuecomment-804606020 ok, I think ecj linter is now in better shape, especially as far as dead code goes. But wow, the amount of dead `private` members and methods that we had was really surprising.

[jira] [Commented] (LUCENE-9861) speed up precommit

2021-03-22 Thread Dawid Weiss (Jira)
[ https://issues.apache.org/jira/browse/LUCENE-9861?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17306803#comment-17306803 ] Dawid Weiss commented on LUCENE-9861: - The improvement is clearly seen on individual