Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/21736
@mgaido91 I am not blaming you broke the current implementation. I am just
testing yours. For example, now it is not clear for me why do you need `lazy
val` instead of just `val` in `@transient lazy val legacySizeOfNull =
SQLConf.get.legacySizeOfNull` and `@transient` if `legacySizeOfNull` must be
instantiated only once at analyzed phase?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
