On Fri, Jun 28, 2019 at 7:15 PM Etsuro Fujita <etsuro.fuj...@gmail.com> wrote: > On Fri, Jun 28, 2019 at 6:54 PM Julien Rouhaud <rjuju...@gmail.com> wrote: > > On Fri, Jun 28, 2019 at 11:39 AM Etsuro Fujita <etsuro.fuj...@gmail.com> > > 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 for removing that redundancy. > > > > It all looks good to me! I marked it as ready for committer. > > Cool! I'll commit the patch if there are no objections. Thanks for > reviewing!
Pushed. Best regards, Etsuro Fujita