| From: | Vik Fearing <vik(at)postgresfriends(dot)org> |
|---|---|
| To: | cary huang <hcary328(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: Add support for AT LOCAL |
| Date: | 2023-09-22 22:54:01 |
| Message-ID: | dbea9877-f39b-1a5b-6e5b-5aad2bbc2821@postgresfriends.org |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 9/22/23 23:46, cary huang wrote:
> 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
>
> I think this feature can be a useful addition in dealing with time zones. I have applied and tried out the patch, The feature works as described and seems promising. The problem with compilation failure was probably reported on CirrusCI when compiled on different platforms. I have run the latest patch on my own Cirrus CI environment and everything checked out fine.
>
> Thank you
Thank you for reviewing!
--
Vik Fearing
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Amit Kapila | 2023-09-23 01:38:37 | Re: Synchronizing slots from primary to standby |
| Previous Message | Thomas Munro | 2023-09-22 22:51:44 | pgsql: Don't trust unvalidated xl_tot_len. |