Robert Haas <robertmhaas@gmail.com> writes:
> On Mon, Sep 14, 2020 at 6:26 AM Ashutosh Sharma <ashu.coek88@gmail.com> wrote:
>> Thanks for reporting. I'm able to reproduce the issue by creating some
>> delay just before "-- now create an unused line pointer" and use the
>> delay to start a new session either with repeatable read or
>> serializable transaction isolation level and run some query on the
>> test table. To fix this, as you suggested I've converted the test
>> table to the temp table. Attached is the patch with the changes.
>> Please have a look and let me know about any concerns.
> Tom, do you have any concerns about this fix?
It seems OK as far as it goes. Two thoughts:
* Do we need a comment in the test pointing out that the table must be
temp to ensure that we get stable vacuum results? Or will the commit
log message be enough documentation?
* Should any of the other tables in the test be converted to temp?
I see that the other test cases are kluging around related issues
by dint of never committing their tables at all. It's not clear
to me how badly those test cases have been distorted by that, or
whether it means they're testing less-than-typical situations.
Anyway, if you're satisfied with leaving the other cases as-is,
I have no objections.
regards, tom lane
