Re: pmrep: add support for swapping columns and rows in stdout output

Hi,

On 2016-11-04 18:14, Marko Myllynen wrote:
> On 2016-11-03 03:05, Mark Goodwin wrote:
>>
>> I love it! Testing it out, could we consider adding an argument to the
>> -X flag to label the instance name column, and maybe some way to specify
>> it's width (independently of the width for the data columns)?
> 
> Yeah, I noticed that it felt a bit of compromise to use the timestamp
> width as the default width, perhaps indeed we could add some sort of
> method to specify width for that, either a separate "field" or something
> like disk.dev.util,'%util',,,6:10, what do you think?

Hmm, specifying this on per-metric basis of course doesn't make much
sense, not sure what I was thinking. Perhaps we could have -X to take an
argument which could be < 0 turn turn this feature off (the default), 0
use timestamp width, and > 0 to explicitly specify the width for the
instance column.

What do you think? Ok, defining something to 0 to enable it is not the
most intuitive way but I'd like to avoid a separate configuration knob
for the instance width, if possible.

>> Also, when metrics with different instance domains are specified, maybe
>> we could have a 'filler' such as '-' or '.' or something for the columns
>> which don't make sense on the line for each instance. This would make it
>> easier to parse and post-process because we'd have a fixed number of
>> columns to pick out in the report.
> 
> Would setting the column delimiter help, e.g., "pmrep -l : ..."? But
> yes, printing "-" or such wouldn't be unreasonable either.

Testing this with "-" I think it's aesthetically ok to be used as the
default and not even worth a separate config option (so probably will
add a constant and use it).

Thanks,

-- 
Marko Myllynen

-=-=-=-=-=-=-=-=-=-=-=-
pcp mailing list
[email protected]
https://groups.io/g/pcp/messages
-=-=-
Groups.io Links:

You receive all messages sent to this group.

View/Reply Online (#14655): https://groups.io/g/pcp/message/14655
View All Messages In Topic (4): https://groups.io/g/pcp/topic/3037310
Mute This Topic: https://groups.io/mt/3037310?uid=174580
New Topic: https://groups.io/g/pcp/post

Change Your Subscription: https://groups.io/g/pcp/editsub?uid=174580

Group Home: https://groups.io/g/pcp
Contact Group Owner: [email protected]

Terms of Service: https://groups.io/static/tos

Unsubscribe: https://groups.io/g/pcp/leave/354243/563757577/xyzzy
-=-=-=-=-=-=-=-=-=-=-=-