Jackie-Jiang commented on a change in pull request #4216: PQL -> SQL enhancement - phase 1 - new Pinot Query Struct URL: https://github.com/apache/incubator-pinot/pull/4216#discussion_r290997578
########## File path: pinot-common/src/main/java/org/apache/pinot/pql/parsers/Pql2Compiler.java ########## @@ -57,9 +61,13 @@ @ThreadSafe public class Pql2Compiler implements AbstractCompiler { + private static final Logger LOGGER = LoggerFactory.getLogger(Pql2Compiler.class); + private static class ErrorListener extends BaseErrorListener { + @Override - public void syntaxError(@Nonnull Recognizer<?, ?> recognizer, @Nullable Object offendingSymbol, int line, + public void syntaxError(@Nonnull Recognizer<?, ?> recognizer, @Nullable Object offendingSymbol, + int line, Review comment: wrong format ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org