Re: pgsql: Separate block sampling functions

Michael Paquier <michael.paquier@gmail.com> writes:
> On Fri, May 15, 2015 at 12:03 PM, Simon Riggs <simon@2ndquadrant.com> wrote:
>> On 15 May 2015 at 03:50, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>>> Simon Riggs <simon@2ndQuadrant.com> writes:
>>>> Separate block sampling functions

>>> This patch broke buildfarm member crake.

>> OK, thanks. I missed that amongst the other unrelated failures. Looking now.

> This needs a patch to file_text_array_fdw which I think is available here:
> https://github.com/adunstan/file_text_array_fdw
> Simon, do you mind if I send a pull request?

TBH, I think that this patch itself was a bad idea and should be reverted.
I don't object to changing APIs used by external modules when there's a
good reason to break them, but having looked at this patch all I see is
change for the sake of change.  What new functionality have you introduced?

Or to put it more baldly: it's likely that you've broken quite a large
number of third-party FDWs, not just this one.  A lot of people have
probably copied-and-pasted what was in the contrib FDWs.

            regards, tom lane