bucardo/check_postgres — comment on commit c0bd7ef322105ca6ff6dd369c884ec49ac936c4b
File:  line 0
By xzilla:

This struck me as odd, and looking at Peter's bug report, I think the comment is the thing that is wrong, not the check. If you are looking for long running transactions, I don't think you really care if the transaction is IDLE or not; you just care the the transaction is still ongoing. AFAICT, with the above change, a long running transaction that runs multiple queries running inside it, with periods of inactivity, would lead to a flapping alert. So, I think you should revert this change, and maybe update the comment.