Re: JMeter : Migrate to Java8

2016-08-04 Thread Milamber
JMeter 3.0 is the first release which drop the Java 6 support. The history of JMeter releases show that the drop of old java version is long (several year after the end of life) Drop the Java 7 support for the 3.1 or 3.2 seems not follow the same strategy that the past. Perhaps, we need to

Re: JMeter : Migrate to Java8

2016-08-04 Thread Milamber
On 03/08/2016 20:40, Philippe Mouawad wrote: Felix wanted a bugfix release ASAP, I support this idea. Which bug(s) need to make a release asap? We are free to decide to start a release if it's needed. It could be named 3.1 as there are some enhancements, for me it could come soon. But m

Re: Default engine script language to JSR223 elements

2016-08-04 Thread Milamber
Why not. If the script language has not chosen on the UI, add a INFO (or WARN) message into the log to inform that the script will be interpreted as a groovy script On 03/08/2016 18:15, Antonio Gomes Rodrigues wrote: Hi all, I have a proposition about default engine script language to JSR22

[GitHub] jmeter pull request #222: fixed garbled label in report dashboard

2016-08-04 Thread scyiwei
GitHub user scyiwei opened a pull request: https://github.com/apache/jmeter/pull/222 fixed garbled label in report dashboard When there are some Chinese characters in the label, the content is all messed up. I change the BufferedReader to UTF-8 encoding. Now it works fine. You can

Re: JMeter : Migrate to Java8

2016-08-04 Thread Felix Schumacher
Am 4. August 2016 11:56:20 MESZ, schrieb Milamber : > > >On 03/08/2016 20:40, Philippe Mouawad wrote: >> Felix wanted a bugfix release ASAP, I support this idea. > >Which bug(s) need to make a release asap? Well asap is a bit hard, but I would really like to release a version, which has all the

Re: JMeter : Migrate to Java8

2016-08-04 Thread Milamber
On 04/08/2016 12:45, Felix Schumacher wrote: Am 4. August 2016 11:56:20 MESZ, schrieb Milamber : On 03/08/2016 20:40, Philippe Mouawad wrote: Felix wanted a bugfix release ASAP, I support this idea. Which bug(s) need to make a release asap? Well asap is a bit hard, but I would really like

Re: Default engine script language to JSR223 elements

2016-08-04 Thread Antonio Gomes Rodrigues
Great idea I will try to implement it asap and make a PR Antonio 2016-08-04 12:02 GMT+02:00 Milamber : > > Why not. > If the script language has not chosen on the UI, add a INFO (or WARN) > message into the log to inform that the script will be interpreted as a > groovy script > > > On 03/08/20

[GitHub] jmeter issue #222: fixed garbled label in report dashboard

2016-08-04 Thread FSchumacher
Github user FSchumacher commented on the issue: https://github.com/apache/jmeter/pull/222 This problem was fixed on trunk with r1754436 (f933590417bc for git). Thanks for the report and the fix, though. --- If your project is set up for it, you can reply to this email and ha

Re: JMeter : Migrate to Java8

2016-08-04 Thread Felix Schumacher
Am 04.08.2016 um 16:52 schrieb Milamber: On 04/08/2016 12:45, Felix Schumacher wrote: Am 4. August 2016 11:56:20 MESZ, schrieb Milamber : On 03/08/2016 20:40, Philippe Mouawad wrote: Felix wanted a bugfix release ASAP, I support this idea. Which bug(s) need to make a release asap? Well as

Re: JMeter : Migrate to Java8

2016-08-04 Thread Felix Schumacher
Am 03.08.2016 um 21:34 schrieb Philippe Mouawad: Hello, What do you think of moving next major JMeter release 3.1 or 4 to Java 8, not the next bugfix release. +1 for me as : - lot of libraries are dropping Java7 compat - lambda, streams,nio2 Java 7 is EOL since April 2015 ( http://www.

Re: JMeter : Migrate to Java8

2016-08-04 Thread Vladimir Sitnikov
> > > Java 7 is EOL since April 2015 ( > > http://www.oracle.com/technetwork/java/eol-135779.html) > Oracle jdk 7 is eol. There is still openjdk. I don't think that has > reached eol yet. > EOL means "no more public updates, including security ones". Is there a vast number of users who absolutely

[GitHub] jmeter pull request #223: Bug59945_DefaultGroovyEngineToJSR223ElementsV2

2016-08-04 Thread ra0077
GitHub user ra0077 opened a pull request: https://github.com/apache/jmeter/pull/223 Bug59945_DefaultGroovyEngineToJSR223ElementsV2 Hi, If the script language has not been chosen on the GUI, I propose to add a WARN message into the log to inform that the script will be

[GitHub] jmeter pull request #223: Bug59945_DefaultGroovyEngineToJSR223ElementsV2

2016-08-04 Thread vlsi
Github user vlsi commented on a diff in the pull request: https://github.com/apache/jmeter/pull/223#discussion_r73605217 --- Diff: src/core/org/apache/jmeter/util/JSR223TestElement.java --- @@ -86,11 +88,16 @@ public JSR223TestElement() { } protected ScriptEn

[GitHub] jmeter issue #223: Bug59945_DefaultGroovyEngineToJSR223ElementsV2

2016-08-04 Thread vlsi
Github user vlsi commented on the issue: https://github.com/apache/jmeter/pull/223 Do you think a test should be added to cover the change? An entry in changes.xml is missing as well. --- If your project is set up for it, you can reply to this email and have your reply appear on G

Re: Default engine script language to JSR223 elements

2016-08-04 Thread Antonio Gomes Rodrigues
Hi, My proposition of pacth PR : https://github.com/apache/jmeter/pull/223 commited Bugzilla : https://bz.apache.org/bugzilla/show_bug.cgi?id=59945 Antonio 2016-08-04 18:34 GMT+02:00 Antonio Gomes Rodrigues : > Great idea > > I will try to implement it asap and make a PR > > Antonio > > 2016

[GitHub] jmeter pull request #223: Bug59945_DefaultGroovyEngineToJSR223ElementsV2

2016-08-04 Thread vlsi
Github user vlsi commented on a diff in the pull request: https://github.com/apache/jmeter/pull/223#discussion_r73605191 --- Diff: src/core/org/apache/jmeter/util/ScriptingTestElement.java --- @@ -36,6 +36,8 @@ private String script = ""; // script (if file not provided)

[GitHub] jmeter pull request #222: fixed garbled label in report dashboard

2016-08-04 Thread scyiwei
Github user scyiwei closed the pull request at: https://github.com/apache/jmeter/pull/222 --- 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 ena

[GitHub] jmeter issue #222: fixed garbled label in report dashboard

2016-08-04 Thread scyiwei
Github user scyiwei commented on the issue: https://github.com/apache/jmeter/pull/222 I will close this one, thanks. --- 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 w

[GitHub] jmeter pull request #223: Bug59945_DefaultGroovyEngineToJSR223ElementsV2

2016-08-04 Thread milamberspace
Github user milamberspace commented on a diff in the pull request: https://github.com/apache/jmeter/pull/223#discussion_r73644512 --- Diff: src/core/org/apache/jmeter/util/JSR223TestElement.java --- @@ -86,11 +88,16 @@ public JSR223TestElement() { } protected

[GitHub] jmeter issue #223: Bug59945_DefaultGroovyEngineToJSR223ElementsV2

2016-08-04 Thread milamberspace
Github user milamberspace commented on the issue: https://github.com/apache/jmeter/pull/223 The manual must be updated too for indicate this new behavior (if no language, then groovy script) --- If your project is set up for it, you can reply to this email and have your reply appear