Re: TODO : Allow parallel cores to be used by vacuumdb [ WIP ] - Mailing list pgsql-hackers

From Dilip kumar
Subject Re: TODO : Allow parallel cores to be used by vacuumdb [ WIP ]
Date
Msg-id 4205E661176A124FAF891E0A6BA913526633BE5E@szxeml509-mbs.china.huawei.com
Whole thread Raw
In response to Re: TODO : Allow parallel cores to be used by vacuumdb [ WIP ]  (Sawada Masahiko <sawada.mshk@gmail.com>)
Responses Re: TODO : Allow parallel cores to be used by vacuumdb [ WIP ]  (Sawada Masahiko <sawada.mshk@gmail.com>)
List pgsql-hackers
On 01 July 2014 22:17, Sawada Masahiko Wrote,

 
> I have executed latest patch.
> One question is that how to use --jobs option is correct?
> $ vacuumdb  -d postgres  --jobs=30
> 
> I got following error.
> vacuumdb: unrecognized option '--jobs=30'
> Try "vacuumdb --help" for more information.
> 

Thanks for comments, Your usage are correct, but there are some problem in code and I have fixed the same in attached
patch.

Apart from this issue fix currently I am working on jeff's comments for making the code common between
pg_dump/parallel.cand scripts/vac_parallel.c.
 
I found that almost 300 lines of code we can move to common place, but only problem is where to keep the common code.

I am thinking of 
1. keeping a common folder in bin folder --> src/bin/common and move common code which is specific to parallel
operationin src/bin/common/parallel_common.c
 
2. Both vacuum db and pg_dump will compile this file in while generating there executables(in future other executable
likereindex can also use same for parallel functionality)
 

Thoughts ?

Thanks & Regards,
Dilip Kumar




Attachment

pgsql-hackers by date:

Previous
From: Abhijit Menon-Sen
Date:
Subject: Re: 9.5 CF1
Next
From: Abhijit Menon-Sen
Date:
Subject: Re: 9.5 CF1