Skip to content
This repository has been archived by the owner on Oct 3, 2023. It is now read-only.

Tracing: Add specifications on default TraceParams. #139

Merged
merged 2 commits into from
Jul 16, 2018

Conversation

songy23
Copy link
Contributor

@songy23 songy23 commented Jul 13, 2018

Related to #138.

/cc @bogdandrutu

Copy link

@codefromthecrypt codefromthecrypt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

128 links sounds very high. what is a use case for tens of links much less over a hundred in the same span?

@songy23
Copy link
Contributor Author

songy23 commented Jul 16, 2018

128 links sounds very high. what is a use case for tens of links much less over a hundred in the same span?

@bogdandrutu Could you answer this question?

@bogdandrutu
Copy link
Contributor

The size of links is small so no memory issue to support 128, but probably the number is too high for real world use-cases.

@songy23 I would reduce that to 32 probably. The reason for this is that if you have some kind of batching you likely have >10 batched operations (not always but sometime).

@songy23
Copy link
Contributor Author

songy23 commented Jul 16, 2018

SG, updated the limit on Links to 32. Will update Java default params as well.

@codefromthecrypt
Copy link

thx folks

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

Successfully merging this pull request may close these issues.

None yet

5 participants