Dave Brolley <brolley@xxxxxxxxxx> writes:
> [...]
>>> For example,
>>> pminfo -a 20140930.0 -a 201408*.* -a /some/path/archives -a
>>> /another/path/
>>> archive*
>> This is possible, but we may get a long way without requiring
>> extension of the user interface of the pmapi tools (namely, it's
>> undesirable to have to use multiple -a flags [...]
> I wasn't actually proposing multiple explicit contexts within pmapi
> code, but rather extending a single PM_CONTEXT_ARCHIVE to be able to
> handle more than one archive. It would be done in a way that would
> allow existing clients to continue to work without changes.
We're both on the same page here, but:
> Given that clarification, do you still see multiple -a flags as
> undesirable?
The trick is that each -a/-h type flag normally maps to new individual
pcp contexts! pminfo doesn't even accept multiple -a/-h's now.
Consider what the pminfo.c code would look like. Would all the -a
OPTION strings be concatenated somehow and given to a single
rc = pmNewContext(PM_CONTEXT_ARCHIVE,bigString);
PMAPI call?
>>> Streaming live data:
>>> [...]
>> Lots of good ideas in there, but how about we leave this part until
>> later?
> Why wait to start developing ideas?
Nothing wrong with developing ideas, even if premature with respect to
timing of implementation. I just wouldn't want to get bogged down
with detailed discussions, when there's already a specific precursor
piece of work ready to bite down on.
- FChE
|