Re: Improvements and additions to COPY progress reporting

From: Josef Šimánek <josef(dot)simanek(at)gmail(dot)com>
To: 0010203112132233 <boekewurm(at)gmail(dot)com>
Cc: Matthias van de Meent <boekewurm+postgres(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Tomas Vondra <tomas(dot)vondra(at)enterprisedb(dot)com>
Subject: Re: Improvements and additions to COPY progress reporting
Date: 2021-02-09 11:53:44
Message-ID: CAFp7QwotqR+aRP34nGiHRHZpEYDQQ1yZhzNDuE6MUQ=rLgNYbA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

út 9. 2. 2021 v 12:51 odesílatel 0010203112132233 <boekewurm(at)gmail(dot)com> napsal:
>
> On Tue, 9 Feb 2021 at 09:32, Josef Šimánek <josef(dot)simanek(at)gmail(dot)com> wrote:
> >
> > po 8. 2. 2021 v 19:35 odesílatel Matthias van de Meent
> > <boekewurm+postgres(at)gmail(dot)com> napsal:
> > > Lastly, 0005 adds 'io_target' to the reported information, that is,
> > > FILE, PROGRAM, STDIO or CALLBACK. Although this can relatively easily
> > > be determined based on the commands in pg_stat_activity, it is
> > > reasonably something that a user would want to query on, as the
> > > origin/target of COPY has security and performance implications,
> > > whereas other options (e.g. format) are less interesting for clients
> > > that are not executing that specific COPY command.
> >
> > I took a little deeper look and I'm not sure if I understand FILE and
> > STDIO. I have finally tried to finalize some initial regress testing
> > of COPY command progress using triggers. I have attached the initial
> > patch applicable to your changes. As you can see COPY FROM STDIN is
> > reported as FILE. That's probably expected, but it is a little
> > confusing for me since STDIN and STDIO sound similar. What is the
> > purpose of STDIO? When is the COPY command reported with io_target of
> > STDIO?
>
> I checked for the type of the copy_src before it was correctly set,
> therefore only reporting FILE type, but this will be fixed shortly in
> v3.

OK, would you mind to integrate my regression test initial patch as
well in v3 or should I submit it later in a separate way?

> Matthias

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Kapila 2021-02-09 11:57:12 Re: repeated decoding of prepared transactions
Previous Message 0010203112132233 2021-02-09 11:51:20 Re: Improvements and additions to COPY progress reporting