[ 
https://issues.apache.org/jira/browse/FLINK-8054?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Chesnay Schepler updated FLINK-8054:
------------------------------------
    Description: 
We currently have 2 {{Time}} classes in Flink, 
{{org.apache.flink.api.common.time.Time}} in flink-core and 
{{org.apache.flink.streaming.api.windowing.time.Time}} in flink-streaming-java.

These classes are effectively identical (the one in flink-core is serializable 
and has a toString implementation), I suggest to merge them. The streaming Time 
class is part of the Streaming API so this would be a breaking change.

We could also investigate whether the java 8 {{Duration}} class fulfills our 
needs.

  was:
We currently have 2 {{Time}} classes in Flink, 
{{org.apache.flink.api.common.time.Time}} in flink-core and 
{{org.apache.flink.streaming.api.windowing.time.Time}} in flink-streaming-java.

These classes are effectively identical (the one in flink-core is serializable 
and has a toString implementation), I suggest to merge them. The streaming Time 
class is part of the Streaming API so this would be a breaking change.


> Deduplicate Time classes
> ------------------------
>
>                 Key: FLINK-8054
>                 URL: https://issues.apache.org/jira/browse/FLINK-8054
>             Project: Flink
>          Issue Type: Improvement
>          Components: Core, Streaming
>    Affects Versions: 1.4.0
>            Reporter: Chesnay Schepler
>            Priority: Trivial
>             Fix For: 2.0.0
>
>
> We currently have 2 {{Time}} classes in Flink, 
> {{org.apache.flink.api.common.time.Time}} in flink-core and 
> {{org.apache.flink.streaming.api.windowing.time.Time}} in 
> flink-streaming-java.
> These classes are effectively identical (the one in flink-core is 
> serializable and has a toString implementation), I suggest to merge them. The 
> streaming Time class is part of the Streaming API so this would be a breaking 
> change.
> We could also investigate whether the java 8 {{Duration}} class fulfills our 
> needs.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to