On Wed, Nov 13, 2019 at 02:48:01PM -0500, Andrew Dunstan wrote:
On 11/13/19 8:08 AM, Bruce Momjian wrote:
Also, why was this patch posted without any discussion of these issues?
Shouldn't we ideally discuss the API first?
This is a very tiny patch. I don't think it's unusual to post such
things without prior discussion. I would agree with your point if it
were thousands of lines instead of 20 or so lines of core code.
Not sure that's really true. I think patches should provide some basic
explanation why the feature is desirable, no matter the number of lines.
Also, there were vague references to issues with passing parameters to
archive_command. A link to details, past discussion, or brief
explanation would be nice.
regards
--
Tomas Vondra http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
