feng-tao commented on a change in pull request #2256: [AIRFLOW-1153] Fix issue
' params ' don't pass to HiveOperator execution context
URL: https://github.com/apache/airflow/pull/2256#discussion_r247335514
##########
File path: airflow/utils/operator_helpers.py
##########
@@ -43,7 +43,8 @@ def context_to_airflow_vars(context,
in_env_var_format=False):
:type in_env_var_format: bool
:return task_instance context as dict.
"""
- params = dict()
+ params = context.get('params')
Review comment:
this won't work if the context doesn't specify params wihch params will be
assigned None. You could do something like
params = context.get('params', dict())
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
