[ 
https://issues.apache.org/jira/browse/FLINK-5431?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15856058#comment-15856058
 ] 

Till Rohrmann commented on FLINK-5431:
--------------------------------------

I think it would be a good start to introduce a new configuration option which 
can be set via {{flink-conf.yaml}}. Would be great to directly use the 
{{ConfigOption}} variant. Take a look at {{HighAvailabilityOptions}} to see how 
to add a new config option.

Maybe we could call the configuration parameter something like 
{{client.logging.time-pattern}}.

Additionally I think we should remove the actual formatting logic from the 
{{ExecutionStateChanged}} class and do it wherever we want to print the 
respective information. In our case it would be the {{JobClientActor}}.

> time format for akka status
> ---------------------------
>
>                 Key: FLINK-5431
>                 URL: https://issues.apache.org/jira/browse/FLINK-5431
>             Project: Flink
>          Issue Type: Improvement
>          Components: Distributed Coordination
>            Reporter: Alexey Diomin
>            Assignee: Anton Solovev
>            Priority: Minor
>
> In ExecutionGraphMessages we have code
> {code}
> private val DATE_FORMATTER: SimpleDateFormat = new 
> SimpleDateFormat("MM/dd/yyyy HH:mm:ss")
> {code}
> But sometimes it cause confusion when main logger configured with 
> "dd/MM/yyyy".
> We need making this format configurable or maybe stay only "HH:mm:ss" for 
> prevent misunderstanding output date-time



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to