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