HeartSaVioR edited a comment on pull request #26935: URL: https://github.com/apache/spark/pull/26935#issuecomment-716517527
I don't only intend to restrict the API usage on read-only path. The purpose of separation is to allow read-only implementation to be differentiated. e.g. In HDFS state store provider, read-only state only copies the specific version of state from loaded map, which doesn't involve any file operations, whereas read-write state additionally creates a new temporary file for delta, and has to handle the cleanup with temp file (either rename or delete). For sure, this can be applied to the other possible providers as well, which opens possibility to optimize for the read-only implementation. ---------------------------------------------------------------- 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
