potiuk commented on a change in pull request #8994:
URL: https://github.com/apache/airflow/pull/8994#discussion_r429754433
##########
File path: airflow/config_templates/config.yml
##########
@@ -672,7 +672,7 @@
version_added: ~
type: string
example: ~
- default: "Airflow HiveOperator task for
{{hostname}}.{{dag_id}}.{{task_id}}.{{execution_date}}"
Review comment:
Yeahj. I was strugling with this one. Hive operator is wrongly written.
It expect the configuration to contain {} references to parameters it passes,
but when airflow reads the values from config, replaces it from environment
variables (which are missing). So basically setting configuration for hive
operator is wrong right now.
The problem was that Hive Operator when imported tried to read that
configuration and since it was missing, it failed. I think I have it right
finally (fallback in the operator but no defaults in .yaml/cfg files). Let's
see how the tests work it out. Eventually we might want to fix the operator.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
