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

Jan Høydahl commented on SOLR-8625:
-----------------------------------

Yep, should also move many of the inline strings into constants, and use the 
constant in the error msg. Some params are also shared between {{SQLHandler}} 
and SolrJ's {{DriverImpl}}.. Perhaps a new class {{SQLParameters.java}} would 
be suitable?

> SQL: Wrong error msg if stmt param is missing
> ---------------------------------------------
>
>                 Key: SOLR-8625
>                 URL: https://issues.apache.org/jira/browse/SOLR-8625
>             Project: Solr
>          Issue Type: Bug
>          Components: Parallell SQL
>    Affects Versions: Trunk
>            Reporter: Jan Høydahl
>             Fix For: Trunk
>
>
> If you call {{/sql?}} without {{stmt}}, the following error msg is printed:
> {code}
> "EXCEPTION": "sql parameter cannot be null"
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to