You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by pm...@apache.org on 2018/11/04 13:49:37 UTC
svn commit: r1845737 - in /jmeter/trunk:
src/core/org/apache/jmeter/report/processor/PercentileAggregator.java
xdocs/changes.xml
Author: pmouawad
Date: Sun Nov 4 13:49:37 2018
New Revision: 1845737
URL: http://svn.apache.org/viewvc?rev=1845737&view=rev
Log:
Bug 62883 - Report / Dashboard : Change the way percentiles are computed for Response Time Percentiles Over Time (successful responses) graph
Bugzilla Id: 62883
Modified:
jmeter/trunk/src/core/org/apache/jmeter/report/processor/PercentileAggregator.java
jmeter/trunk/xdocs/changes.xml
Modified: jmeter/trunk/src/core/org/apache/jmeter/report/processor/PercentileAggregator.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/report/processor/PercentileAggregator.java?rev=1845737&r1=1845736&r2=1845737&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/report/processor/PercentileAggregator.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/report/processor/PercentileAggregator.java Sun Nov 4 13:49:37 2018
@@ -50,7 +50,7 @@ public class PercentileAggregator implem
* @param lastAggregator {@link PercentileAggregator}
*/
public PercentileAggregator(PercentileAggregator lastAggregator) {
- this.statistics = new DescriptiveStatistics(lastAggregator.statistics);
+ statistics = new DescriptiveStatistics(SLIDING_WINDOW_SIZE);
this.percentileIndex = lastAggregator.percentileIndex;
}
Modified: jmeter/trunk/xdocs/changes.xml
URL: http://svn.apache.org/viewvc/jmeter/trunk/xdocs/changes.xml?rev=1845737&r1=1845736&r2=1845737&view=diff
==============================================================================
--- jmeter/trunk/xdocs/changes.xml [utf-8] (original)
+++ jmeter/trunk/xdocs/changes.xml [utf-8] Sun Nov 4 13:49:37 2018
@@ -69,6 +69,8 @@ Summary
<ch_section>Incompatible changes</ch_section>
+<li>In Response Time Percentiles Over Time (successful responses) graph of the HTML report, before this version, percentile computation of each timeslot used the percentile data
+of previous time stot as a base, starting with this version, each timeslot is independant. See <bugzilla>62883</bugzilla></li>
<!-- =================== Improvements =================== -->
@@ -94,7 +96,7 @@ Summary
<h3>Timers, Assertions, Config, Pre- & Post-Processors</h3>
<ul>
- <li><bug>62766</bug>Keystore Config: We should load all aliases by default. Contributed by Ubik Load Pack (support at ubikloadpack.com)</li>
+ <li><bug>62766</bug>Keystore Config : We should load all aliases by default. Contributed by Ubik Load Pack (support at ubikloadpack.com)</li>
<li><bug>62832</bug>JDBC Connection Configuration: Be able to set init SQL statements. Contributed by Ubik Load Pack (support at ubikloadpack.com)</li>
</ul>
@@ -108,6 +110,7 @@ Summary
<h3>Report / Dashboard</h3>
<ul>
+ <li><bug>62883</bug>Report / Dashboard : Change the way percentiles are computed for Response Time Percentiles Over Time (successful responses) graph</li>
</ul>
<h3>General</h3>
@@ -158,7 +161,7 @@ Summary
<h3>Timers, Assertions, Config, Pre- & Post-Processors</h3>
<ul>
- <li><bug>62774</bug>XPath2Extractor: Scope variable is broken. Contributed by Ubik Load Pack (support at ubikloadpack.com)</li>
+ <li><bug>62774</bug>XPath2Extractor : Scope variable is broken. Contributed by Ubik Load Pack (support at ubikloadpack.com)</li>
<li><bug>62860</bug>JSON Extractor: Avoid NPE and noisy error message "<code>Error processing JSON content in</code>" when variable is not found</li>
</ul>
@@ -172,10 +175,10 @@ Summary
<h3>Report / Dashboard</h3>
<ul>
- <li><bug>62777</bug>Web Report / Dashboard: Hide All in <code>Response Time Percentiles Over Time (successful responses)</code> fails.</li>
- <li><bug>62780</bug>Web Report / Dashboard: Display All in <code>Response Time Vs Request</code> fails.</li>
- <li><bug>62781</bug>Web Report / Dashboard: Display All in <code>Response Time Overview</code> fails.</li>
- <li><bug>62782</bug>Web Report / Dashboard: Remove duplicate/unused dependencies</li>
+ <li><bug>62777</bug>Web Report / Dashboard : Hide All in <code>Response Time Percentiles Over Time (successful responses)</code> fails.</li>
+ <li><bug>62780</bug>Web Report / Dashboard : Display All in <code>Response Time Vs Request</code> fails.</li>
+ <li><bug>62781</bug>Web Report / Dashboard : Display All in <code>Response Time Overview</code> fails.</li>
+ <li><bug>62782</bug>Web Report / Dashboard : Remove duplicate/unused dependencies</li>
</ul>
<h3>Documentation</h3>
Re: svn commit: r1845737 - in /jmeter/trunk: src/core/org/apache/jmeter/report/processor/PercentileAggregator.java
xdocs/changes.xml
Posted by Philippe Mouawad <ph...@gmail.com>.
Fixed. Thanks
On Sun, Nov 4, 2018 at 2:59 PM Felix Schumacher <
felix.schumacher@internetallee.de> wrote:
>
>
> Am 4. November 2018 14:49:37 MEZ schrieb pmouawad@apache.org:
> >Author: pmouawad
> >Date: Sun Nov 4 13:49:37 2018
> >New Revision: 1845737
> >
> >URL: http://svn.apache.org/viewvc?rev=1845737&view=rev
> >Log:
> >Bug 62883 - Report / Dashboard : Change the way percentiles are
> >computed for Response Time Percentiles Over Time (successful responses)
> >graph
> >Bugzilla Id: 62883
> >
> >Modified:
>
> >jmeter/trunk/src/core/org/apache/jmeter/report/processor/PercentileAggregator.java
> > jmeter/trunk/xdocs/changes.xml
> >
> >Modified:
>
> >jmeter/trunk/src/core/org/apache/jmeter/report/processor/PercentileAggregator.java
> >URL:
> >
> http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/report/processor/PercentileAggregator.java?rev=1845737&r1=1845736&r2=1845737&view=diff
>
> >==============================================================================
> >---
>
> >jmeter/trunk/src/core/org/apache/jmeter/report/processor/PercentileAggregator.java
> >(original)
> >+++
>
> >jmeter/trunk/src/core/org/apache/jmeter/report/processor/PercentileAggregator.java
> >Sun Nov 4 13:49:37 2018
> >@@ -50,7 +50,7 @@ public class PercentileAggregator implem
> > * @param lastAggregator {@link PercentileAggregator}
> > */
> > public PercentileAggregator(PercentileAggregator lastAggregator) {
> >- this.statistics = new
> >DescriptiveStatistics(lastAggregator.statistics);
> >+ statistics = new DescriptiveStatistics(SLIDING_WINDOW_SIZE);
> > this.percentileIndex = lastAggregator.percentileIndex;
> > }
> >
> >
> >Modified: jmeter/trunk/xdocs/changes.xml
> >URL:
> >
> http://svn.apache.org/viewvc/jmeter/trunk/xdocs/changes.xml?rev=1845737&r1=1845736&r2=1845737&view=diff
>
> >==============================================================================
> >--- jmeter/trunk/xdocs/changes.xml [utf-8] (original)
> >+++ jmeter/trunk/xdocs/changes.xml [utf-8] Sun Nov 4 13:49:37 2018
> >@@ -69,6 +69,8 @@ Summary
> >
> > <ch_section>Incompatible changes</ch_section>
> >
> >+<li>In Response Time Percentiles Over Time (successful responses)
> >graph of the HTML report, before this version, percentile computation
> >of each timeslot used the percentile data
> >+of previous time stot as a base, starting with this version, each
>
> s/ stot/slot/
>
> Felix
>
> >timeslot is independant. See <bugzilla>62883</bugzilla></li>
> >
> > <!-- =================== Improvements =================== -->
> >
> >@@ -94,7 +96,7 @@ Summary
> >
> > <h3>Timers, Assertions, Config, Pre- & Post-Processors</h3>
> > <ul>
> >- <li><bug>62766</bug>Keystore Config: We should load all aliases by
> >default. Contributed by Ubik Load Pack (support at
> >ubikloadpack.com)</li>
> >+ <li><bug>62766</bug>Keystore Config : We should load all aliases
> >by default. Contributed by Ubik Load Pack (support at
> >ubikloadpack.com)</li>
> ><li><bug>62832</bug>JDBC Connection Configuration: Be able to set init
> >SQL statements. Contributed by Ubik Load Pack (support at
> >ubikloadpack.com)</li>
> > </ul>
> >
> >@@ -108,6 +110,7 @@ Summary
> >
> > <h3>Report / Dashboard</h3>
> > <ul>
> >+ <li><bug>62883</bug>Report / Dashboard : Change the way
> >percentiles are computed for Response Time Percentiles Over Time
> >(successful responses) graph</li>
> > </ul>
> >
> > <h3>General</h3>
> >@@ -158,7 +161,7 @@ Summary
> >
> > <h3>Timers, Assertions, Config, Pre- & Post-Processors</h3>
> > <ul>
> >- <li><bug>62774</bug>XPath2Extractor: Scope variable is broken.
> >Contributed by Ubik Load Pack (support at ubikloadpack.com)</li>
> >+ <li><bug>62774</bug>XPath2Extractor : Scope variable is broken.
> >Contributed by Ubik Load Pack (support at ubikloadpack.com)</li>
> ><li><bug>62860</bug>JSON Extractor: Avoid NPE and noisy error message
> >"<code>Error processing JSON content in</code>" when variable is not
> >found</li>
> > </ul>
> >
> >@@ -172,10 +175,10 @@ Summary
> >
> > <h3>Report / Dashboard</h3>
> > <ul>
> >- <li><bug>62777</bug>Web Report / Dashboard: Hide All in
> ><code>Response Time Percentiles Over Time (successful responses)</code>
> >fails.</li>
> >- <li><bug>62780</bug>Web Report / Dashboard: Display All in
> ><code>Response Time Vs Request</code> fails.</li>
> >- <li><bug>62781</bug>Web Report / Dashboard: Display All in
> ><code>Response Time Overview</code> fails.</li>
> >- <li><bug>62782</bug>Web Report / Dashboard: Remove
> >duplicate/unused dependencies</li>
> >+ <li><bug>62777</bug>Web Report / Dashboard : Hide All in
> ><code>Response Time Percentiles Over Time (successful responses)</code>
> >fails.</li>
> >+ <li><bug>62780</bug>Web Report / Dashboard : Display All in
> ><code>Response Time Vs Request</code> fails.</li>
> >+ <li><bug>62781</bug>Web Report / Dashboard : Display All in
> ><code>Response Time Overview</code> fails.</li>
> >+ <li><bug>62782</bug>Web Report / Dashboard : Remove
> >duplicate/unused dependencies</li>
> > </ul>
> >
> > <h3>Documentation</h3>
>
--
Cordialement.
Philippe Mouawad.
Re: svn commit: r1845737 - in /jmeter/trunk: src/core/org/apache/jmeter/report/processor/PercentileAggregator.java xdocs/changes.xml
Posted by Felix Schumacher <fe...@internetallee.de>.
Am 4. November 2018 14:49:37 MEZ schrieb pmouawad@apache.org:
>Author: pmouawad
>Date: Sun Nov 4 13:49:37 2018
>New Revision: 1845737
>
>URL: http://svn.apache.org/viewvc?rev=1845737&view=rev
>Log:
>Bug 62883 - Report / Dashboard : Change the way percentiles are
>computed for Response Time Percentiles Over Time (successful responses)
>graph
>Bugzilla Id: 62883
>
>Modified:
>jmeter/trunk/src/core/org/apache/jmeter/report/processor/PercentileAggregator.java
> jmeter/trunk/xdocs/changes.xml
>
>Modified:
>jmeter/trunk/src/core/org/apache/jmeter/report/processor/PercentileAggregator.java
>URL:
>http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/report/processor/PercentileAggregator.java?rev=1845737&r1=1845736&r2=1845737&view=diff
>==============================================================================
>---
>jmeter/trunk/src/core/org/apache/jmeter/report/processor/PercentileAggregator.java
>(original)
>+++
>jmeter/trunk/src/core/org/apache/jmeter/report/processor/PercentileAggregator.java
>Sun Nov 4 13:49:37 2018
>@@ -50,7 +50,7 @@ public class PercentileAggregator implem
> * @param lastAggregator {@link PercentileAggregator}
> */
> public PercentileAggregator(PercentileAggregator lastAggregator) {
>- this.statistics = new
>DescriptiveStatistics(lastAggregator.statistics);
>+ statistics = new DescriptiveStatistics(SLIDING_WINDOW_SIZE);
> this.percentileIndex = lastAggregator.percentileIndex;
> }
>
>
>Modified: jmeter/trunk/xdocs/changes.xml
>URL:
>http://svn.apache.org/viewvc/jmeter/trunk/xdocs/changes.xml?rev=1845737&r1=1845736&r2=1845737&view=diff
>==============================================================================
>--- jmeter/trunk/xdocs/changes.xml [utf-8] (original)
>+++ jmeter/trunk/xdocs/changes.xml [utf-8] Sun Nov 4 13:49:37 2018
>@@ -69,6 +69,8 @@ Summary
>
> <ch_section>Incompatible changes</ch_section>
>
>+<li>In Response Time Percentiles Over Time (successful responses)
>graph of the HTML report, before this version, percentile computation
>of each timeslot used the percentile data
>+of previous time stot as a base, starting with this version, each
s/ stot/slot/
Felix
>timeslot is independant. See <bugzilla>62883</bugzilla></li>
>
> <!-- =================== Improvements =================== -->
>
>@@ -94,7 +96,7 @@ Summary
>
> <h3>Timers, Assertions, Config, Pre- & Post-Processors</h3>
> <ul>
>- <li><bug>62766</bug>Keystore Config: We should load all aliases by
>default. Contributed by Ubik Load Pack (support at
>ubikloadpack.com)</li>
>+ <li><bug>62766</bug>Keystore Config : We should load all aliases
>by default. Contributed by Ubik Load Pack (support at
>ubikloadpack.com)</li>
><li><bug>62832</bug>JDBC Connection Configuration: Be able to set init
>SQL statements. Contributed by Ubik Load Pack (support at
>ubikloadpack.com)</li>
> </ul>
>
>@@ -108,6 +110,7 @@ Summary
>
> <h3>Report / Dashboard</h3>
> <ul>
>+ <li><bug>62883</bug>Report / Dashboard : Change the way
>percentiles are computed for Response Time Percentiles Over Time
>(successful responses) graph</li>
> </ul>
>
> <h3>General</h3>
>@@ -158,7 +161,7 @@ Summary
>
> <h3>Timers, Assertions, Config, Pre- & Post-Processors</h3>
> <ul>
>- <li><bug>62774</bug>XPath2Extractor: Scope variable is broken.
>Contributed by Ubik Load Pack (support at ubikloadpack.com)</li>
>+ <li><bug>62774</bug>XPath2Extractor : Scope variable is broken.
>Contributed by Ubik Load Pack (support at ubikloadpack.com)</li>
><li><bug>62860</bug>JSON Extractor: Avoid NPE and noisy error message
>"<code>Error processing JSON content in</code>" when variable is not
>found</li>
> </ul>
>
>@@ -172,10 +175,10 @@ Summary
>
> <h3>Report / Dashboard</h3>
> <ul>
>- <li><bug>62777</bug>Web Report / Dashboard: Hide All in
><code>Response Time Percentiles Over Time (successful responses)</code>
>fails.</li>
>- <li><bug>62780</bug>Web Report / Dashboard: Display All in
><code>Response Time Vs Request</code> fails.</li>
>- <li><bug>62781</bug>Web Report / Dashboard: Display All in
><code>Response Time Overview</code> fails.</li>
>- <li><bug>62782</bug>Web Report / Dashboard: Remove
>duplicate/unused dependencies</li>
>+ <li><bug>62777</bug>Web Report / Dashboard : Hide All in
><code>Response Time Percentiles Over Time (successful responses)</code>
>fails.</li>
>+ <li><bug>62780</bug>Web Report / Dashboard : Display All in
><code>Response Time Vs Request</code> fails.</li>
>+ <li><bug>62781</bug>Web Report / Dashboard : Display All in
><code>Response Time Overview</code> fails.</li>
>+ <li><bug>62782</bug>Web Report / Dashboard : Remove
>duplicate/unused dependencies</li>
> </ul>
>
> <h3>Documentation</h3>