[camel] 07/07: CAMEL-14996: Splitter/Multicast EIP can cause a thread to starve from endless stackframes when splitting as it does not collapse its stackframes but keep scheduling for next split/task.

[da...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

davsclaus pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git

commit 4e95bad28364c53d340af5b1dc11f41856268d13
Author: Claus Ibsen <cl...@gmail.com>
AuthorDate: Mon May 4 17:26:00 2020 +0200

    CAMEL-14996: Splitter/Multicast EIP can cause a thread to starve from endless stackframes when splitting as it does not collapse its stackframes but keep scheduling for next split/task.
---
 .../processor/aggregator/SplitAggregateStackOverflowIssueTest.java    | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/core/camel-core/src/test/java/org/apache/camel/processor/aggregator/SplitAggregateStackOverflowIssueTest.java b/core/camel-core/src/test/java/org/apache/camel/processor/aggregator/SplitAggregateStackOverflowIssueTest.java
index d5fcaf7..7858177 100644
--- a/core/camel-core/src/test/java/org/apache/camel/processor/aggregator/SplitAggregateStackOverflowIssueTest.java
+++ b/core/camel-core/src/test/java/org/apache/camel/processor/aggregator/SplitAggregateStackOverflowIssueTest.java
@@ -35,7 +35,9 @@ public class SplitAggregateStackOverflowIssueTest extends ContextTestSupport {
 
         MockEndpoint.assertIsSatisfied(60, SECONDS, result);
 
-        assertTrue("Stackframe must not be too high, was " + count.get(), count.get() < 100);
+        // the stackframe is 48 at this time of coding but can grow a little bit over time so lets just assume 70
+        // is fine, as if we get the endless bug again then this test fails before and this counter goes to 1024
+        assertTrue("Stackframe must not be too high, was " + count.get(), count.get() < 70);
     }
 
     @Override