Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/11674
It's different from `evaluateRequiredVariables`,
`evaluateRequiredVariables` pulls out the code to be evaluated and put it in
the beginning of the generated code. However here we just clear the code, which
looks unsafe. If we can't come up with a real case, shall we revert this?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
