Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/19940
1. Yes, to ensure that there are no problems, variables' value should not
be changed. I agree that we can find a better API name, to enforce this: what
about `addImmutableStateIfNotExists`? Or do you have any suggestion?
2. Beacause as @viirya pointed out
[here](https://github.com/apache/spark/pull/19940#issuecomment-350724795), they
are not initialized in the declaration, but in the `init` method of the
generated class, thus it cannot be made final unfortunately.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
