Github user diegoceccarelli commented on a diff in the pull request:

    https://github.com/apache/lucene-solr/pull/295#discussion_r158969387
  
    --- Diff: solr/core/src/java/org/apache/solr/search/QParser.java ---
    @@ -360,6 +360,10 @@ public static QParser getParser(String qstr, String 
parserName, boolean allowLoc
         }
     
         QParserPlugin qplug = req.getCore().getQueryPlugin(parserName);
    +    if (qplug == null){
    +      // error: log ?
    --- End diff --
    
    I would log an error here, but `QParser` doesn't have a logger, should we 
add one? 


---

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

Reply via email to