java.lang.OutOfMemoryError: Java heap space

[SummerOfServenteen]

Please answer these questions before submitting your issue.

• Why do you submit this issue?
• Question or discussion
• Bug
• Requirement
• Feature or performance improvement

---

Question

• What do you want to know?

---

Bug

• Which version of SkyWalking, OS and JRE?
  SkyWalking：6.1.0,jre:1.8
• Which company or project?
  jingling
• What happen?
  If possible, provide a way for reproducing the error. e.g. demo application, component version.

---

Requirement or improvement

• Please describe about your requirements or improvement suggestions.

[kezhenxu94]

@SummerOfServenteen Can you provide more information?

[wu-sheng]

@kezhenxu94 We only set version if we plan to accept PR. Before that, we discuss.

[SummerOfServenteen]

hello.When we use skyWalking ,it cause Java heap space。after analyze the dump file ,we find it's conflict with Sentinel.
Here is the detail.

Finally,i find it's method of org.apache.skywalking.apm.agent.core.context.util.ThrowableTransformer.convert2String that cause outOfMemory .

Code analyzing...

Because of Sentinel had overWrite the method of fillInStackTrace(),so the stackTrace of size is 0,
the return value of printStackElement is always false.Then the method of convert2String is in Infinite loop.

My idea is rewrite the method of printStackElement like this :

[SummerOfServenteen]

@wu-sheng,@kezhenxu94,can you help me to solve this problem~

[kezhenxu94]

@SummerOfServenteen Thanks for reporting and elaborate on this, seems you've found a solution, could you please send a pull request to fix this?

[SummerOfServenteen]

👌

[kezhenxu94]

I'll take a look at the issue tomorrow

[wu-sheng]

Why don't we fix append listenser bug and make the overMaxLength works?

[wu-sheng]

Quote from expert

1. Worst case: little memory and very deep stack.
2. Use StackWalker and unsafe API before Java 9.
3. Like Sentinel, do intentionally.

I will raise a PR to make a better fix.