Depends on D1219
Details
Details
- Reviewers
alangenfeld - Group Reviewers
Restricted Project - Commits
- R1:c51387a521de: add client-side compute log size cap
Set client-side cap low, saw subscription connection close, warning sign
Diff Detail
Diff Detail
- Repository
- R1 dagster
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Event Timeline
Comment Actions
should we continue to stream logs but only retain/show the last X MB, similar to Buildkite?
js_modules/dagit/src/plan/ComputeLogContent.tsx | ||
---|---|---|
80–84 | Just a very minor suggestion, but maybe change this to a link that downloads the log directly? So the user doesn't have to read this and then go find the download button everytime, and instead the action is inline with the call to action. |