You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2019/02/01 01:11:27 UTC
[GitHub] patrickmckenna commented on issue #2206: [AIRFLOW-922] Update
PrestoHook to enable synchronous execution
patrickmckenna commented on issue #2206: [AIRFLOW-922] Update PrestoHook to enable synchronous execution
URL: https://github.com/apache/airflow/pull/2206#issuecomment-459568693
Apologies for my slow reply! All of the currently failing builds show the same 2 errors, which appear unrelated to this PR:
<details>
<summary> failing tests </summary>
```
======================================================================
84) FAIL: test_scheduler_sla_miss_callback_exception (tests.test_jobs.SchedulerJobTest)
----------------------------------------------------------------------
Traceback (most recent call last):
tests/test_jobs.py line 3025 in test_scheduler_sla_miss_callback_exception
sla_callback.assert_called()
.tox/py35-backend_postgres-env_docker/lib/python3.5/site-packages/mock/mock.py line 906 in assert_called
raise AssertionError(msg)
AssertionError: Expected 'None' to have been called.
-------------------- >> begin captured stdout << ---------------------
[2019-02-01 00:36:45,730] {test_task_view_type_check.py:52} INFO - class_instance type: <class 'unusual_prefix_61c0ab525b75d060ea41c0aa11a98c88efc72c26_test_task_view_type_check.CallableClass'>
--------------------- >> end captured stdout << ----------------------
-------------------- >> begin captured logging << --------------------
root: INFO: class_instance type: <class 'unusual_prefix_61c0ab525b75d060ea41c0aa11a98c88efc72c26_test_task_view_type_check.CallableClass'>
--------------------- >> end captured logging << ---------------------
======================================================================
85) FAIL: test_scheduler_sla_miss_email_exception (tests.test_jobs.SchedulerJobTest)
----------------------------------------------------------------------
Traceback (most recent call last):
/usr/lib/python3.5/unittest/mock.py line 1157 in patched
return func(*args, **keywargs)
tests/test_jobs.py line 3069 in test_scheduler_sla_miss_email_exception
'test_sla_miss')
.tox/py35-backend_postgres-env_docker/lib/python3.5/site-packages/mock/mock.py line 925 in assert_called_with
raise AssertionError('Expected call: %s\nNot called' % (expected,))
AssertionError: Expected call: exception('Could not send SLA Miss email notification for DAG %s', 'test_sla_miss')
Not called
-------------------- >> begin captured stdout << ---------------------
[2019-02-01 00:36:45,820] {test_task_view_type_check.py:52} INFO - class_instance type: <class 'unusual_prefix_61c0ab525b75d060ea41c0aa11a98c88efc72c26_test_task_view_type_check.CallableClass'>
--------------------- >> end captured stdout << ----------------------
-------------------- >> begin captured logging << --------------------
root: INFO: class_instance type: <class 'unusual_prefix_61c0ab525b75d060ea41c0aa11a98c88efc72c26_test_task_view_type_check.CallableClass'>
--------------------- >> end captured logging << ---------------------
```
</details><br>
I'm not sure how to understand this (and am all the more confused because the [latest build](https://travis-ci.org/apache/airflow/builds/487111076) of `apache/master` is passing). I'll try rebasing this branch to generate the same tree as the [currently failing build](https://travis-ci.org/apache/airflow/builds/487203679) but trigger a new test...
----------------------------------------------------------------
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:
users@infra.apache.org
With regards,
Apache Git Services