You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@jmeter.apache.org by bu...@apache.org on 2016/01/12 14:56:56 UTC

[Bug 58841] New: Add average metric in BackendListener and change samplersList by a regex match

https://bz.apache.org/bugzilla/show_bug.cgi?id=58841

            Bug ID: 58841
           Summary: Add average metric in BackendListener and change
                    samplersList by a regex match
           Product: JMeter
           Version: 2.13
          Hardware: All
                OS: All
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: Main
          Assignee: issues@jmeter.apache.org
          Reporter: maxime.chassagneux@gmail.com

Created attachment 33427
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=33427&action=edit
Add avg metric and regex for samplersList.

Hi, 

I use the backendlistner to send metric on InfluxDB. So far, I didn't
understand how i could find the average response time from current metric. Only
min max, but no way to calculate average. 

So, I made this patch to send new metric to influxdb. 

I've made also a modification, to change the behaviour of the samplersList. I
think it's better and more power to use a regex to select which sampler you
want.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 58841] Add average metric in BackendListener and change samplersList by a regex match

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=58841

--- Comment #2 from maxime.chassagneux@gmail.com ---
I'm working on a PTO team (Permit To Operate) and made dozen of test by week.
All projects need to known the average response time of each transaction.
I use influxdb / grafana for create report, and I really need to know the
average response time of each transaction...

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 58841] Add average metric in BackendListener and change samplersList by a regex match

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=58841

--- Comment #6 from Antonio Gomes Rodrigues <ra...@gmail.com> ---
Hi,

I think to have the possibility to use a list or regexp to select which sampler
we want to have in InfluxDB is very great

Antonio

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 58841] BackendListener : Allow samplersList to be a regex

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=58841

Philippe Mouawad <p....@ubik-ingenierie.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |DUPLICATE

--- Comment #8 from Philippe Mouawad <p....@ubik-ingenierie.com> ---


*** This bug has been marked as a duplicate of bug 59099 ***

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 58841] Add average metric in BackendListener and change samplersList by a regex match

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=58841

--- Comment #4 from maxime.chassagneux@gmail.com ---
I also use percentiles, but all the datas feedback system we use (like
introscope etc..) only use avg metric. So in term of comparaison between
acceptance environnement and production I only use avg metric. 

Furthermore, in term of communication report, for most of users, the avg metric
is more significant than percentile.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 58841] Add average metric in BackendListener and change samplersList by a regex match

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=58841

--- Comment #5 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Hello,
Avg is a misleading and meaningless metric so I am not keen on having it by
default.

If team agrees, a possible option would be to enable it by an additional
property. 
Can you open discussion on dev mailing list, and if ok for others and if you
provide a patch for this , we could integrated it.

Regards
Philippe

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 58841] BackendListener : Allow samplersList to be a regex

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=58841

Philippe Mouawad <p....@ubik-ingenierie.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Add average metric in       |BackendListener : Allow
                   |BackendListener and change  |samplersList to be a regex
                   |samplersList by a regex     |
                   |match                       |

--- Comment #7 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Renaming issue to only keep the possibility to use a Regex instead of a list

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 58841] Add average metric in BackendListener and change samplersList by a regex match

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=58841

--- Comment #3 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Why don't you use percentiles ?:
https://blog.optimizely.com/2013/12/11/why-cdn-balancing/

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 58841] Add average metric in BackendListener and change samplersList by a regex match

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=58841

Philippe Mouawad <p....@ubik-ingenierie.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |p.mouawad@ubik-ingenierie.c
                   |                            |om

--- Comment #1 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Hello,
Thanks for contribution.
Avg metric has no real value in Load Testing so I don't think it should be
added.
Regarding the switch to regexp, it seems a good idea but introduces a backward
incompatibility.

What do other team members think ?
Regards
Philippe

-- 
You are receiving this mail because:
You are the assignee for the bug.