| From: | Cary Huang <cary(dot)huang(at)highgo(dot)ca> |
|---|---|
| To: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Cc: | Gurjeet Singh <gurjeet(at)singh(dot)im> |
| Subject: | Re: Automatic notification of top transaction IDs |
| Date: | 2021-09-17 20:40:47 |
| Message-ID: | 163191124798.1167.8854857510549120481.pgcf@coridan.postgresql.org |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
The following review has been posted through the commitfest application:
make installcheck-world: tested, passed
Implements feature: tested, passed
Spec compliant: tested, passed
Documentation: tested, passed
Hello
This patch applies fine to master branch and the regression tests are passing.
Regarding the parallel worker case, the AssignTransactionId() function is already handling that and it will error out if IsParallelWorker() is true. In a normal case, this function is only called by the main backend, and the parallel workers will synchronize the transaction ID when they are spawned and they will not call this function anyway.
thank you
Cary Huang
----------------
HighGo Software Canada
www.highgo.ca
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bossart, Nathan | 2021-09-17 21:15:27 | Re: prevent immature WAL streaming |
| Previous Message | Sehrope Sarkuni | 2021-09-17 20:36:57 | Re: Add jsonlog log_destination for JSON server logs |