This is an automated email from the ASF dual-hosted git repository. ephraimanierobi pushed a commit to branch v2-4-test in repository https://gitbox.apache.org/repos/asf/airflow.git
commit 332959208ffe9c7043d4756095e34f833ed245e9 Author: blag <[email protected]> AuthorDate: Sat Oct 1 15:16:08 2022 -0700 A few docs fixups (#26788) (cherry picked from commit 674f9ce6eaae533cfe31bc92cc92fa75ed7223fc) --- airflow/providers/microsoft/winrm/hooks/winrm.py | 4 ++-- airflow/providers/ssh/hooks/ssh.py | 4 ++-- docs/apache-airflow-providers-common-sql/operators.rst | 2 ++ docs/apache-airflow/executor/local.rst | 2 +- 4 files changed, 7 insertions(+), 5 deletions(-) diff --git a/airflow/providers/microsoft/winrm/hooks/winrm.py b/airflow/providers/microsoft/winrm/hooks/winrm.py index ee11dbac9c..23d31af59e 100644 --- a/airflow/providers/microsoft/winrm/hooks/winrm.py +++ b/airflow/providers/microsoft/winrm/hooks/winrm.py @@ -40,8 +40,8 @@ class WinRMHook(BaseHook): :seealso: https://github.com/diyan/pywinrm/blob/master/winrm/protocol.py :param ssh_conn_id: connection id from airflow Connections from where - all the required parameters can be fetched like username and password. - Thought the priority is given to the param passed during init + all the required parameters can be fetched like username and password, + though priority is given to the params passed during init. :param endpoint: When not set, endpoint will be constructed like this: 'http://{remote_host}:{remote_port}/wsman' :param remote_host: Remote host to connect to. Ignored if `endpoint` is set. diff --git a/airflow/providers/ssh/hooks/ssh.py b/airflow/providers/ssh/hooks/ssh.py index 17545b1755..5b4170aeca 100644 --- a/airflow/providers/ssh/hooks/ssh.py +++ b/airflow/providers/ssh/hooks/ssh.py @@ -48,8 +48,8 @@ class SSHHook(BaseHook): :param ssh_conn_id: :ref:`ssh connection id<howto/connection:ssh>` from airflow Connections from where all the required parameters can be fetched like - username, password or key_file. Thought the priority is given to the - param passed during init + username, password or key_file, though priority is given to the + params passed during init. :param remote_host: remote host to connect :param username: username to connect to the remote_host :param password: password of the username to connect to the remote_host diff --git a/docs/apache-airflow-providers-common-sql/operators.rst b/docs/apache-airflow-providers-common-sql/operators.rst index 039e10b966..402197fb52 100644 --- a/docs/apache-airflow-providers-common-sql/operators.rst +++ b/docs/apache-airflow-providers-common-sql/operators.rst @@ -49,6 +49,7 @@ mapping is a set of three nested dictionaries and looks like: Where col_name is the name of the column to run checks on, and each entry in its dictionary is a check. The valid checks are: + - null_check: checks the number of NULL values in the column - distinct_check: checks the COUNT of values in the column that are distinct - unique_check: checks the number of distinct values in a column against the number of rows @@ -57,6 +58,7 @@ The valid checks are: Each entry in the check's dictionary is either a condition for success of the check or the tolerance. The conditions for success are: + - greater_than - geq_to - less_than diff --git a/docs/apache-airflow/executor/local.rst b/docs/apache-airflow/executor/local.rst index 6721cf93f1..256c612553 100644 --- a/docs/apache-airflow/executor/local.rst +++ b/docs/apache-airflow/executor/local.rst @@ -43,7 +43,7 @@ The following strategies are implemented: | LocalExecutor receives the call to shutdown the executor a poison token is sent to the | workers to terminate them. Processes used in this strategy are of class :class:`~airflow.executors.local_executor.QueuedLocalWorker`. -Arguably, :class:`~airflow.executors.sequential_executor.SequentialExecutor` could be thought as a ``LocalExecutor`` with limited +Arguably, :class:`~airflow.executors.sequential_executor.SequentialExecutor` could be thought of as a ``LocalExecutor`` with limited parallelism of just 1 worker, i.e. ``self.parallelism = 1``. This option could lead to the unification of the executor implementations, running locally, into just one :class:`~airflow.executors.local_executor.LocalExecutor` with multiple modes.
