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

KAFKA-6207 : Include start of record when RecordIsTooLarge #4234

Closed
wants to merge 0 commits into from

Conversation

samjawale
Copy link

When a message is too large to be sent (at org.apache.kafka.clients.producer.KafkaProducer#doSend), the RecordTooLargeException should carry the start of the record (for example, the first 1KB) so that the calling application can debug which message caused the error.

To resolve this, I created a new function which will get the first 1KB of the message (if the RecordTooLargeException is occurring), and the same will be logged...

Thanks,
Sumit Jawale

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

21 similar comments
@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
No test results found.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
7975 tests run, 5 skipped, 1 failed.
--none--

1 similar comment
@asfgit
Copy link

asfgit commented Nov 19, 2017

FAILURE
7975 tests run, 5 skipped, 1 failed.
--none--

@asfgit
Copy link

asfgit commented Nov 19, 2017

SUCCESS
8083 tests run, 5 skipped, 0 failed.
--none--

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