dstandish commented on a change in pull request #6886: [AIRFLOW-6327]
http_hook: Accept json= parameter for payload
URL: https://github.com/apache/airflow/pull/6886#discussion_r361250944
##########
File path: airflow/hooks/http_hook.py
##########
@@ -175,7 +181,8 @@ def run_and_check(self, session, prepped_request,
extra_options):
return response
except requests.exceptions.ConnectionError as ex:
- self.log.warning(str(ex) + ' Tenacity will retry to execute the
operation')
+ self.log.warning(
Review comment:
as @zhongjiajie has stated, you are introducing line breaks unnecessarily,
which makes the diff unnecessarily noisy (and the lines were not overly long to
begin with)
currently flake8 is set at 110 characters, so probably best to set it to 110
in your editor, and these extraneous line breaks.
see https://github.com/apache/airflow/blob/master/.flake8#L2
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
