[
https://issues.apache.org/jira/browse/FLINK-26640?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17507145#comment-17507145
]
Gyula Fora commented on FLINK-26640:
------------------------------------
I think using the FlinkVersion enum is a good idea [~twalthr] , for now that we
depend on 1.14 that doesnt have this we could temporarily copy this class into
the operator and remove it once the dependency is bumped.
What do you think?
> Consider changing flinkVersion to enum type or removing it completely
> ---------------------------------------------------------------------
>
> Key: FLINK-26640
> URL: https://issues.apache.org/jira/browse/FLINK-26640
> Project: Flink
> Issue Type: Sub-task
> Reporter: Gyula Fora
> Priority: Major
>
> Currently the flinkVersion is a string field that we do not use anywhere.
> There might be some cases in the future where knowing the flink version might
> be valuable but an optional string field will not work.
> We should either make this a required enum of the supported flink versions (I
> suggest only major.minor -> 1.14, 1.15...) or remove it completely for now.
>
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
