codenamestif opened a new pull request #15173:
URL: https://github.com/apache/airflow/pull/15173


   Related: #15144. That's my first pr. Here is the main idea of the feature. 
The handler reads the timestamp of Cloudwatch events. So timestamp can be seen 
on the `Log` page of the task. Also I wanted to add a support of 
`DEFAULT_UI_TIMEZONE` here. Basically to format UTC timestamp with respect to 
specified UI timezone. However i found only the code that resolves 
`DEFAULT_TIMEZONE`. So i used it for now. Also i'm making `event_dt` as naive 
right before formatting in order to remove timezone information in logs. 
   
   So my main question is is it fine to use `DEFAULT_TIMEZONE` for formatting 
of the timestamps? Or there should be similar resolver for 
`DEFAULT_UI_TIMEZONE` and timestamps are formatted with respect to specified ui 
timezone?


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


Reply via email to