uranusjr commented on code in PR #50117:
URL: https://github.com/apache/airflow/pull/50117#discussion_r2111078514
##########
task-sdk/src/airflow/sdk/api/client.py:
##########
@@ -470,7 +471,27 @@ def get_sequence_item(
},
)
raise
- return XComResponse.model_validate_json(resp.read())
+ return XComSequenceIndexResponse.model_validate_json(resp.read())
+
+ def get_sequence_slice(
+ self,
+ dag_id: str,
+ run_id: str,
+ task_id: str,
+ key: str,
+ start: int | None,
+ stop: int | None,
+ step: int | None,
Review Comment:
These values are parsed from a `slice` object and passed in as variables
anyway. Defaults aren’t useful, at least for what we use the API in this PR.
https://github.com/apache/airflow/pull/50117/files#diff-b16266575e3be5e029fe7cbe66e3c510e87e2a5b64db95c6c702b96898770e74R129-R143
--
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]
