Re: [Puppet-dev] Not quite understanding 'sync'

2011-03-07 Thread Luke Kanies
On Mar 7, 2011, at 1:05 PM, Trevor Vaughan wrote: > So, I have this working, but I'm not quite sure why I needed to do it this > way. > > I took a look at the exec:returns property and the associated 'sync' define. > > I was getting an error about returning an invalid event and I wasn't > sure

Re: [Puppet-dev] Not quite understanding 'sync'

2011-03-07 Thread Daniel Pittman
On Mon, Mar 7, 2011 at 14:24, Trevor Vaughan wrote: > Erg? I thought we were on a push to try and avoid reading the contents > of unknown size files into memory where possible? Well, it isn't a great idea; you are not obliged to do anything in prefetch: if your file is of unbounded or unreasonab

Re: [Puppet-dev] Not quite understanding 'sync'

2011-03-07 Thread Trevor Vaughan
Erg? I thought we were on a push to try and avoid reading the contents of unknown size files into memory where possible? Also, does this work with 0.24? Still waiting for those last few bumps to be ironed out of 2.6. Thanks, Trevor On Mon, Mar 7, 2011 at 5:21 PM, Daniel Pittman wrote: > On Mo

Re: [Puppet-dev] Not quite understanding 'sync'

2011-03-07 Thread Daniel Pittman
On Mon, Mar 7, 2011 at 14:14, Trevor Vaughan wrote: > I *think* I get what you're saying, but how would you translate this > to something that needs to see if an entry in a file is synchronized? So, let me be clear: *if* you can avoid replacing sync, that would be good. If you can't, and it is

Re: [Puppet-dev] Not quite understanding 'sync'

2011-03-07 Thread Trevor Vaughan
Daniel, I *think* I get what you're saying, but how would you translate this to something that needs to see if an entry in a file is synchronized? Yes, I know that I could possibly use parsed file, but it didn't work out of the box the way I needed and it was faster to start from scratch than try

Re: [Puppet-dev] Not quite understanding 'sync'

2011-03-07 Thread Daniel Pittman
On Mon, Mar 7, 2011 at 13:05, Trevor Vaughan wrote: > So, I have this working, but I'm not quite sure why I needed to do it this > way. > I took a look at the exec:returns property and the associated 'sync' define. This is, for reference, a bad model to build new types off; like file, it predat

Re: [Puppet-dev] Not quite understanding 'sync'

2011-03-07 Thread Trevor Vaughan
Fancy. I tried searching for several things but came up empty. This is probably something that should go on the dev wiki (if anyone is of a mind to do so). Thanks! Trevor On Mon, Mar 7, 2011 at 4:51 PM, Stefan Schulte wrote: > On Mon, Mar 07, 2011 at 04:05:18PM -0500, Trevor Vaughan wrote: >>

Re: [Puppet-dev] Not quite understanding 'sync'

2011-03-07 Thread Stefan Schulte
On Mon, Mar 07, 2011 at 04:05:18PM -0500, Trevor Vaughan wrote: > So, I have this working, but I'm not quite sure why I needed to do it this > way. > > I took a look at the exec:returns property and the associated 'sync' define. > > I was getting an error about returning an invalid event and I w

[Puppet-dev] Not quite understanding 'sync'

2011-03-07 Thread Trevor Vaughan
So, I have this working, but I'm not quite sure why I needed to do it this way. I took a look at the exec:returns property and the associated 'sync' define. I was getting an error about returning an invalid event and I wasn't sure what type of event I was supposed to return in sync. I noticed th