Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Recursion Limit Hit Issue #577

Closed
humphriest opened this issue Apr 6, 2018 · 1 comment
Closed

Recursion Limit Hit Issue #577

humphriest opened this issue Apr 6, 2018 · 1 comment

Comments

@humphriest
Copy link

When printing out breadcrumbs on Android I am getting <recursion limit hit>. This is causing a problem when debugging errors by not showing much needed information.

private static final int MAX_LENGTH_LIST = 10;
private static final int MAX_SIZE_MAP = 50;
private static final int MAX_LENGTH_STRING = 400;
private static final int MAX_NESTING = 3;
private static final String ELIDED = "...";

This is where the variable MAX_NESTING is being set and applying the <recursion limit hit> message when recursion is over 3. Is it possible to remove this limit altogether or even higher it to a much larger number that won't hinder debugging?

@bretthoerner
Copy link

The main issue is that events can be of a certain size. In addition, Jackson seems to offer no way for us to see how many bytes we have written so far, so that we could truncate by size.

And so since we know nothing about user's data structures we err towards accepting more events vs dropping them due to size.

We have an issue to make these options configurable, so I'm closing this as a dupe of #543.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants