Skip to content
This repository has been archived by the owner on Jul 20, 2019. It is now read-only.

Allow clients to enforce a maximum produce message size. This allows pro... #11

Merged
merged 1 commit into from
Jan 10, 2013
Merged

Allow clients to enforce a maximum produce message size. This allows pro... #11

merged 1 commit into from
Jan 10, 2013

Conversation

sholsapp
Copy link
Contributor

...ducers

to accomodate consumers that have a maximum fetch size. This is especially
important for kafka producers that batch messages.

This change introduces a new interface to the producer's send method. First,
now the method may send multiple messages to a kafka broker (this is the case
if the list of messages exceeds the maximum allowed size of kafka produce
message). Additionally, if a message is not sent in its entirety, an IOError
is now raised.

nosetests

.........

Ran 9 tests in 0.541s

OK

…producers

to accomodate consumers that have a maximum fetch size. This is especially
important for kafka producers that batch messages.

This change introduces a new interface to the producer's send method. First,
now the method may send multiple messages to a kafka broker (this is the case
if the list of messages exceeds the maximum allowed size of kafka produce
message). Additionally, if a message is not sent in its entirety, an IOError
is now raised.

nosetests
.........
----------------------------------------------------------------------
Ran 9 tests in 0.541s

OK
@sholsapp
Copy link
Contributor Author

Hmm github markdown removed nosetests output, but all tests are currently passing.

dsully pushed a commit that referenced this pull request Jan 10, 2013
Allow clients to enforce a maximum produce message size. This allows pro...
@dsully dsully merged commit 0d5af6b into dsully:master Jan 10, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants