From 08524e586bf5aff8843dbe2222f68ac91bf5ead0 Mon Sep 17 00:00:00 2001
From: joehni <joerg.schaible@gmx.de>
Date: Mon, 28 Oct 2024 23:47:10 +0100
Subject: [PATCH] Adjust Parser Benchmark to measure deeply nested structure.

---
 xstream-jmh/pom.xml                           |  6 ++---
 .../benchmark/jmh/ParserBenchmark.java        | 26 ++++++++++++++++---
 2 files changed, 25 insertions(+), 7 deletions(-)

diff --git a/xstream-jmh/pom.xml b/xstream-jmh/pom.xml
index 248a57752..427b73335 100644
--- a/xstream-jmh/pom.xml
+++ b/xstream-jmh/pom.xml
@@ -1,6 +1,6 @@
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
     <!--
-     Copyright (C) 2015, 2017, 2018, 2019, 2020, 2021, 2022, 2023 XStream committers.
+     Copyright (C) 2015, 2017, 2018, 2019, 2020, 2021, 2022, 2023, 2024 XStream committers.
      All rights reserved.
      
      The software in this package is published under the terms of the BSD
@@ -140,7 +140,7 @@
     <dependency>
       <groupId>com.fasterxml.woodstox</groupId>
       <artifactId>woodstox-core</artifactId>
-      <scope>runtime</scope>
+      <scope>compile</scope>
     </dependency>
     <dependency>
       <groupId>org.dom4j</groupId>
@@ -165,7 +165,7 @@
     <dependency>
       <groupId>org.codehaus.jettison</groupId>
       <artifactId>jettison</artifactId>
-      <scope>runtime</scope>
+      <scope>compile</scope>
     </dependency>
     <dependency>
       <groupId>jakarta.xml.bind</groupId>
diff --git a/xstream-jmh/src/java/com/thoughtworks/xstream/benchmark/jmh/ParserBenchmark.java b/xstream-jmh/src/java/com/thoughtworks/xstream/benchmark/jmh/ParserBenchmark.java
index eb462d23e..a12060688 100644
--- a/xstream-jmh/src/java/com/thoughtworks/xstream/benchmark/jmh/ParserBenchmark.java
+++ b/xstream-jmh/src/java/com/thoughtworks/xstream/benchmark/jmh/ParserBenchmark.java
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2015, 2017, 2021 XStream Committers.
+ * Copyright (C) 2015, 2017, 2021, 2024 XStream Committers.
  * All rights reserved.
  *
  * The software in this package is published under the terms of the BSD
@@ -18,6 +18,9 @@
 import java.util.List;
 import java.util.concurrent.TimeUnit;
 
+import javax.xml.stream.XMLInputFactory;
+
+import org.codehaus.jettison.json.JSONObject;
 import org.openjdk.jmh.annotations.Benchmark;
 import org.openjdk.jmh.annotations.BenchmarkMode;
 import org.openjdk.jmh.annotations.Fork;
@@ -33,6 +36,7 @@
 import org.openjdk.jmh.annotations.Warmup;
 import org.openjdk.jmh.infra.BenchmarkParams;
 
+import com.ctc.wstx.api.WstxInputProperties;
 import com.thoughtworks.xstream.XStream;
 import com.thoughtworks.xstream.io.HierarchicalStreamDriver;
 import com.thoughtworks.xstream.io.HierarchicalStreamWriter;
@@ -69,6 +73,10 @@
 @Threads(1)
 @Warmup(iterations = 5)
 public class ParserBenchmark {
+    {
+        // Increase max recursion depth for Jettison
+        JSONObject.setGlobalRecursionDepthLimit(2000);
+    }
 
     /**
      * Driver factory. Enum values used as parameter for the parser benchmark methods.
@@ -106,7 +114,15 @@ public enum DriverFactory {
          *
          * @since 1.4.9
          */
-        Woodstox(new WstxDriver()), //
+        Woodstox(new WstxDriver() {
+            @Override
+            protected XMLInputFactory createInputFactory() {
+                final XMLInputFactory inputFactory = super.createInputFactory();
+                // Increase max recursion depth for Woodstox
+                inputFactory.setProperty(WstxInputProperties.P_MAX_ELEMENT_DEPTH, Integer.valueOf(2000));
+                return inputFactory;
+            }
+        }), //
         /**
          * Factory for the {@link BEAStaxDriver}.
          *
@@ -163,7 +179,7 @@ public HierarchicalStreamWriter createWriter(final Writer out) {
 
         private final HierarchicalStreamDriver driver;
 
-        private DriverFactory(final HierarchicalStreamDriver driver) {
+        DriverFactory(final HierarchicalStreamDriver driver) {
             this.driver = driver;
         }
 
@@ -229,7 +245,7 @@ public void checkData(final Object o) {
             }
         },
         /**
-         * Nested list in list structure, 500 elements deep.
+         * Nested list in list structure, 1000 elements deep.
          *
          * @author J&ouml;rg Schaible
          * @since 1.4.9
@@ -294,6 +310,7 @@ public void checkData(final Object o) {
                 assert 0 == array[LENGTH - 1] : ManyChildren + " fails end";
             }
         };
+
         /**
          * Write the data of the factory into the writer of the hierarchical stream.
          *
@@ -331,6 +348,7 @@ public void init() {
         xstream.addPermission(PrimitiveTypePermission.PRIMITIVES);
         xstream.allowTypes(List.class, String.class);
         xstream.setMode(XStream.NO_REFERENCES);
+        xstream.setCollectionUpdateLimit(0);
         driver = driverFactory.getDriver();
     }