-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
fix(ingestion/airflow-plugin): warning log for non-materialized iolets #10421
fix(ingestion/airflow-plugin): warning log for non-materialized iolets #10421
Conversation
5a18105
to
efb5e6d
Compare
d7a9823
to
c99b5b7
Compare
…terialize_iolets is false
c99b5b7
to
6fcfe44
Compare
clone_inlets=False if config and not config.materialize_iolets else True, | ||
clone_outlets=False if config and not config.materialize_iolets else True, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These could just be config.materialize_iolets
in both cases. That is a boolean, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, but implemented this way as there might be a case where config
is not having materialize_iolets
as config
is of type Optional[DatahubLineageConfig]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use config and not config.materialize_iolets
instead
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is generating linter error
src/datahub_airflow_plugin/client/airflow_generator.py:431: error: Argument "clone_inlets" to "from_datajob" of "DataProcessInstance" has incompatible type "Union[DatahubLineageConfig, None, bool]"; expected "bool" [arg-type]
used config is None or config.materialize_iolets,
instead
This would cause it to not be respected for data process instances. Also fixes a bug from #10421.
Checklist