[jira] [Commented] (FLINK-27277) consist the naming style of SessionJob's state field

["Gyula Fora (Jira)" <ji...@apache.org>]

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

Gyula Fora commented on FLINK-27277:
------------------------------------

We could store the jobstatus/state field in lowercase if that makes it look nicer but those 2 fields refer to 2 different things.

spec/job/state refers to the application/sessionjob deployment state which can be either running/suspended
jobstatus/state refers to the actual flink jobState which can be a number of different thins like starting/running/failing/restarting/cancelled etc.

> consist the naming style of SessionJob's state field
> ----------------------------------------------------
>
>                 Key: FLINK-27277
>                 URL: https://issues.apache.org/jira/browse/FLINK-27277
>             Project: Flink
>          Issue Type: Improvement
>          Components: Kubernetes Operator
>            Reporter: Xin Hao
>            Priority: Minor
>
> {code:java}
> spec:
>   clusterId: session-cluster-test
>   job:
>     state: running
>     ...
> status:
>   jobStatus:
>     state: RUNNING
>     ...{code}
> I think it's better to change the spec.job.state with upper case to consist of the field in the status spec.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)