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

Limit size of messages in preview #710

Closed
adamdubiel opened this issue Feb 6, 2017 · 1 comment
Closed

Limit size of messages in preview #710

adamdubiel opened this issue Feb 6, 2017 · 1 comment
Milestone
0.11.3

Comments

@adamdubiel
Copy link
Collaborator

adamdubiel commented Feb 6, 2017 •
edited
Loading

Some topics might publish messages too big to fit into the preview and into ZK document in general, which causes IOException at ZK driver level. We should set limit for preview size and either trim messages or discard them (and leave a Message too big to preview note).
@mszarlinski
Copy link

I'm working on it.

mszarlinski pushed a commit to mszarlinski/hermes that referenced this issue Apr 5, 2017
Limit size of messages in preview allegro#710
8ba5057
mszarlinski pushed a commit to mszarlinski/hermes that referenced this issue Apr 6, 2017
Limit size of messages in preview allegro#710
ec82867
adamdubiel added a commit that referenced this issue Apr 7, 2017
@adamdubiel
Merge pull request #752 from mszarlinski/limit_size_of_messages_in_pr…
e84f114 
…eview

Limit size of messages in preview #710
@adamdubiel adamdubiel added this to the 0.11.3 milestone Apr 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.11.3
Development

No branches or pull requests
2 participants
@adamdubiel @mszarlinski