[GitHub] kaxil commented on a change in pull request #4263: [AIRFLOW-XXX] Fix display of SageMaker operators/hook docs

[GitBox <gi...@apache.org>]

kaxil commented on a change in pull request #4263: [AIRFLOW-XXX] Fix display of SageMaker operators/hook docs
URL: https://github.com/apache/incubator-airflow/pull/4263#discussion_r238093908
 
 

 ##########
 File path: airflow/contrib/hooks/sagemaker_hook.py
 ##########
 @@ -253,10 +253,11 @@ def log_stream(self, log_group, stream_name, start_time=0, skip=0):
         :param skip: The number of log entries to skip at the start (default: 0).
             This is for when there are multiple entries at the same timestamp.
         :type skip: int
-        :return:A CloudWatch log event with the following key-value pairs:
-            'timestamp' (int): The time in milliseconds of the event.
-            'message' (str): The log event data.
-            'ingestionTime' (int): The time in milliseconds the event was ingested.
+        :rtype: dict
+        :return: | A CloudWatch log event with the following key-value pairs:
+                 |   'timestamp' (int): The time in milliseconds of the event.
 
 Review comment:
   what does the pipe (`|`) do here?

----------------------------------------------------------------
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