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
