You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hi,
I would like to use react-timeseries-charts/pond on my project (a monitoring infrastructure for a cloud provider) but I stumble on a kind of "dumb" problem related to field specs.
The metrics in this system are all organized using a naming scheme which uses dots. In this case, the dots do not have any relationship to a hierarchy of maps, they are just a way to name the metrics in a logical way. For instance, we have:
os.disk.inodes.count
os.disk.space.free
os.disk.space.min
os.multiprocess.runs.zookeeper.status
os.process.runs.nimbus.status
os.process.runs.redis.status
os.process.runs.storm.status
...
As you can see, this naming scheme prevents me from using pond, because pond interprets all the dots in metric names as field separators.
My question is: would you be willing to accept a patch to pond in which the user could specify the field separator used for field specs in the get() method of an event. '.' would remain the defaut (thus having no impact on existing code), but one could specify something else ('/' for instance in my case). I would then propose changes to react-timeseries-chart to add extra 'fieldSeparator' property to the various charts (again, with '.' default so that there is no impact on existing code) so one could specify the field separator for a given chart.
Regards
Lukas
The text was updated successfully, but these errors were encountered:
laaglu
pushed a commit
to laaglu/pond
that referenced
this issue
Aug 28, 2019
Hi,
I would like to use react-timeseries-charts/pond on my project (a monitoring infrastructure for a cloud provider) but I stumble on a kind of "dumb" problem related to field specs.
The metrics in this system are all organized using a naming scheme which uses dots. In this case, the dots do not have any relationship to a hierarchy of maps, they are just a way to name the metrics in a logical way. For instance, we have:
...
As you can see, this naming scheme prevents me from using pond, because pond interprets all the dots in metric names as field separators.
My question is: would you be willing to accept a patch to pond in which the user could specify the field separator used for field specs in the get() method of an event. '.' would remain the defaut (thus having no impact on existing code), but one could specify something else ('/' for instance in my case). I would then propose changes to react-timeseries-chart to add extra 'fieldSeparator' property to the various charts (again, with '.' default so that there is no impact on existing code) so one could specify the field separator for a given chart.
Regards
Lukas
The text was updated successfully, but these errors were encountered: