chieryw / distcc

Automatically exported from code.google.com/p/distcc
GNU General Public License v2.0
0 stars 0 forks source link

"all servers busy" dcc_lock_pause is hardcoded at 1 sec, could be tunable #102

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
distcc 3.2rc1 SVN r755

The hardcoded 1 second dcc_lock_pause() when all servers are busy may pose an 
unnecessary throughput restriction for some configurations.

Attached patch makes the pause time tunable via a new env var 
DISTCC_PAUSE_TIME_MSEC.  The default remains set for 1 second.

Original issue reported on code.google.com by kamal@whence.com on 4 Apr 2012 at 5:13

Attachments:

GoogleCodeExporter commented 9 years ago
The new environment variable should be documented in the "ENVIRONMENT 
VARIABLES" section of the distcc man page.
Could you please update the patch to also update man/distcc.1 to document the 
new variable?

Also put spaces around the '*' in 'pause_time_ms*1000'.

Original comment by fergus.h...@gmail.com on 4 Apr 2012 at 5:33

GoogleCodeExporter commented 9 years ago
> The new environment variable should be documented

> Also put spaces around the '*' in 'pause_time_ms*1000'.

Done.  New patch attached.

Original comment by kamal@whence.com on 4 Apr 2012 at 5:56

Attachments:

GoogleCodeExporter commented 9 years ago
Looks good.

Original comment by fergus.h...@gmail.com on 4 Apr 2012 at 6:53

GoogleCodeExporter commented 9 years ago

Original comment by kamal@whence.com on 4 Apr 2012 at 7:05

GoogleCodeExporter commented 9 years ago
Fix applied as SVN r760.

Original comment by kamal@whence.com on 4 Apr 2012 at 8:23