From: | Dilip kumar <dilip(dot)kumar(at)huawei(dot)com> |
---|---|
To: | Jeff Janes <jeff(dot)janes(at)gmail(dot)com> |
Cc: | Sawada Masahiko <sawada(dot)mshk(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Jan Lentfer <Jan(dot)Lentfer(at)web(dot)de>, "Euler Taveira" <euler(at)timbira(dot)com(dot)br> |
Subject: | Re: TODO : Allow parallel cores to be used by vacuumdb [ WIP ] |
Date: | 2014-06-27 11:10:00 |
Message-ID: | 4205E661176A124FAF891E0A6BA9135266336F6F@szxeml509-mbs.china.huawei.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 27 June 2014 02:57, Jeff Wrote,
> Based on that, I find most importantly that it doesn't seem to
> correctly vacuum tables which have upper case letters in the name,
> because it does not quote the table names when they need quotes.
Thanks for your comments....
There are two problem
First -> When doing the vacuum of complete database that time if any table with upper case letter, it was giving error
--FIXED by adding quotes for table name
Second -> When user pass the table using -t option, and if it has uppercase letter
--This is the existing problem (without parallel implementation),
One solution to this is, always add Quote to the relation name passed by user, but this can break existing applications for some users..
> Of course that needs to be fixed, but taking it as it is, the resulting
> error message to the console is just:
FIXED
>
> Which is not very informative. I get the same error if I do a "pg_ctl
> shutdown -mi" while running the parallel vacuumdb. Without the -j
> option it produces a more descriptive error message "FATAL:
> terminating connection due to administrator command", so something
> about the new feature suppresses the informative error messages.
>
> I get some compiler warnings with the new patch:
>
> vac_parallel.c: In function 'parallel_msg_master':
> vac_parallel.c:147: warning: function might be possible candidate for
> 'gnu_printf' format attribute
> vac_parallel.c:147: warning: function might be possible candidate for
> 'gnu_printf' format attribute
> vac_parallel.c: In function 'exit_horribly':
> vac_parallel.c:1071: warning: 'noreturn' function does return
FIXED
> In the usage message, the string has a tab embedded within it
> (immediately before "use") that should be converted to literal spaces,
> otherwise the output of --help gets misaligned:
>
> printf(_(" -j, --jobs=NUM use this many parallel
> jobs to vacuum\n"));
>
FIXED
Updated patch is attached in the mail..
Thanks & Regards,
Dilip Kumar
Attachment | Content-Type | Size |
---|---|---|
vacuumdb_parallel_v5.patch | application/octet-stream | 49.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Fujii Masao | 2014-06-27 11:10:35 | Re: pg_receivexlog add synchronous mode |
Previous Message | Fujii Masao | 2014-06-27 10:42:01 | Re: pg_resetxlog to clear backup start/end locations. |