gengliangwang commented on code in PR #46251: URL: https://github.com/apache/spark/pull/46251#discussion_r1581597745
########## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala: ########## @@ -190,7 +190,7 @@ case class Abs(child: Expression, failOnError: Boolean = SQLConf.get.ansiEnabled abstract class BinaryArithmetic extends BinaryOperator with NullIntolerant with SupportQueryContext { - protected val evalMode: EvalMode.Value + protected[sql] val evalMode: EvalMode.Value Review Comment: This is to make sure we can access `evalMode` in tests. -- 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. To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org