henry3260 commented on code in PR #70771:
URL: https://github.com/apache/airflow/pull/70771#discussion_r3946850385
##########
airflow-ctl/src/airflowctl/ctl/help_texts.yaml:
##########
@@ -28,6 +28,11 @@ assets:
delete-queued-events: "Delete all queued events for a given asset"
delete-dag-queued-events: "Delete all queued asset events for a given Dag"
delete-queued-event: "Delete a specific queued asset event for a given Dag
and asset"
+ list-state-store: "List all state store entries for an asset"
+ get-state-store: "Retrieve a single asset state store entry by its ID and
key"
+ set-state-store: "Set an asset state store value. Parsed as JSON if valid,
otherwise stored as a string"
Review Comment:
The `json.loads` fallback turns the literal string `null` into `None`, which
the client-side `AssetStateStoreBody` accepts but the core validator rejects
with a 422 ("value cannot be null"). The CLI surfaces it as a server error and
exits 1, so nothing crashes, but the help text ("Parsed as JSON if valid,
otherwise stored as a string") could note that `null` is not allowed.
--
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]
