rkhachatryan commented on pull request #15200: URL: https://github.com/apache/flink/pull/15200#issuecomment-856970264
Thanks for the thorough reviews @curcur and @Myasuka ! I addressed most of the comments, PTAL. Regarding the state mutation via iterators, @curcur : - both RocksDB and Heap allow to update the state via iterators - multiple updates via different iterators may or may not fail; but if they succeed then the changes should be reflected by logging iterators in the correct order (assuming single-threaded environment) -- 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]
