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

Xin Hao commented on FLINK-27277:
---------------------------------

Thanks for the explanation. Maybe we should consider explicit mark the 
difference?

 

 
{code:java}
spec:
  clusterId: session-cluster-test
  job:
    state: running
    ...
status:
  state: running   
  flinkJobStatus:
    state: RUNNING {code}
I think the users can figure out the difference if they look at the info 
carefully (Maybe thinking for seconds).
But sometimes it's a bit confusing when they take the first glance. 
also, the current state should be in the status spec.

 

> 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)

Reply via email to