ashutosh264 commented on PR #68904:
URL: https://github.com/apache/airflow/pull/68904#issuecomment-4903603626
Thanks for the review.
One follow-up idea I'd like your take on: show conf content expanded by
default in the Dag Runs list (i.e. drop collapsed on the conf cell), rather
than the folded { ... } preview users have to open row by row. (If I am not
wrong, this case is there in AF2)
Motivation: when I'm triaging or comparing runs, I often want to quickly
scan or diff conf values across rows (env, batch id, feature flags, etc.). With
the column visible but folded, I still have to expand each row manually, which
doesn't help much for search/compare workflows.
Trade-off I'm aware of: expanded JSON will make rows taller and the table
noisier, especially for large conf objects — which is probably why XCom/Assets
use collapsed previews in list views. Conf might be different, though: it's
usually smaller and more often the thing you're looking at on this page.
Questions:
Do you think unfolded conf by default is reasonable on Dag Runs, or should
we keep collapsed for consistency with other JSON columns?
If it's worth pursuing, I feel it should be tracked as a separate issue.
Happy to prototype and share a screenshot if that would help the discussion.
--
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]
