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

Displaying large messages in topics is stalling the UI #564

Closed
rlamcaiso opened this issue Jan 15, 2021 · 10 comments · Fixed by #970
Closed

Displaying large messages in topics is stalling the UI #564

rlamcaiso opened this issue Jan 15, 2021 · 10 comments · Fixed by #970
Assignees
Labels
backend Need a backend update enhancement New feature or request ui Need an ui update

Comments

@rlamcaiso
Copy link

Hello @tchiotludo. Thank you for your tremendous effort with this project. We are just starting to use the application in our Kafka development and it has been a big help.

An issue that we are encountering is that the app would hang when we view a topic with many large messages. Would it be possible to disable the message preview window, and provide a button for individual messages to view an abbreviated string of the message (ie. 250 characters) and with a download button to download the full message?

Thank you so much for your time.

Regards,
Rick

@tchiotludo
Copy link
Owner

What size is your "with many large messages" ?
250 is really to low so I need to understand where we need to put the limit (backend or frontend) !

@rlamcaiso
Copy link
Author

Thank you for the prompt response. Our bulk messages range from 2MB to 60MB, and there could be hundreds of these messages generated per day and published to the topic.

@tchiotludo
Copy link
Owner

ok so I your case, the need will be server side !
60 mb will always froze the brower 😄

@tchiotludo tchiotludo added backend Need a backend update enhancement New feature or request ui Need an ui update labels Jan 15, 2021
@rlamcaiso
Copy link
Author

Yes. We don't know how else to manage this performance issue with our current payloads. I appreciate your help!

@EdwinFajardoBarrera
Copy link
Contributor

Hello @tchiotludo, I would like to work on a solution to this, can u assign it to me?

@tchiotludo
Copy link
Owner

How will you handle this ?

@EdwinFajardoBarrera
Copy link
Contributor

How will you handle this ?

Well, I think the backend is required to catch large messages and only send a small preview. I see the limit is almost 10MB but if we send 10MB messages the problem could just persist, so maybe I think 5MB could be ok. What do you think?

@tchiotludo
Copy link
Owner

It's a good solution, please add a configuration options with a decent default value (5MB)

@EdwinFajardoBarrera
Copy link
Contributor

EdwinFajardoBarrera commented Dec 29, 2021

Hello @tchiotludo, I still working on a solution to this issue. I'm catching the size of messages in the backend and the problem stills if there are many messages of length near to the limit (4.9MB for example. Thinking in change limit to 2.5 oe 3 MB) and I think cutting the message does not give a good UX because JSON format fails and maybe is not worth seeing only a cut part of a message. I'm thinking about only showing a message informing this situation and making more visible the "share" option to see the full message.

Something like this:

image

What do you think?

@tchiotludo
Copy link
Owner

I still think the truncate is the better option than nothing, debugging without any message is impossible, I prefer a snippet
Why not the share button can display the full message, it's a good idea.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Need a backend update enhancement New feature or request ui Need an ui update
Projects
Status: Done
3 participants