Re: PR221 : Guava or concurrentlinkedhashmap or Caffeine after Java8 migration

2016-08-08 Thread Philippe Mouawad
Hello,
Any thoughts on this ?
Regards

On Wednesday, August 3, 2016, Philippe Mouawad 
wrote:

> Hello,
>
> We have a PR-221 which relies on Guava.
>
> There is a discussion on github on the dependency on guava.
>
> What do team members think about this:
> - Should we introduce guava (which can be used for other things by the way
> not only for this feature)
> - Should we use https://github.com/ben-manes/concurrentlinkedhashmap
> - Should we migrate to Java 8 and then use https://github.com/ben-manes/
> caffeine
>
>
>
> I vote for guava + PR integration as :
>
>- I see interesting things in Guava for JMeter even if we move to Java
>8:
>   - cache
>   - annotations
>   - collections
>   - ...
>
> Although it's a minor agurment, the PR is ready to go, it's not if we
> select CLHM or caffeine.
>
>
>
>
> --
> Regards.
> Philippe
>


-- 
Cordialement.
Philippe Mouawad.


buildbot failure in on jmeter-nightly

2016-08-08 Thread buildbot
The Buildbot has detected a new failure on builder jmeter-nightly while 
building . Full details are available at:
https://ci.apache.org/builders/jmeter-nightly/builds/395

Buildbot URL: https://ci.apache.org/

Buildslave for this Build: hemera_ubuntu

Build Reason: The Nightly scheduler named 'jmeterNightly' triggered this build
Build Source Stamp: [branch jmeter/trunk] HEAD
Blamelist: 

BUILD FAILED: failed shell_8

Sincerely,
 -The Buildbot





[GitHub] jmeter issue #225: BUG59951_BetterScriptLanguageChooseDisplayJSR223

2016-08-08 Thread ra0077
Github user ra0077 commented on the issue:

https://github.com/apache/jmeter/pull/225
  
Hi Philippe,

My opinion is that it's not interesting for the majority of users.
Is it possible to put these information in an other part of JMeter (log, 
...)?

Antonio




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: svn commit: r1755458 - /jmeter/trunk/src/core/org/apache/jmeter/report/processor/NormalizerSampleConsumer.java

2016-08-08 Thread Philippe Mouawad
Hi Felix,
Oups, fixed.
Thanks

On Mon, Aug 8, 2016 at 12:25 PM, Felix Schumacher <
felix.schumac...@internetallee.de> wrote:

>
>
> Am 7. August 2016 22:18:50 MESZ, schrieb pmoua...@apache.org:
> >Author: pmouawad
> >Date: Sun Aug  7 20:18:50 2016
> >New Revision: 1755458
> >
> >URL: http://svn.apache.org/viewvc?rev=1755458=rev
> >Log:
> >Fix typo
> >
> >Modified:
> >jmeter/trunk/src/core/org/apache/jmeter/report/processor/
> NormalizerSampleConsumer.java
> >
> >Modified:
> >jmeter/trunk/src/core/org/apache/jmeter/report/processor/
> NormalizerSampleConsumer.java
> >URL:
> >http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/report/processor/NormalizerSampleConsumer.java?
> rev=1755458=1755457=1755458=diff
> >===
> ===
> >---
> >jmeter/trunk/src/core/org/apache/jmeter/report/processor/
> NormalizerSampleConsumer.java
> >(original)
> >+++
> >jmeter/trunk/src/core/org/apache/jmeter/report/processor/
> NormalizerSampleConsumer.java
> >Sun Aug  7 20:18:50 2016
> >@@ -116,8 +116,8 @@ public class NormalizerSampleConsumer ex
> > data[i] = s.getData(i);
> > }
> > }
> >-Sample rewrited = new Sample(s.getSampleRow(), sampleMetadata,
> >data);
> >-super.produce(rewrited, 0);
> >+Sample rewriten = new Sample(s.getSampleRow(), sampleMetadata,
>
> rewritten?
>
> Regards,
> Felix
>
> >data);
> >+super.produce(rewriten, 0);
> > }
> >
> > @Override
>
>


-- 
Cordialement.
Philippe Mouawad.


Re: svn commit: r1755458 - /jmeter/trunk/src/core/org/apache/jmeter/report/processor/NormalizerSampleConsumer.java

2016-08-08 Thread Felix Schumacher


Am 7. August 2016 22:18:50 MESZ, schrieb pmoua...@apache.org:
>Author: pmouawad
>Date: Sun Aug  7 20:18:50 2016
>New Revision: 1755458
>
>URL: http://svn.apache.org/viewvc?rev=1755458=rev
>Log:
>Fix typo
>
>Modified:
>jmeter/trunk/src/core/org/apache/jmeter/report/processor/NormalizerSampleConsumer.java
>
>Modified:
>jmeter/trunk/src/core/org/apache/jmeter/report/processor/NormalizerSampleConsumer.java
>URL:
>http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/report/processor/NormalizerSampleConsumer.java?rev=1755458=1755457=1755458=diff
>==
>---
>jmeter/trunk/src/core/org/apache/jmeter/report/processor/NormalizerSampleConsumer.java
>(original)
>+++
>jmeter/trunk/src/core/org/apache/jmeter/report/processor/NormalizerSampleConsumer.java
>Sun Aug  7 20:18:50 2016
>@@ -116,8 +116,8 @@ public class NormalizerSampleConsumer ex
> data[i] = s.getData(i);
> }
> }
>-Sample rewrited = new Sample(s.getSampleRow(), sampleMetadata,
>data);
>-super.produce(rewrited, 0);
>+Sample rewriten = new Sample(s.getSampleRow(), sampleMetadata,

rewritten?

Regards, 
Felix 

>data);
>+super.produce(rewriten, 0);
> }
> 
> @Override



Early Access builds of JDK 8u112 b04, JDK 9 b130 are available on java.net

2016-08-08 Thread Rory O'Donnell


Hi Philippe,

Early Access b130  for JDK 9 is 
available on java.net, summary of  changes are listed here 
.


Early Access b129  (#5332) for JDK 9 with 
Project Jigsaw is available on java.net, summary of changes are listed 
here 



Early Access b04  for JDK 8u112 is 
available on java.net, summary of  changes are listed here 




Rgds,Rory

--
Rgds,Rory O'Donnell
Quality Engineering Manager
Oracle EMEA , Dublin, Ireland