pierrejeambrun commented on code in PR #43783:
URL: https://github.com/apache/airflow/pull/43783#discussion_r1834561105
##########
airflow/api_fastapi/common/parameters.py:
##########
@@ -406,6 +407,37 @@ def depends(self, dag_id: str | None = None) ->
_DagIdFilter:
return self.set_value(dag_id)
+class _UriPatternSearch(_SearchParam):
+ """Search on uri."""
+
+ def __init__(self, skip_none: bool = True) -> None:
+ super().__init__(AssetModel.uri, skip_none)
+
+ def depends(self, uri_pattern: str | None = None) -> _UriPatternSearch:
Review Comment:
Yes my bad, the comment is placed at the wrong position.
This was meant to be on the `dag_ids` line. The `uri_pattern` is correct and
don't need any modification.
But this line
https://github.com/apache/airflow/pull/43783/files#diff-183a76763667e545ddd8c2aadf6858fdd7a80fecdd04b71e61eaa4ff344239cdR427
with the dag_ids need to be converted to a native list. (and we can remove
the string parsing/splitting)
--
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]
