feluelle commented on a change in pull request #7148: [AIRFLOW-6472] Correct 
short and long option in cli
URL: https://github.com/apache/airflow/pull/7148#discussion_r385840222
 
 

 ##########
 File path: airflow/models/taskinstance.py
 ##########
 @@ -335,30 +335,46 @@ def generate_command(dag_id,
         :type local: bool
         :param pickle_id: If the DAG was serialized to the DB, the ID
             associated with the pickled DAG
-        :type pickle_id: unicode
+        :type pickle_id: str
         :param file_path: path to the file containing the DAG definition
+        :type file_path: str
         :param raw: raw mode (needs more details)
+        :type raw: bool
         :param job_id: job ID (needs more details)
+        :type job_id: str
         :param pool: the Airflow pool that the task should run in
-        :type pool: unicode
+        :type pool: str
 
 Review comment:
   Same for the other optional fields.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to