Page MenuHomeElementl

add prefix to asset key query
ClosedPublic

Authored by prha on Oct 13 2020, 9:22 PM.

Details

Summary

This will not improve the performance of the root asset catalog, but it will improve the performance of directly linked assets and asset directory paths

Test Plan

loaded direct linked asset page, saw prefix query instead of full asset query

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Oct 13 2020, 9:25 PM
Harbormaster failed remote builds in B19467: Diff 23654!
prha requested review of this revision.Oct 13 2020, 9:43 PM
alangenfeld added inline comments.
python_modules/dagster/dagster/core/storage/event_log/sql_event_log.py
407–415

Screen Shot 2020-10-14 at 2.58.20 PM.png (315×718 px, 35 KB)

This revision now requires changes to proceed.Oct 14 2020, 7:58 PM
prha planned changes to this revision.Oct 14 2020, 8:03 PM

refactor asset UI into root catalog, and a matching page, which first checks for exact asset key matches, and then falls back to a prefix search

This revision is now accepted and ready to land.Oct 15 2020, 2:22 PM
This revision was automatically updated to reflect the committed changes.
prha marked an inline comment as done.