Postgres Pro
Downloads
Home   >   mailing lists

Re: patch: option --if-exists for pg_dump - Mailing list pgsql-hackers

From Jeevan Chalke
Subject Re: patch: option --if-exists for pg_dump
Date
Msg-id CAM2+6=WMsD+55+prCEtb5XsHA=FJktixmXnm-FETjnzsxBUvYQ@mail.gmail.com
Whole thread Raw
In response to Re: patch: option --if-exists for pg_dump  (Pavel Stehule <pavel.stehule@gmail.com>)
Responses Re: patch: option --if-exists for pg_dump  (Pavel Stehule <pavel.stehule@gmail.com>)
Re: patch: option --if-exists for pg_dump  (Alvaro Herrera <alvherre@2ndquadrant.com>)
List pgsql-hackers
Tree view
Hi Pavel,

Now patch looks good to me and I think it is in good shape to pass it on to
the committer as well.

However, I have
- Tweaked few comments
- Removed white-space errors
- Fixed typos
- Fixed indentation

Attached patch with my changes. However entire design and code logic is
untouched.

Please have a quick look and pass it on to committor if you have no issues OR
ask me to assign it to the committor, NO issues either way.

Feel free to reject my changes if you didn't like them.

Thanks
--
Jeevan B Chalke
Principal Software Engineer, Product Development
EnterpriseDB Corporation
The Enterprise PostgreSQL Company

Attachment

pgsql-hackers by date:

Previous
From: Fujii Masao
Date:
Subject: Re: [PATCH] Support for pg_stat_archiver view
Next
From: "MauMau"
Date:
Subject: Re: [bug fix] postgres.exe fails to start on Windows Server 2012 due to ASLR