[jira] [Comment Edited] (SOLR-14557) eDisMax parser unable to parse {!lucene}(gigabyte)

["Mikhail Khludnev (Jira)" <ji...@apache.org>]

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

Mikhail Khludnev edited comment on SOLR-14557 at 6/30/20, 9:46 AM:
-------------------------------------------------------------------

Note that local params pattern doesn't include closing parenthesis 
{code}
| <LPARAMS:       ("{!" ( (<_WHITESPACE>)* (~["=","}"])+ ( "=" (<QUOTED> | ("'" (<_SQUOTED_CHAR>)* "'") | (~[" ","}"])+ )? )? )* "}")+  (~[")"," ","\t","\n","{","^"])*  >
{code}
If I include it like
{code}
 (~[")"," ","\t","\n","{","^"])* ([")"])? >
{code}
It fixes my test {{\{!lucene}(gigabyte)}}, but breaks another parenthesis {{(\{!lucene}gigabyte)}}.


was (Author: mkhludnev):
Note that local params pattern doesn't include closing parenthesis 
{code}
| <LPARAMS:       ("{!" ( (<_WHITESPACE>)* (~["=","}"])+ ( "=" (<QUOTED> | ("'" (<_SQUOTED_CHAR>)* "'") | (~[" ","}"])+ )? )? )* "}")+  (~[")"," ","\t","\n","{","^"])*  >
{code}
If I include it like
{code}
 (~[")"," ","\t","\n","{","^"])* ([")"])? >
{code}
It fixes my test {{\{!lucene}(gigabyte)}}, but breaks another parenthesis {{(\{!lucne}gigabyte)}}.

> eDisMax parser unable to parse {!lucene}(gigabyte)
> --------------------------------------------------
>
>                 Key: SOLR-14557
>                 URL: https://issues.apache.org/jira/browse/SOLR-14557
>             Project: Solr
>          Issue Type: Bug
>      Security Level: Public(Default Security Level. Issues are Public) 
>          Components: query parsers
>            Reporter: Mikhail Khludnev
>            Priority: Major
>              Labels: painful
>         Attachments: SOLR-14557.patch, SOLR-14557.patch
>
>
> h2. Solr 4.5
> {{/select?defType=edismax&q=\{!lucene}(foo)&debugQuery=true}} 
>  
>  goes like
>  {code}
>  <str name="rawquerystring">\{!lucene}(foo)</str>
>  <str name="parsedquery_toString">content:foo</str>
>  <str name="QParser">LuceneQParser</str>
> {code}
> fine
> h2. Solr 8.2 
> with luceneMatchVersion=4.5 following SOLR-11501 I know it's a grey zone but it's a question of migrating existing queries. 
> {{/select?defType=edismax&q=\{!lucene}(foo)&debugQuery=true}} 
> goes like 
> {code}
> "querystring":"\{!lucene}(foo)",
>  "parsedquery":"+DisjunctionMaxQuery(((Project.Address:lucene Project.Address:foo) | (Project.OwnerType:lucene Project.OwnerType:foo) 
>  "QParser":"ExtendedDismaxQParser",
> {code}
> blah... 
> but removing braces in 8.2 works perfectly fine 
> {code}
> "querystring":"\{!lucene}foo",
>  "parsedquery":"+content:foo",
>  "parsedquery_toString":"+content:foo",
>  "QParser":"ExtendedDismaxQParser",
>  {code}



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

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