[ https://issues.apache.org/jira/browse/LUCENE-8332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16499167#comment-16499167 ]
Lucene/Solr QA commented on LUCENE-8332: ---------------------------------------- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 7 new or modified test files. {color} | || || || || {color:brown} master Compile Tests {color} || | {color:green}+1{color} | {color:green} compile {color} | {color:green} 1m 5s{color} | {color:green} master passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:red}-1{color} | {color:red} compile {color} | {color:red} 0m 22s{color} | {color:red} suggest in the patch failed. {color} | | {color:red}-1{color} | {color:red} javac {color} | {color:red} 0m 22s{color} | {color:red} suggest in the patch failed. {color} | | {color:green}+1{color} | {color:green} Release audit (RAT) {color} | {color:green} 0m 41s{color} | {color:green} Release audit (RAT) rat-sources passed {color} | | {color:red}-1{color} | {color:red} Release audit (RAT) {color} | {color:red} 0m 5s{color} | {color:red} suggest in the patch failed. {color} | | {color:red}-1{color} | {color:red} Check forbidden APIs {color} | {color:red} 0m 41s{color} | {color:red} Check forbidden APIs check-forbidden-apis failed {color} | | {color:red}-1{color} | {color:red} Validate source patterns {color} | {color:red} 0m 41s{color} | {color:red} Check forbidden APIs check-forbidden-apis failed {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 7m 47s{color} | {color:green} common in the patch passed. {color} | | {color:red}-1{color} | {color:red} unit {color} | {color:red} 0m 12s{color} | {color:red} suggest in the patch failed. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 12m 31s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | JIRA Issue | LUCENE-8332 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12926180/LUCENE-8332.patch | | Optional Tests | compile javac unit ratsources checkforbiddenapis validatesourcepatterns | | uname | Linux lucene1-us-west 3.13.0-88-generic #135-Ubuntu SMP Wed Jun 8 21:10:42 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | ant | | Personality | /home/jenkins/jenkins-slave/workspace/PreCommit-LUCENE-Build/sourcedir/dev-tools/test-patch/lucene-solr-yetus-personality.sh | | git revision | master / 3dc4fa1 | | ant | version: Apache Ant(TM) version 1.9.3 compiled on April 8 2014 | | Default Java | 1.8.0_172 | | compile | https://builds.apache.org/job/PreCommit-LUCENE-Build/24/artifact/out/patch-compile-lucene_suggest.txt | | javac | https://builds.apache.org/job/PreCommit-LUCENE-Build/24/artifact/out/patch-compile-lucene_suggest.txt | | Release audit (RAT) | https://builds.apache.org/job/PreCommit-LUCENE-Build/24/artifact/out/patch-compile-lucene_suggest.txt | | Check forbidden APIs | https://builds.apache.org/job/PreCommit-LUCENE-Build/24/artifact/out/patch-check-forbidden-apis-lucene.txt | | Validate source patterns | https://builds.apache.org/job/PreCommit-LUCENE-Build/24/artifact/out/patch-check-forbidden-apis-lucene.txt | | unit | https://builds.apache.org/job/PreCommit-LUCENE-Build/24/artifact/out/patch-unit-lucene_suggest.txt | | Test Results | https://builds.apache.org/job/PreCommit-LUCENE-Build/24/testReport/ | | modules | C: lucene/analysis/common lucene/suggest U: lucene | | Console output | https://builds.apache.org/job/PreCommit-LUCENE-Build/24/console | | Powered by | Apache Yetus 0.7.0 http://yetus.apache.org | This message was automatically generated. > New ConcatenateGraphTokenStream (move/rename CompletionTokenStream) > ------------------------------------------------------------------- > > Key: LUCENE-8332 > URL: https://issues.apache.org/jira/browse/LUCENE-8332 > Project: Lucene - Core > Issue Type: New Feature > Components: modules/analysis > Reporter: David Smiley > Assignee: David Smiley > Priority: Major > Attachments: LUCENE-8332.patch, LUCENE-8332.patch, LUCENE-8332.patch > > Time Spent: 3h 10m > Remaining Estimate: 0h > > Lets move and rename the CompletionTokenStream in the suggest module into the > analysis module renamed as ConcatenateGraphTokenStream. See comments in > LUCENE-8323 leading to this idea. Such a TokenStream (or TokenFilter?) has > several uses: > * for the suggest module > * by the SolrTextTagger for NER/ERD use cases – SOLR-12376 > * for doing complete match search efficiently > It will need a factory – a TokenFilterFactory, even though we don't have a > TokenFilter based subclass of TokenStream. > It appears there is no back-compat concern in it suddenly disappearing from > the suggest module as it's marked experimental and it only seems to be public > now perhaps due to some technicality (it has package level constructors). -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org