Re: [PR] [feat][txn] Support for idempotent commit and abort. [pulsar]

["merlimat (via GitHub)" <gi...@apache.org>]

merlimat commented on PR #19662:
URL: https://github.com/apache/pulsar/pull/19662#issuecomment-6357057269

   Hi @thetumbled, thanks for this, and sorry it sat so long. Like #20356, it implements PIP-255 (#19744), which was never accepted. The open question was how much TC memory it takes to keep ended transactions. The PR also still has an open change request. On master, the V5 (PIP-473) transaction coordinator already treats a repeated commit/abort of an ended transaction as a retry, within its GC retention window. As part of the open-PR backlog cleanup (https://lists.apache.org/thread/cgy7w0qgnq8s7j0ykyzgrn3hj4nb67wf) I'm closing this. #19629 stays open for the classic coordinator. If you'd like to pursue it there, restarting the PIP-255 discussion on dev@ is the way forward.


-- 
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: commits-unsubscribe@pulsar.apache.org

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