Page MenuHomeElementl

[dagit] Assets: Only show latest materialization in list
ClosedPublic

Authored by dish on Apr 16 2021, 8:00 PM.

Details

Summary

When viewing a list of an asset's partitioned materializations, bucket them by partition and only show the most recent one in the table. If there are predecessor materializations (by timestamp), link to a dialog that displays these in the same style of table.

Additionally:

  • Put the partition name in the leftmost column, as it is the most key-like value.
  • Sort the table by partition name, with partition-less materializations at the bottom.
Test Plan

View an asset, switch to the list view of materializations. For partitions (or non-partiioned materializations) with predecessors, verify correct rendering of ButtonLink and its dialog.

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dish requested review of this revision.Apr 16 2021, 8:07 PM

This seems like a good workflow improvement! Certainly makes the list easier to scan through and emphasizes that the graphs, etc. also only show the most recent run for the partition.

js_modules/dagit/packages/core/src/assets/AssetMaterializationTable.tsx
20

I feel like this may not need to be both optional and also | null?

This revision is now accepted and ready to land.Apr 16 2021, 8:46 PM