tgravescs commented on pull request #28918: URL: https://github.com/apache/spark/pull/28918#issuecomment-649597732
Note, we updated to use local timezone (https://github.com/apache/spark/pull/19640/files#diff-77d1ca904f4bd8645c31f148047d9270L42). the utils.js has functions similar to patch here - formatTimeMillis. The only difference is date vs timeMillis which we might be able to use with just Date.getTime which is what is done by ApplicationAttemptInfo. I'm wondering if we shouldn't so a similar thing here to keep things consistent. Let me investigate a bit more. ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
