Re: NewDriver : MalformedURLException

2016-12-30 Thread Philippe Mouawad
Hello, I have modified the code to throw. Please review and comment. Thanks On Wed, Dec 28, 2016 at 9:43 PM, Philippe Mouawad < philippe.moua...@gmail.com> wrote: > Hello, > Looking at sonar analysis, we see in NewDriver class that a lot > MalformedURLException are ignored and just logged (using

Jenkins build is back to normal : JMeter-trunk #5677

2016-12-30 Thread Apache Jenkins Server
See

Build failed in Jenkins: JMeter-trunk #5676

2016-12-30 Thread Apache Jenkins Server
See Changes: [pmouawad] Bug 60507 - Added 'Or' Function into ResponseAssertion Fix contributor name Bugzilla Id: 60507 [pmouawad] Bug 60507 - Added 'Or' Function into ResponseAssertion Based on contribution by 忻隆 Bugzilla Id: 60507 ---

[GitHub] jmeter issue #237: Adds a time boxed sampling for backend listeners

2016-12-30 Thread pmouawad
Github user pmouawad commented on the issue: https://github.com/apache/jmeter/pull/237 @max3163 , are you using GraphiteBackendListener or something else (I don't fully understand your comment on "I'm thinking to send a PR to share it too". If not, why aren't you using it ? Tha

[GitHub] jmeter pull request #238: Adds API to create JMeter threads on the fly

2016-12-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/jmeter/pull/238 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enab

buildbot success in on jmeter-trunk

2016-12-30 Thread buildbot
The Buildbot has detected a restored build on builder jmeter-trunk while building . Full details are available at: https://ci.apache.org/builders/jmeter-trunk/builds/1886 Buildbot URL: https://ci.apache.org/ Buildslave for this Build: bb_slave1_ubuntu Build Reason: The AnyBranchScheduler sc

buildbot failure in on jmeter-trunk

2016-12-30 Thread buildbot
The Buildbot has detected a new failure on builder jmeter-trunk while building . Full details are available at: https://ci.apache.org/builders/jmeter-trunk/builds/1884 Buildbot URL: https://ci.apache.org/ Buildslave for this Build: bb_slave1_ubuntu Build Reason: The AnyBranchScheduler sched

Re: AMQP/MQTT/Stomp/... support in JMeter

2016-12-30 Thread Philippe Mouawad
I agree provided the chosen library has not too much limitations. On Fri, Dec 30, 2016 at 4:17 PM, Antonio Gomes Rodrigues wrote: > In my opinion, I think it's better to get a library which support MQTT and > other protocols (AMQP, Stomp...) than taka a library which support only > MQTT > > 2016

Re: AMQP/MQTT/Stomp/... support in JMeter

2016-12-30 Thread Antonio Gomes Rodrigues
In my opinion, I think it's better to get a library which support MQTT and other protocols (AMQP, Stomp...) than taka a library which support only MQTT 2016-12-30 16:09 GMT+01:00 Philippe Mouawad : > On Fri, Dec 30, 2016 at 3:59 PM, Antonio Gomes Rodrigues > > wrote: > > > 2016-12-30 15:34 GMT+0

Re: AMQP/MQTT/Stomp/... support in JMeter

2016-12-30 Thread Philippe Mouawad
On Fri, Dec 30, 2016 at 3:59 PM, Antonio Gomes Rodrigues wrote: > 2016-12-30 15:34 GMT+01:00 Philippe Mouawad : > > > Hello, > > The answer probably depends on what we want to do with Rabbit MQ. > > > > What protocol do we intend to support and for what use cases ? > > > In my company, someone ha

Re: AMQP/MQTT/Stomp/... support in JMeter

2016-12-30 Thread Antonio Gomes Rodrigues
2016-12-30 15:34 GMT+01:00 Philippe Mouawad : > Hello, > The answer probably depends on what we want to do with Rabbit MQ. > > What protocol do we intend to support and for what use cases ? > In my company, someone have to test MQTT protocol In a French website which sells train ticket neet AMQP p

Re: AMQP/MQTT/Stomp/... support in JMeter

2016-12-30 Thread Philippe Mouawad
Hello, The answer probably depends on what we want to do with Rabbit MQ. What protocol do we intend to support and for what use cases ? Supporting a lot of protocol is not feasible IMO considering the size of our team. IMO we should concentrate our efforts on: - HTTP/2 - IOT protocol (MQTT

Re: GC policy for 3.2 release

2016-12-30 Thread Philippe Mouawad
Hello, Yes I think we should switch to it. I have been using it for a while now and I think it's much easier to adjust and leads without too much tuning to much shorter STW pauses. Regards On Fri, Dec 30, 2016 at 11:37 AM, Antonio Gomes Rodrigues wrote: > Hi, > > We will migrate to Java 8 to J

[GitHub] jmeter pull request #244: Bug60535 error prone boxed primitive constructor

2016-12-30 Thread FSchumacher
Github user FSchumacher commented on a diff in the pull request: https://github.com/apache/jmeter/pull/244#discussion_r94225584 --- Diff: src/core/org/apache/jmeter/report/processor/Top5ErrorsSummaryData.java --- @@ -29,7 +29,7 @@ */ public class Top5ErrorsSummaryData {

Re: Error Prone : Class Can Be Static

2016-12-30 Thread Vladimir Sitnikov
Hi, >Inner class is non-static but does not >reference enclosing class The change makes sense. Vladimir

[GitHub] jmeter issue #243: Bug60534 error prone default charset sample result java v...

2016-12-30 Thread pmouawad
Github user pmouawad commented on the issue: https://github.com/apache/jmeter/pull/243 Hi Antonio, Same remark as for other PR. Just commit directly the code unless you have a doubt. Regards --- If your project is set up for it, you can reply to this email and have yo

[GitHub] jmeter issue #244: Bug60535 error prone boxed primitive constructor

2016-12-30 Thread pmouawad
Github user pmouawad commented on the issue: https://github.com/apache/jmeter/pull/244 Hi Antonio, I wouldn't create a bugzilla and a PR for only this. Besides, the PR modifies other files that shouldn't . In this particular case, just commit directly as you have the rights.

[GitHub] jmeter pull request #244: Bug60535 error prone boxed primitive constructor

2016-12-30 Thread ra0077
GitHub user ra0077 opened a pull request: https://github.com/apache/jmeter/pull/244 Bug60535 error prone boxed primitive constructor You can merge this pull request into a Git repository by running: $ git pull https://github.com/ra0077/jmeter Bug60535_ErrorProne_BoxedPrimitiv

Error Prone : Class Can Be Static

2016-12-30 Thread Antonio Gomes Rodrigues
Hi, I have run error prone into the latest source and I have these wrnings [javac] C:\Util\0_perso\JMeterSVN\jmeter\src\core\org\apache\jmeter\report\dashboard\HtmlTemplateExporter.java:120: warning: [ClassCanBeStatic] Inner class is non-static but does not reference enclosing class [java

[GitHub] jmeter pull request #243: Bug60534 error prone default charset sample result...

2016-12-30 Thread ra0077
GitHub user ra0077 opened a pull request: https://github.com/apache/jmeter/pull/243 Bug60534 error prone default charset sample result java v2 Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=60534 You can merge this pull request into a Git repository by running: $ git pull h

GC policy for 3.2 release

2016-12-30 Thread Antonio Gomes Rodrigues
Hi, We will migrate to Java 8 to JMeter 3.2 release. Anybody have alredy tested GC G1 with JMeter? If yes, do we need to use it for next release or we keep current GC? Thank Antonio

AMQP/MQTT/Stomp/... support in JMeter

2016-12-30 Thread Antonio Gomes Rodrigues
Hi all, I am thinking about adding support for more messaging protocols (AMQP/MQTT/Stomp/...) To do it, we have 2 choices : Use ActiveMQ Java client Use RabbitMQ Java client I have checked some plugin to JMeter and Gatling and all use RabbitMQ Java client. ActiveMQ is compatible with only AMQP 1

Re: New Popup menu item to apply a Naming Policy

2016-12-30 Thread Antonio Gomes Rodrigues
Hi, I have already made a patch allow to add prefix during recording ( https://github.com/apache/jmeter/pull/156) But your proposition seems fine to modify/complete the name of TC Can we parameter the Apply Naming Policy? For example : prefix = something + original name + suffix = something Anto