Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20021
Well, I proposed to check it only for tests at the beginning, but I don't
have a strong preference now as the new approach I took can guarantee that no
place would violate it, by looking at all the caller sides of
`ctx.splitExpressions`.
Anyway only checking it in tests is safer, WDYT @viirya ?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
