Support Airflow 2.x operators (runtime specific components)#3263
Support Airflow 2.x operators (runtime specific components)#3263lresende wants to merge 1 commit into
Conversation
2db5db2 to
9876c89
Compare
| # if "airflow.sensors.base" == node_module and name.name == "BaseSensorOperator": | ||
| # imported_operator_classes.append(name.name) |
Check notice
Code scanning / CodeQL
Commented-out code
Signed-off-by: Luciano Resende <lresende@apple.com>
9876c89 to
4a528d2
Compare
|
Yeah, there are still some changes necessary at airflow_component_parser, though, regarding the properties and values of each airflow operator. Thank you though for making this starter-PR more complete in the other aspects here. I will see what I can find out about the inner workings of component_parser_airflow in conjunction with the new airflow operator classes, like None vs. Empty String for Bash operator cwd property (empty string is wrong ...) ... I just never had time yet to fully debug. Last time I tested, for example the Bash operator I am thinking: there is a test for the kfp component parser but not the airflow component parser ... so maybe it would be good starting a combination of a test for airflow component parser plus modifications to |
What changes were proposed in this pull request?
Support Airflow 2.x operators (runtime specific components)
Fixes #2124
After updating the elyra examples to have Airflow 2.x components, and register that in Elyra, I can see the airflow components and have the properties show up ok on the Pipeline Editor
Developer's Certificate of Origin 1.1