Horton
Menu

Thread

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

  1. Add regression tests for pg_stat_progress_copy.tuples_skipped.

  1. tests for pg_stat_progress_copy.tuples_skipped

    jian he <jian.universality@gmail.com> — 2025-01-10T14:50:32Z

    hi.
seems no regress tests for pg_stat_progress_copy.tuples_skipped.

CopyFrom
pgstat_progress_update_param(PROGRESS_COPY_TUPLES_SKIPPED,
                              cstate->num_errors);

there is coverage for it. but in regress test, we didn't really print
out this value (cstate->num_errors)

The attached patch did minor changes on src/test/regress/sql/copy.sql.
so we can check if pgstat_progress_update_param(PROGRESS_COPY_TUPLES_SKIPPED..)
is working as intended or not.

  2. Re: tests for pg_stat_progress_copy.tuples_skipped

    Fujii Masao <masao.fujii@oss.nttdata.com> — 2025-03-03T09:05:23Z

    
On 2025/01/10 23:50, jian he wrote:
> hi.
> seems no regress tests for pg_stat_progress_copy.tuples_skipped.
> 
> CopyFrom
> pgstat_progress_update_param(PROGRESS_COPY_TUPLES_SKIPPED,
>                                cstate->num_errors);
> 
> there is coverage for it. but in regress test, we didn't really print
> out this value (cstate->num_errors)
> 
> The attached patch did minor changes on src/test/regress/sql/copy.sql.
> so we can check if pgstat_progress_update_param(PROGRESS_COPY_TUPLES_SKIPPED..)
> is working as intended or not.

Thanks for the patch!

The patch basically looks good to me.
I’ve made some minor cosmetic adjustments — the updated patch is attached.

Unless there are any objections, I'm thinking to commit it.

Regards,

-- 
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION

  3. Re: tests for pg_stat_progress_copy.tuples_skipped

    jian he <jian.universality@gmail.com> — 2025-03-04T08:57:49Z

    On Mon, Mar 3, 2025 at 5:05 PM Fujii Masao <masao.fujii@oss.nttdata.com> wrote:
>
> Thanks for the patch!
>
> The patch basically looks good to me.
> I’ve made some minor cosmetic adjustments — the updated patch is attached.
>
> Unless there are any objections, I'm thinking to commit it.
>

I checked it again manually.
It looks good to me.

  4. Re: tests for pg_stat_progress_copy.tuples_skipped

    Josef Šimánek <josef.simanek@gmail.com> — 2025-03-04T09:40:04Z

    po 3. 3. 2025 v 10:05 odesílatel Fujii Masao
<masao.fujii@oss.nttdata.com> napsal:
>
>
>
> On 2025/01/10 23:50, jian he wrote:
> > hi.
> > seems no regress tests for pg_stat_progress_copy.tuples_skipped.
> >
> > CopyFrom
> > pgstat_progress_update_param(PROGRESS_COPY_TUPLES_SKIPPED,
> >                                cstate->num_errors);
> >
> > there is coverage for it. but in regress test, we didn't really print
> > out this value (cstate->num_errors)
> >
> > The attached patch did minor changes on src/test/regress/sql/copy.sql.
> > so we can check if pgstat_progress_update_param(PROGRESS_COPY_TUPLES_SKIPPED..)
> > is working as intended or not.
>
> Thanks for the patch!
>
> The patch basically looks good to me.
> I’ve made some minor cosmetic adjustments — the updated patch is attached.
>
> Unless there are any objections, I'm thinking to commit it.

Looks good to me.

> Regards,
>
> --
> Fujii Masao
> Advanced Computing Technology Center
> Research and Development Headquarters
> NTT DATA CORPORATION

  5. Re: tests for pg_stat_progress_copy.tuples_skipped

    Fujii Masao <masao.fujii@oss.nttdata.com> — 2025-03-04T15:02:21Z

    
On 2025/03/04 17:57, jian he wrote:
> On Mon, Mar 3, 2025 at 5:05 PM Fujii Masao <masao.fujii@oss.nttdata.com> wrote:
>>
>> Thanks for the patch!
>>
>> The patch basically looks good to me.
>> I’ve made some minor cosmetic adjustments — the updated patch is attached.
>>
>> Unless there are any objections, I'm thinking to commit it.
>>
> 
> I checked it again manually.
> It looks good to me.

I've pushed the patch. Thanks!

Regards,

-- 
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION