Horton

Thread

Commits

  1. postgres_fdw: Remove redundancy in postgresAcquireSampleRowsFunc().

  1. postgres_fdw: Minor improvement to postgresAcquireSampleRowsFunc

    Etsuro Fujita <etsuro.fujita@gmail.com> — 2019-06-28T09:38:51Z

    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.

Best regards,
Etsuro Fujita

  2. Re: postgres_fdw: Minor improvement to postgresAcquireSampleRowsFunc

    Julien Rouhaud <rjuju123@gmail.com> — 2019-06-28T09:54:34Z

    On Fri, Jun 28, 2019 at 11:39 AM Etsuro Fujita <etsuro.fujita@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.

  3. Re: postgres_fdw: Minor improvement to postgresAcquireSampleRowsFunc

    Etsuro Fujita <etsuro.fujita@gmail.com> — 2019-06-28T10:15:56Z

    Hi Julien,

On Fri, Jun 28, 2019 at 6:54 PM Julien Rouhaud <rjuju123@gmail.com> wrote:
> On Fri, Jun 28, 2019 at 11:39 AM Etsuro Fujita <etsuro.fujita@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!

Best regards,
Etsuro Fujita

  4. Re: postgres_fdw: Minor improvement to postgresAcquireSampleRowsFunc

    Etsuro Fujita <etsuro.fujita@gmail.com> — 2019-07-03T09:01:36Z

    On Fri, Jun 28, 2019 at 7:15 PM Etsuro Fujita <etsuro.fujita@gmail.com> wrote:
> On Fri, Jun 28, 2019 at 6:54 PM Julien Rouhaud <rjuju123@gmail.com> wrote:
> > On Fri, Jun 28, 2019 at 11:39 AM Etsuro Fujita <etsuro.fujita@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