Hi Nikhil,
Any progress on the issues discussed in the last couple of messages?
That is:
1) removing of the sleep() from tests
2) changes to systable_getnext() wrt. TransactionIdIsInProgress()
3) adding asserts / checks to codepaths not going through systable_*
4) (not) adding this as a per-plugin option
5) handling cases where the downstream does not have 2PC enabled
I guess it'd be good an updated patch or further discussion before
continuing the review efforts.
regards
--
Tomas Vondra http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services