From: | Dong Wook Lee <sh95119(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, daniel(at)yesql(dot)se |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: old_snapshot: add test for coverage |
Date: | 2022-12-06 15:00:26 |
Message-ID: | CAAcByaKHb3AwCvL=Nz2S6+fC2pnuwhq98-tO89mt6Bpvpzv34w@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Aug 8, 2022 at 2:37 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
> Dong Wook Lee <sh95119(at)gmail(dot)com> writes:
> > I wrote a test of the old_snapshot extension for coverage.
>
> Hmm, does this really provide any meaningful coverage? The test
> sure looks like it's not doing much.
Previously written tests were simply test codes to increase coverage.
Therefore, we can make a better test.
I'll think about it by this week. If this work exceeds my ability, I
will let you know by reply.
It's okay that the issue should be closed unless I write a meaningful test.
---
Regards,
DongWook Lee.
From | Date | Subject | |
---|---|---|---|
Next Message | Alexander Korotkov | 2022-12-06 15:00:54 | Re: Allow placeholders in ALTER ROLE w/o superuser |
Previous Message | Tom Lane | 2022-12-06 14:52:51 | Re: Missing MaterialPath support in reparameterize_path_by_child |