[AIRFLOW-6510] - Fix Druid operator rendering from file#7102
[AIRFLOW-6510] - Fix Druid operator rendering from file#7102alexnu wants to merge 5 commits intoapache:masterfrom
Conversation
|
Congratulations on your first Pull Request and welcome to the Apache Airflow community! Here are some useful points:
Apache Airflow is a community-driven project and together we are making it better 🚀. In case of doubts contact the developers at: |
26fc25c to
17e0197
Compare
Codecov Report
@@ Coverage Diff @@
## master #7102 +/- ##
=========================================
Coverage ? 84.95%
=========================================
Files ? 682
Lines ? 39043
Branches ? 0
=========================================
Hits ? 33168
Misses ? 5875
Partials ? 0
Continue to review full report at Codecov.
|
Dear maintainers,
Please accept my PR which fixes the way druid specification json is loaded by leveraging template fields as in most other operators, rather that manually parsing the file in the operator.
Fixed existing tests and deleted one unneeded test case, as the operator is no longer opening a file.
Issue link: AIRFLOW-6510
[AIRFLOW-NNNN]. AIRFLOW-NNNN = JIRA ID** For document-only changes commit message can start with
[AIRFLOW-XXXX].In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.
Read the Pull Request Guidelines for more information.