You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@doris.apache.org by GitBox <gi...@apache.org> on 2020/06/11 15:11:42 UTC

[GitHub] [incubator-doris] vagetablechicken commented on pull request #3837: [FE] set timezone of SimpleDateFormat & add broker load log

vagetablechicken commented on pull request #3837:
URL: https://github.com/apache/incubator-doris/pull/3837#issuecomment-642725825


   > hi @vagetablechicken . This PR #3799 is trying to replace the `Calender` with `ZonedDateTime`.
   > So I think some of your change overlap with his work.
   > 
   > Could you help to review his PR to see if it meets your needs?
   
   I just meet the UT failure(issue#3829) when I want to add db&tbl info in broker load log. So I don't replace `Calendar` with `ZonedDateTime`, just add timezone which is easy to change. I'll delete the change about timezone, leave the broker load log.
   


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org