Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22355#discussion_r217054661
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/CodeGeneratorWithInterpretedFallback.scala
---
@@ -37,19 +37,22 @@ object CodegenObjectFactoryMode extends Enumeration {
*/
abstract class CodeGeneratorWithInterpretedFallback[IN, OUT] extends
Logging {
- def createObject(in: IN): OUT = {
+ def createObject(in: IN): OUT =
+ createObject(in, subexpressionEliminationEnabled = false)
--- End diff --
Can we eliminate it? I think we can use `SQLConf.get` directly when we need
to access the conf. This should be done in a different PR though.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
