dheerajturaga commented on PR #54377:
URL: https://github.com/apache/airflow/pull/54377#issuecomment-3235463090

   > If you try to do this with tasks that have big logs (10M) the collapsing 
will help, but after expanding a couple of them, the page still becomes 
unresponsive. (Without this, it's unresponsive/crash on load).
   > 
   > Yes, as mentionned by @guan404ming ideally I think on the 'overview' we 
only want to show a log sample, maybe the last 100 lines of the logs to be able 
to debug most cases, if not enough they will need to go specifically check the 
TI logs on the TI page to load full content. For now we can just do this 
truncation in the frontend. (for instance custom hook wrapping useLogs and 
truncating response based on a param right after the backend response is 
received, i.e onSuccess, `useLogs(limit=100)`).
   > 
   > The long term solution would be to implement a backend feature to be able 
to paginate logs as well as a read order (start to end, end to start), so we 
could read the most recent 100 lines of the log file and return that directly, 
I don't think we support that yet cc: @jason810496
   
   @pierrejeambrun , I have updated this to `useLogs(limit=100)` this should 
prevent the dag page ending up in a bad state. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to