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.