Michael Paquier <michael.paqu...@gmail.com> writes:
> On Thu, Aug 17, 2017 at 6:01 AM, Tom Lane <t...@sss.pgh.pa.us> wrote:
>> I'm not really qualified to review the Python coding
>> style, but I did fix a typo in a comment.
> No pythonist here, but a large confusing "if" condition without any
> comments is better if split up and explained with comments if that can
> help in clarifying what the code is doing in any language, so thanks
> for keeping the code intact.
Certainly agreed on splitting up the logic into multiple statements.
I just meant that I don't know enough Python to know if there are
better ways to do these tests. (It probably doesn't matter, since
performance of this script is not an issue, and it's not likely to
undergo a lot of further development either.)
regards, tom lane
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
