GitHub user ijuma opened a pull request:
https://github.com/apache/kafka/pull/1547
KAFKA-3594; After calling MemoryRecords.close() method, hasRoomFor() method
should return false
This exception is occurring when producer is trying to append a record to a
Re-enqueued record batch in the accumulator. We should not allow to add a
record to Re-enqueued record batch. This is due a bug in
MemoryRecords.java/hasRoomFor() method. After calling MemoryRecords.close()
method, hasRoomFor() method should return false.
This is a backport to the 0.9.0 branch.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ijuma/kafka kafka-3594-for-0.9.0
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/kafka/pull/1547.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1547
----
commit 3e3e8a28d88f5d191bbdc0c35f086a925bc8c512
Author: Manikumar reddy O <manikumar.re...@gmail.com>
Date: 2016-04-21T22:13:25Z
KAFKA-3594; After calling MemoryRecords.close() method, hasRoomFor() method
should return false
This exception is occurring when producer is trying to append a record to a
Re-enqueued record batch in the accumulator. We should not allow to add a
record to Re-enqueued record batch. This is due a bug in
MemoryRecords.java/hasRoomFor() method. After calling MemoryRecords.close()
method, hasRoomFor() method should return false.
Author: Manikumar reddy O <manikumar.re...@gmail.com>
Reviewers: Ismael Juma, Grant Henke, Guozhang Wang
Closes #1249 from omkreddy/KAFKA-3594
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
