jmeter-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From max3163 <...@git.apache.org>
Subject [GitHub] jmeter pull request #321: [Influxdb] Allow users to add theirs owns tags
Date Tue, 07 Nov 2017 09:25:33 GMT
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 


---

Mime
View raw message