-
Notifications
You must be signed in to change notification settings - Fork 1.9k
[pac] Add get_asset_check_partition_info storage method #33242
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
OwenKephart
wants to merge
1
commit into
01-09-_pac_update_user-facing_apis_and_snapshots
Choose a base branch
from
01-09-_pac_add_get_asset_check_partitions_storage_method
base: 01-09-_pac_update_user-facing_apis_and_snapshots
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[pac] Add get_asset_check_partition_info storage method #33242
OwenKephart
wants to merge
1
commit into
01-09-_pac_update_user-facing_apis_and_snapshots
from
01-09-_pac_add_get_asset_check_partitions_storage_method
+334
−2
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
This was referenced Jan 12, 2026
d8441d5 to
fb873cd
Compare
323aaa3 to
c752369
Compare
This was referenced Jan 14, 2026
c752369 to
35d9920
Compare
fb873cd to
d619bd8
Compare
35d9920 to
fc68c31
Compare
d619bd8 to
1306e55
Compare
fc68c31 to
e438462
Compare
2a8b764 to
da88a16
Compare
e438462 to
fccb6fe
Compare
da88a16 to
3fd4bc5
Compare
94fb89a to
b01a20b
Compare
3fd4bc5 to
1ef4712
Compare
b01a20b to
4a49aff
Compare
1ef4712 to
ef7199b
Compare
4a49aff to
7577749
Compare
ef7199b to
5bcf6e9
Compare
7577749 to
243e63c
Compare
243e63c to
a3d5b7e
Compare
5bcf6e9 to
0bf6f2e
Compare
a3d5b7e to
3055488
Compare
0bf6f2e to
a2c10b7
Compare
3055488 to
a8e5728
Compare
gibsondan
approved these changes
Jan 20, 2026
python_modules/dagster/dagster/_core/storage/asset_check_execution_record.py
Outdated
Show resolved
Hide resolved
python_modules/dagster/dagster/_core/storage/asset_check_execution_record.py
Show resolved
Hide resolved
python_modules/dagster/dagster/_core/storage/asset_check_execution_record.py
Outdated
Show resolved
Hide resolved
python_modules/dagster/dagster/_core/storage/asset_check_execution_record.py
Outdated
Show resolved
Hide resolved
a8e5728 to
da542f8
Compare
a2c10b7 to
2e7bcb0
Compare
da542f8 to
fdff5e0
Compare
2e7bcb0 to
5bc1fd2
Compare
0f52d09 to
f645bf8
Compare
5bc1fd2 to
970a32f
Compare
970a32f to
27b0720
Compare
58cd914 to
137c25d
Compare
27b0720 to
790a12b
Compare
137c25d to
8a6b75f
Compare
8a6b75f to
a9ee90b
Compare
790a12b to
e2c8637
Compare
a9ee90b to
707bd1b
Compare
707bd1b to
888c311
Compare
e2c8637 to
7a80952
Compare
7a80952 to
69716e5
Compare
888c311 to
1bd7f5d
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.

Summary & Motivation
This PR adds a new storage method for fetching a summary record about the status of multiple partitions (of multiple checks) at once.
The current implementation relies on a series of explicit queries against the asset check executions table, but one could imagine a world in which a separate summary table was created and updated on event store, which would result in faster queries.
How I Tested These Changes
Changelog
NOCHANGELOG