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

Log display on very large logs from Dagu takes minutes to load in browser without clear indication it will work #640

Open
zph opened this issue Aug 2, 2024 · 1 comment
Labels
enhancement New feature to be implemented someday

Comments

@zph
Copy link
Contributor

zph commented Aug 2, 2024

I have a use case where the logs can be very large (example is 216MB on frontend once it fully loads).

The user experience is that the page fails to load while trying and loading the data. You can see it loading in network tab gradually.

Once the 216MB log is delivered clientside, there's a processing phase before it's rendered.

Options:

  1. Improve messaging/loading screen to show progress of load (simpler?)
  2. Improve performance

I haven't looked into a solution much yet beyond discovering the issue. If I discover an approach that doesn't require major changes I'll submit as a PR.

Version: around 9ef8153 from about ~4wk ago

As a secondary factor, I see the frontend continuing to poll and each polling appears to fetch full content of log which works in small cases but not large logs.

image

@yohamta
Copy link
Collaborator

yohamta commented Aug 2, 2024

Thank you for raising the issue. I agree that it is a problem. It appears that Airflow seems displays only a portion of log when the the full-log isn't necessary. This approach could serve as a potential solution to the problem. Additionally, I think that we should consider separating the API for polling and downloading the log file to address the the secondary factor.
https://stackoverflow.com/questions/74832250/airflow-logs-break-long-logs-into-smaller-multiple-files

@yohamta yohamta added the enhancement New feature to be implemented label Aug 2, 2024
@yohamta yohamta added the someday label Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature to be implemented someday
Projects
None yet
Development

No branches or pull requests

2 participants