Re: [HACKERS] Block level parallel vacuum

On Thu, Oct 17, 2019 at 3:18 PM Mahendra Singh <mahi6run(at)gmail(dot)com> wrote:
>
> Hi
> I applied all 3 patches and ran regression test. I was getting one regression failure.
>
>> diff -U3 /home/mahendra/postgres_base_rp/postgres/src/test/regress/expected/vacuum.out /home/mahendra/postgres_base_rp/postgres/src/test/regress/results/vacuum.out
>> --- /home/mahendra/postgres_base_rp/postgres/src/test/regress/expected/vacuum.out 2019-10-17 10:01:58.138863802 +0530
>> +++ /home/mahendra/postgres_base_rp/postgres/src/test/regress/results/vacuum.out 2019-10-17 11:41:20.930699926 +0530
>> @@ -105,7 +105,7 @@
>> CREATE TEMPORARY TABLE tmp (a int PRIMARY KEY);
>> CREATE INDEX tmp_idx1 ON tmp (a);
>> VACUUM (PARALLEL 1) tmp; -- error, cannot parallel vacuum temporary tables
>> -WARNING: skipping "tmp" --- cannot parallel vacuum temporary tables
>> +WARNING: skipping vacuum on "tmp" --- cannot vacuum temporary tables in parallel
>> -- INDEX_CLEANUP option
>> CREATE TABLE no_index_cleanup (i INT PRIMARY KEY, t TEXT);
>> -- Use uncompressed data stored in toast.
>
>
> It look likes that you changed warning message for temp table, but haven't updated expected out file.
>

Thank you!
I forgot to change the expected file. I'll fix it in the next version patch.

Regards,

--
Masahiko Sawada