-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Added MetaDataSet to target_set #1106
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
I have been looking for something like this in collectd for a while. Glad to see some work is being done. I would like to see this get merged. I'll patch this locally and see how it works for my needs and post back here. |
See my comment here #1107 (comment) on my findings. Turns out I needed both PRs not just this one. |
@ymettier - this patch restores support for TypeInstance in ts_create()
|
This is a very nice feature, fairly crucial for deployments to many machines with various identifying characteristics (ie.g. datacenter, rack, cluster, etc.). Without additional metadata, slicing and dicing metrics gets messy and difficult. |
👍 this feature will be very useful for us. This PR is 6 months old, is there any reason for not being merged? |
@yogeswaran : no reason as far as I know. |
if (orig == NULL) | ||
return (0); | ||
|
||
if(NULL == *dest) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
*dest = = NULL please, and space after if.
@ymettier This URL:#1107 I have a doubt:The following few write where? tsdb_prefix Will prefix the OpenTSDB (also prefix tsdb_id if defined) |
Hello,
This patch allows to set metadata with filters and
target_set
target.It adds a new keyword
MetaDataSet
.Example :
Note : this patch adds some functions to the daemon/meta_data.c in order to clone_and_merge meta data. I prefered to share my new functions instead of implementing them in target_set.c.
Regards,
Yves