On Fri, Jun 28, 2019 at 7:15 PM Etsuro Fujita wrote:
> On Fri, Jun 28, 2019 at 6:54 PM Julien Rouhaud wrote:
> > On Fri, Jun 28, 2019 at 11:39 AM Etsuro Fujita
> > wrote:
> > > In postgresAcquireSampleRowsFunc, we 1) determine the fetch size and
> > > then 2) construct the fetch command in
Hi Julien,
On Fri, Jun 28, 2019 at 6:54 PM Julien Rouhaud wrote:
> On Fri, Jun 28, 2019 at 11:39 AM Etsuro Fujita
> wrote:
> > In postgresAcquireSampleRowsFunc, we 1) determine the fetch size and
> > then 2) construct the fetch command in each iteration of fetching some
> > rows from the
On Fri, Jun 28, 2019 at 11:39 AM Etsuro Fujita wrote:
>
> In postgresAcquireSampleRowsFunc, we 1) determine the fetch size and
> then 2) construct the fetch command in each iteration of fetching some
> rows from the remote, but that would be totally redundant.
Indeed.
> Attached
> is a patch
Hi,
In postgresAcquireSampleRowsFunc, we 1) determine the fetch size and
then 2) construct the fetch command in each iteration of fetching some
rows from the remote, but that would be totally redundant. Attached
is a patch for removing that redundancy.
I'll add this to the upcoming commitfest.
