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 <claus.ib...@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