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.

Sign up for GitHub

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

Jump to bottom

Added upper-bound size limit to the kafka producer #375

Closed
wants to merge 4 commits into from
Closed

Added upper-bound size limit to the kafka producer #375

wants to merge 4 commits into from

Conversation

rogaha
Copy link

@rogaha rogaha commented May 12, 2015

No description provided.

@dpkp dpkp mentioned this pull request May 12, 2015
Closed
@rogaha
Copy link
Author

rogaha commented May 12, 2015

I'm fine to close this in favor of #304. What do you prefer @dpkp?

@rogaha rogaha force-pushed the add_upper_bound_size_limit branch 2 times, most recently from e88ecec to 2721a29 Compare May 12, 2015 23:46
@rogaha rogaha force-pushed the add_upper_bound_size_limit branch from 32aec02 to ee47bbb Compare May 13, 2015 03:48
@dpkp
Copy link
Owner

dpkp commented May 13, 2015

would like to get #331 merged for this issue instead. If you have a second to review, would appreciate more eyeballs and/or testing.

@rogaha
Copy link
Author

rogaha commented May 13, 2015

Sounds good! I'll close this one in favor of #331. Definitely, I'll review that one! Thanks @dpkp

@rogaha rogaha closed this May 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@rogaha @dpkp