Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/19578#discussion_r147381885 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/ParseDriver.scala --- @@ -151,9 +152,30 @@ object CatalystSqlParser extends AbstractSqlParser { * have the ANTLRNoCaseStringStream implementation. */ -private[parser] class ANTLRNoCaseStringStream(input: String) extends ANTLRInputStream(input) { +private[parser] class ANTLRNoCaseStringStream(wrapped: CodePointCharStream) extends CharStream { --- End diff -- Maybe we should rename this to matter match what it is actually doing. `NoCaseCharStream`?
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org