Adding a procstat_data_t struct to represent shared procstat data.

Stackdriver / collectd

Stackdriver's monitoring agent based on collectd (http://collectd.org).

https://cloud.google.com/monitoring/agent/

Other

51 stars 15 forks source link

Adding a procstat_data_t struct to represent shared procstat data. #122

Closed bmoyles0117 closed 6 years ago

bmoyles0117 commented 6 years ago

I have a feeling we can go with something more concise for the field name in the struct such as "data", so that we can reduce the character count per line easier. Validating the general implementation, I will go back and rename if necessary.

Also logging the pid when debugging to help narrow down specific processes across log lines.

bmoyles0117 commented 6 years ago

PTAL.

bmoyles0117 commented 6 years ago

PTAL.

qingling128 commented 6 years ago

Hi, any progress on this one?

qingling128 commented 6 years ago

Hi, do we have any other improvements to make on this PR? In order to have updated benchmarking results, we would need a new Monitoring Agent release including this fix. Is there an ETA?