Using StringTokenizer instead of String.split() in AbstractComponent (#8759)

Avoids wasteful creation and parsing of regular expressions, which are not required here.

Change-Id: I247088fbfc4eca6c2500bd6b4da9afc9d66d0a51

1 files changed, 52 insertions, 0 deletions

diff --git a/server/tests/src/com/vaadin/benchmarks/PerformanceTester8759.java b/server/tests/src/com/vaadin/benchmarks/PerformanceTester8759.java
new file mode 100644
index 0000000000..968edd4b24
--- /dev/null
+++ b/server/tests/src/com/vaadin/benchmarks/PerformanceTester8759.java
@@ -0,0 +1,52 @@
+package com.vaadin.benchmarks;
+
+import com.vaadin.ui.Label;
+
+/*
+ * This simple test shows the performance difference between the StringTokenizer implementation and the String.split() implementation in AbstractComponent.
+ * Your results will vary.
+ * The real world use case motivating it was a 10k Row table, which generated labels for 10 columns. 
+ * This is 1/10th of what this performance tester demonstrates.
+ * 
+ * Please run with -server and -Xloggc:/tmp/gclog.vgc -verbose:gc -XX:+PrintCompilation
+ * 
+ * My results Win 7 64, i7 2760QM 2.4Ghz, Java 7 21.
+ * 
+ * Proposed Patch with StringTokenizer:
+ * 13 GC activations, 1.009GB allocated memory over time, total time 948ms
+ * 
+ * Current String.split implementation:
+ * 31 GC activations, 2.277 GB allocated memory over time, total time 1557ms
+ * 
+ */
+public class PerformanceTester8759 {
+
+    public static void main(String[] args) throws InterruptedException {
+        warmup();
+
+        long start = System.currentTimeMillis();
+        runBenchmark(1000000);
+        long end = System.currentTimeMillis();
+        System.out.println("took " + (end - start) + " ms");
+
+    }
+
+    private static void warmup() throws InterruptedException {
+        runBenchmark(10000);
+        System.gc();
+        System.out.println("warmup and gc complete. sleeping 5 seconds.");
+        Thread.sleep(5000l);
+        System.out.println("woke up - go.");
+    }
+
+    private static void runBenchmark(int loops) {
+        Label label = null;
+        for (int i = 0; i < loops; i++) {
+            label = new Label();
+            label.setStyleName("mainStyle");
+            label.addStyleName("foo bar  baz");
+            label.addStyleName("vaadin");
+        }
+    }
+
+}