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

   > > glad the original issue was resolved. I still think having these 
collapsed by default is useful incase there are dags with too many task 
failures. Given this is the default landing page for the dag its worth keeping 
the load times on this page fast
   > 
   > I think it would be clearer to show the error log directly instead of 
requiring an extra click. But I agree it could slow down load times if there 
are many error logs. How about limiting the number of displayed logs in 
overview page instead?
   
   Im not sure about that aswell. Say we limit to 5 failed tasks and have them 
expanded by default. If the user is interested in the 6th task that isn't 
displayed here, they would still have to make multiple clicks to get the 
desired error log. Having them all available in the landing page but collapsed 
for performance seems to be better IMO. 
   
   Open to feedback here.


-- 
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