On Thursday 09 August 2007 17:30, Marc Schiffbauer wrote:
> Hi,
>
> while reading the docs about job concurrency and such I found some
> conflicting statements which should be fixed IMO to make it more
> consistent.
>
> 1. What to do to enable concurrent jobs
> ######################################
> http://bacula.org/dev-manual/Tips_Suggestions.html#SECTION00381700000000000
>00000
>
> I guess I am not the only one who has to read those paragraphs
> several times to understand it. I think they can be updated a bit to
> make it more readable and better understandable.

Oh, there are probably a number of such places and other people such as 
yourself, except they didn't take the time and effort to send in a patch 
file :-)

>
>
> "The downside is that it can be a bit tricky to set it up for the
> first time as you need to set the concurrency in at least five
> different places."
>
> I consider this wrong: Its *at least* 4 places in *one* config file:
> bacula-dir.conf: Director{}, Job{}, Client{}, Storage{}
> And telling here that its only one file you have to modify would
> help to understand more quickly.

I think the original SD conf file had Maximum Concurrent Jobs = 1, which is 
why I wrote at least five.  As you noted this is no longer true.

>
> later:
> "Below is a super stripped down bacula-dir.conf file showing you the
> four places where the the file has been modified..."
>
> So fixing this to read
> "The downside is that it can be a bit tricky to
> set it up for the first time as you need to set the concurrency in
> at least four different places in the Director's conf file
> (bacula-dir.conf). "
> makes this sentence quiet needless because it sounds much like what
> is written later on.
>
> So attached is a patch to docs/manual/tips.tex to "fix" this.

I have applied your patch to the current code base, and since I was reading it 
over, I tweaked it a bit more as well ...

Thanks for the patch.  I wish more people would do the same :-)

>
> -Marc

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
Bacula-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bacula-devel

Reply via email to