dawidwys commented on pull request #17120: URL: https://github.com/apache/flink/pull/17120#issuecomment-914985789
My personal take is that 2 and 3 are the same. In either case the end goal is to have no transaction. What I find a bit confusing if we use a `TransactionHolder#empty` is the transaction time which does not have a valid value. Whatever approach you choose, we should not serialize the transaction start time. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
