```

    def check_status(self, non_terminal_states,
                     failed_state, key,
                     describe_function, *args):
        """
        :param non_terminal_states: the set of non_terminal states
        :type non_terminal_states: dict
        :param failed_state: the set of failed states
        :type failed_state: dict
        :param key: the key of the response dict
        that points to the state
        :type key: string
        :param describe_function: the function used to retrieve the status
        :type describe_function: python callable
        :param args: the arguments for the function
        :return: None
        """
```

The `non_terminal_states` and `failed_state` should be `dict`'s according to 
the docs. Can we make these one `set`'s as well? `{'InProgress', 'Stopping', 
'Stopped'}`

[ Full content available at: 
https://github.com/apache/incubator-airflow/pull/3767 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to