Github user max3163 commented on a diff in the pull request:
https://github.com/apache/jmeter/pull/321#discussion_r149310069
--- Diff:
src/components/org/apache/jmeter/visualizers/backend/influxdb/InfluxdbBackendListenerClient.java
---
@@ -318,6 +335,13 @@ public void setupTest(BackendListenerContext context)
throws Exception {
}
}
}
+ // Check if more fields are filled ( corresponding to user tag )
+ context.getParameterNamesIterator().forEachRemaining(name->{
+ if ( !defaultArg.containsKey(name)) {
+ TAG_USER += "," +
AbstractInfluxdbMetricsSender.tagToStringValue(name.trim()) + "=" +
AbstractInfluxdbMetricsSender.tagToStringValue(context.getParameter(name).trim());
--- End diff --
I don't think the user will add a lot of tag and it's more clear for me in
this way and it's easier to parse data into influxdb line protocol.
If you want, we can add a '_' at the start of each user tag but for me it's
not mandatory as real property of the InfluxdbBackendListenerClient are default
setted
---
