On Monday 23 November 2009 19:36:10 Steve Polyack wrote:
> Steve Polyack wrote:
> > I've simply added another configuration option for any Storage {}
> > resource in the Director's config.  An admin may set
> > AllowCompression=No for a particular storage resource, causing the
> > director to prevent any GZIP compression options from being sent to
> > the FD when a job is run against that particular storage resource.
> > The default for the directive is Yes/true so that behavior is
> > unchanged for anyone who does not choose to set it.  There may be a
> > better way to strip the compression flags from the fo->opts[] going
> > out to the FD, but this works great and appears readable to me.
> >
> > Any feedback or suggestions would be appreciated.  I'd like to see the
> > code added to a later version of Bacula.
>
> No questions/comments/suggestions/hate mail?

We always appreciate patches :-)

As I mentioned on the bacula-devel list, I was on the road last week.  I did 
see your patch but have not worked through the stacked up emails yet.

Preliminary remark 1: cleaver idea. Preliminary remark 2: I think some users 
might be annoyed if compression is turned off with no warning, which would 
not be too hard to fix.

Regards,

Kern

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Bacula-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bacula-devel

Reply via email to