From: | "movead(dot)li(at)highgo(dot)ca" <movead(dot)li(at)highgo(dot)ca> |
---|---|
To: | michael <michael(at)paquier(dot)xyz> |
Cc: | petr <petr(at)2ndquadrant(dot)com>, simon <simon(at)2ndquadrant(dot)com>, "Madan Kumar" <madankumar1993(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, craig <craig(at)2ndquadrant(dot)com>, "Alvaro Herrera" <alvherre(at)2ndquadrant(dot)com>, andres <andres(at)anarazel(dot)de>, soumyadeep2007 <soumyadeep2007(at)gmail(dot)com> |
Subject: | Re: A patch for get origin from commit_ts. |
Date: | 2020-07-08 01:31:24 |
Message-ID: | 2020070809312199165531@highgo.ca |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
>Cool, thanks. I have gone through your patch in details, and updated
>it as the attached. Here are some comments.
>'8000' as OID for the new function was not really random, so to be
>fair with the other patches, I picked up the first random value
>unused_oids has given me instead.
>
>There were some indentation issues, and pgindent got that fixed.
>I think that it is better to use "replication origin" in the docs
>instead of just origin. I have kept "origin" in the functions for
>now as that sounded cleaner to me, but we may consider using something
>like "reporigin" as well as attribute name.
>
>The tests could just use tstzrange() to make sure that the timestamps
>have valid values, so I have switched to that, and did not resist to
>do the same in the existing tests.
>
>+-- Test when it can not find the transaction
>+SELECT * FROM pg_xact_commit_timestamp_origin((:'txid_set_origin'::text::int +
>10)::text::xid) x;
>This test could become unstable, particularly if it gets used in a
>parallel environment, so I have removed it. Perhaps I am just
>over-pessimistic here though..
>
>As a side note, I think that we could just remove the alternate output
>of commit_ts/, as it does not really get used because of the
>NO_INSTALLCHECK present in the module's Makefile. That would be the
>job of a different patch, so I have updated it accordingly. Glad to
>see that you did not forget to adapt it in your own patch.
>
>(The change in catversion.h is a self-reminder...)
Thanks for all of that, so many details I still need to pay attention when
submit a patch.
Regards,
Highgo Software (Canada/China/Pakistan)
URL : www.highgo.ca
EMAIL: mailto:movead(dot)li(at)highgo(dot)ca
From | Date | Subject | |
---|---|---|---|
Next Message | Andreas Karlsson | 2020-07-08 01:32:11 | Re: [HACKERS] Look-behind regular expressions |
Previous Message | movead.li@highgo.ca | 2020-07-08 01:21:08 | Re: pg_resetwal --next-transaction-id may cause database failed to restart. |