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

BufferOverflowException when sending events with large text bodies #23

Open
NeVeX opened this issue Dec 16, 2016 · 3 comments
Open

BufferOverflowException when sending events with large text bodies #23

NeVeX opened this issue Dec 16, 2016 · 3 comments
Labels

Comments

@NeVeX
Copy link

NeVeX commented Dec 16, 2016

Using version 2.3.

Not sure if this is limitation or not, but I could not find an obvious answer.
When sending events with the text body larger than the buffer limit (1400 bytes in the 2.3 version), a BufferOverflowException occurs.
When building the Event object, there's no mention of the body size limit, perhaps there needs to be? Or can there be a way to handle events with big/larger text bodies?

An example of the exception received in the handler I register with the client looks like:

java.nio.BufferOverflowException
	at java.nio.HeapByteBuffer.put(HeapByteBuffer.java:189)
	at java.nio.ByteBuffer.put(ByteBuffer.java:859)
	at com.timgroup.statsd.NonBlockingStatsDClient$QueueConsumer.run(NonBlockingStatsDClient.java:872)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
	at java.lang.Thread.run(Thread.java:745)
@aantono
Copy link

aantono commented Feb 5, 2017

Looks related to #17

@schilling
Copy link
Contributor

@NeVeX once #66 is merged in you can configure the buffer limit and #65 proactively looks for messages over the limit so they're passed to the client's error handler. Do those two changes in conjunction sufficiently help you here?

@NeVeX
Copy link
Author

NeVeX commented Feb 19, 2019

Hi @schilling, I think the new configuration options and error handling will help me for sure, thanks.

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

No branches or pull requests

4 participants