You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by tu...@apache.org on 2020/10/11 16:07:04 UTC
[airflow] branch master updated: Fix spelling in CeleryExecutor
(#11407)
This is an automated email from the ASF dual-hosted git repository.
turbaszek pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/master by this push:
new b786327 Fix spelling in CeleryExecutor (#11407)
b786327 is described below
commit b78632704124930eaf883142a0045a1ed348911c
Author: John Bampton <jb...@users.noreply.github.com>
AuthorDate: Mon Oct 12 02:06:26 2020 +1000
Fix spelling in CeleryExecutor (#11407)
---
airflow/executors/celery_executor.py | 4 ++--
tests/executors/test_celery_executor.py | 2 +-
tests/utils/test_dag_processing.py | 2 +-
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/airflow/executors/celery_executor.py b/airflow/executors/celery_executor.py
index f77ea91..1bed9b1 100644
--- a/airflow/executors/celery_executor.py
+++ b/airflow/executors/celery_executor.py
@@ -82,7 +82,7 @@ def execute_command(command_to_exec: CommandType) -> None:
log.info("Executing command in Celery: %s", command_to_exec)
if settings.EXECUTE_TASKS_NEW_PYTHON_INTERPRETER:
- _execute_in_subprocees(command_to_exec)
+ _execute_in_subprocess(command_to_exec)
else:
_execute_in_fork(command_to_exec)
@@ -118,7 +118,7 @@ def _execute_in_fork(command_to_exec: CommandType) -> None:
os._exit(ret) # pylint: disable=protected-access
-def _execute_in_subprocees(command_to_exec: CommandType) -> None:
+def _execute_in_subprocess(command_to_exec: CommandType) -> None:
env = os.environ.copy()
try:
# pylint: disable=unexpected-keyword-arg
diff --git a/tests/executors/test_celery_executor.py b/tests/executors/test_celery_executor.py
index fcf0e36..c53b0e0 100644
--- a/tests/executors/test_celery_executor.py
+++ b/tests/executors/test_celery_executor.py
@@ -227,7 +227,7 @@ class TestCeleryExecutor(unittest.TestCase):
))
def test_command_validation(self, command, expected_exception):
# Check that we validate _on the receiving_ side, not just sending side
- with mock.patch('airflow.executors.celery_executor._execute_in_subprocees') as mock_subproc, \
+ with mock.patch('airflow.executors.celery_executor._execute_in_subprocess') as mock_subproc, \
mock.patch('airflow.executors.celery_executor._execute_in_fork') as mock_fork:
if expected_exception:
with pytest.raises(expected_exception):
diff --git a/tests/utils/test_dag_processing.py b/tests/utils/test_dag_processing.py
index cc15143..26a441f 100644
--- a/tests/utils/test_dag_processing.py
+++ b/tests/utils/test_dag_processing.py
@@ -247,7 +247,7 @@ class TestDagFileProcessorManager(unittest.TestCase):
session.add(local_job)
session.commit()
- # TODO: If there was an actual Relationshop between TI and Job
+ # TODO: If there was an actual Relationship between TI and Job
# we wouldn't need this extra commit
session.add(ti)
ti.job_id = local_job.id