HeartSaVioR commented on pull request #32796: URL: https://github.com/apache/spark/pull/32796#issuecomment-855552990
Also let's not just avoid it in existing codebase, but prevent it as well via revisiting the contract of interface. * The contract of `put` should include the requirement that value should be also non-null. We could also add `require` at the first line of `put()` to the HDFS state store implementation. That can be covered by test. -- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
