HeartSaVioR commented on pull request #28707:
URL: https://github.com/apache/spark/pull/28707#issuecomment-638644781


   > What are other stateful operations that use unsafe row? I think we can 
apply the check everywhere.
   
   State store itself stores UnsafeRow, hence it applies to everywhere in 
stateful operations. I'd propose to do it like 
https://github.com/apache/spark/pull/28707#issuecomment-637926400 instead of 
fixing everywhere.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to