Skip to main content


Eclipse Community Forums
Forum Search:

Search      Help    Register    Login    Home
Home » Archived » Hudson » Plugin - Grinder patch(Patch to make grinder reports work)
Plugin - Grinder patch [message #1151890] Wed, 23 October 2013 17:24 Go to next message
Damian Dragowski is currently offline Damian DragowskiFriend
Messages: 2
Registered: October 2013
Junior Member
Conversion problem.


diff --git a/src/main/java/hudson/plugins/grinder/ResultReader.java b/src/main/java/hudson/plugins/grinder/ResultReader.java
index a5b5943..5d086a1 100644
--- a/src/main/java/hudson/plugins/grinder/ResultReader.java
+++ b/src/main/java/hudson/plugins/grinder/ResultReader.java
@@ -72,7 +72,8 @@ public class ResultReader {
    }
 
    private Test readTest(String testLine, boolean isTotals, boolean hasTPS) {
-      Scanner scanner = new Scanner(testLine).useDelimiter("\\s{2,}").useLocale(Locale.ENGLISH);
+      Scanner scanner = new Scanner(testLine).useDelimiter("\\s{2,}");
+      hudsonConsoleWriter.println(testLine);
       String id = scanner.next();
       int testCount = scanner.nextInt();
       int errorCount = scanner.nextInt();
@@ -91,9 +92,17 @@ public class ResultReader {
             scanner.next(); // reported as '?' in log
          }
          int respErrorCount = scanner.nextInt();
-         double resolveHostMeanTime = scanner.nextDouble();
+         double resolveHostMeanTime = 0.0;
+         if (scanner.hasNextDouble()) {
+             resolveHostMeanTime = scanner.nextDouble();
+         } else {
+             hudsonConsoleWriter.println(scanner.next());
+         }
          double establishConnMeanTime = scanner.nextDouble();
-         double firstByteMeanTime = Double.valueOf(scanner.next().replaceAll("[)]",""));
+         double firstByteMeanTime = 0.0;
+         if ( scanner.hasNextDouble()) {
+             firstByteMeanTime = scanner.nextDouble();
+         }
          String name = isTotals ? "" : scanner.next().replaceAll("\"", "");
 
          return new Test(

Report message to a moderator

Re: Plugin - Grinder patch [message #1152788 is a reply to message #1151890] Thu, 24 October 2013 07:07 Go to previous messageGo to next message
Damian Dragowski is currently offline Damian DragowskiFriend
Messages: 2
Registered: October 2013
Junior Member
Sometimes grinder log char "-" in oposite to expected value so I made better patch above:

diff --git a/src/main/java/hudson/plugins/grinder/ResultReader.java b/src/main/java/hudson/plugins/grinder/ResultReader.java
index a5b5943..7cf9a5e 100644
--- a/src/main/java/hudson/plugins/grinder/ResultReader.java
+++ b/src/main/java/hudson/plugins/grinder/ResultReader.java
@@ -72,7 +72,8 @@ public class ResultReader {
    }
 
    private Test readTest(String testLine, boolean isTotals, boolean hasTPS) {
-      Scanner scanner = new Scanner(testLine).useDelimiter("\\s{2,}").useLocale(Locale.ENGLISH);
+      Scanner scanner = new Scanner(testLine).useDelimiter("\\s{2,}");
+      hudsonConsoleWriter.println(testLine);
       String id = scanner.next();
       int testCount = scanner.nextInt();
       int errorCount = scanner.nextInt();
@@ -91,9 +92,22 @@ public class ResultReader {
             scanner.next(); // reported as '?' in log
          }
          int respErrorCount = scanner.nextInt();
-         double resolveHostMeanTime = scanner.nextDouble();
-         double establishConnMeanTime = scanner.nextDouble();
-         double firstByteMeanTime = Double.valueOf(scanner.next().replaceAll("[)]",""));
+         double resolveHostMeanTime = 0.0;
+         if (scanner.hasNextDouble()) {
+             resolveHostMeanTime = scanner.nextDouble();
+         } else {
+             hudsonConsoleWriter.println(scanner.next());
+         }
+         String eCMT = scanner.next();
+         hudsonConsoleWriter.println(eCMT);
+         eCMT = eCMT.replaceAll(",", ".").replaceAll("-","0");
+         double establishConnMeanTime = Double.valueOf(eCMT);
+         double firstByteMeanTime = 0.0;
+         if ( scanner.hasNextDouble()) {
+             firstByteMeanTime = scanner.nextDouble();
+         } else {
+             scanner.next();
+         }
          String name = isTotals ? "" : scanner.next().replaceAll("\"", "");
 
          return new Test(

Report message to a moderator

Re: Plugin - Grinder patch [message #1154014 is a reply to message #1152788] Fri, 25 October 2013 01:03 Go to previous message
Winston Prakash is currently offline Winston PrakashFriend
Messages: 534
Registered: August 2011
Location: Fremont, CA USA
Senior Member
Hi Damian, thanks for the patch. Is it possible to provide the patch as a pull request to https://github.com/hudson3-plugins/grinder-plugin ?. If that is inconvenient for you, I can manually apply the patch.

Winston Prakash
Eclipse Hudson team

Report message to a moderator

Previous Topic:hudson-remoting-3.0.1.jar not being found by mavin
Next Topic:Duplicate checkout using SVN and multiconfiguration project
Goto Forum:
  

[ Syndicate this forum (XML) ] [ RSS ]

Current Time: Fri Apr 19 01:26:00 GMT 2024

Powered by FUDForum. Page generated in 0.02644 seconds
.:: Contact :: Home ::.

Powered by: FUDforum 3.0.2.
Copyright ©2001-2010 FUDforum Bulletin Board Software

Back to the top

Sign up to our Newsletter

A fresh new issue delivered monthly