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

Change auto-flush time interval in all clients to 1sec #151

Closed
yurishkuro opened this issue May 8, 2017 · 2 comments
Closed

Change auto-flush time interval in all clients to 1sec #151

yurishkuro opened this issue May 8, 2017 · 2 comments
Assignees

Comments

@yurishkuro
Copy link
Member

For some reason we set it to 10 seconds in some libs. This is an umbrella issue to check that all clients are sending at least every second (they will send more often if the data exceeds 64k)

@yurishkuro
Copy link
Member Author

yurishkuro commented Dec 23, 2017

In Java it's already 1000ms https://github.com/jaegertracing/jaeger-client-java/blob/master/jaeger-core/src/main/java/com/uber/jaeger/Configuration.java#L418

public static class ReporterConfiguration {
    private static final int DEFAULT_FLUSH_INTERVAL_MS = 1000;

@yurishkuro
Copy link
Member Author

Same in Python https://github.com/jaegertracing/jaeger-client-python/blob/master/jaeger_client/constants.py#L28

# How often remote reporter does a preemptive flush of its buffers
DEFAULT_FLUSH_INTERVAL = 1

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

1 participant