From: | Michael Paquier <michael(at)paquier(dot)xyz> |
---|---|
To: | Dong Wook Lee <sh95119(at)gmail(dot)com> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Add tap test for --extra-float-digits option |
Date: | 2020-06-13 00:44:07 |
Message-ID: | 20200613004407.GJ3362@paquier.xyz |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Jun 12, 2020 at 06:15:36PM +0900, Dong Wook Lee wrote:
> Oh, now I understand. and I added a test of --row-per-insert option.
That's more of an habit to look around, find similar patterns and the
check if these are covered.
I have applied your patch, and you may want to be careful about a
couple of things:
- Please avoid top-posting on the mailing lists:
https://en.wikipedia.org/wiki/Posting_style#Top-posting
Top-posting breaks the logic of a thread.
- Your patch format is good. When sending a new version of the patch,
it may be better to send things as a complete diff on the master
branch (or the branch you are working on), instead of just sending one
patch that applies on top of something you sent previously. Here for
example your patch 0002 applied on top of 0001 that was sent at the
top of the thread. We have also guidelines about patch submission:
https://wiki.postgresql.org/wiki/Submitting_a_Patch
Thanks!
--
Michael
From | Date | Subject | |
---|---|---|---|
Next Message | Chapman Flack | 2020-06-13 02:05:16 | Re: what can go in root.crt ? |
Previous Message | David Rowley | 2020-06-13 00:36:36 | Re: Speedup usages of pg_*toa() functions |