Skip to content

[Bug] FlinkTrackController's trackIds, canceling, jobStatuses uniformly use appId as Key #1666

Open
@xinzhuxiansheng

Description

@xinzhuxiansheng

Search before asking

  • I had searched in the issues and found no similar issues.

What happened

FlinkTrackController's trackIds, canceling, endpoints, jobStatuses uniformly use appId as Key.

FlinkTrackController
图片

trackIds,canceling use appId as key
图片

jobStatuses use TrackId as key
图片

StreamPark Version

1.2.4-SNAPSHOT

Java Version

No response

Flink Version

1.14.5

Scala Version of Flink

2.12

Error Exception

Only one job is run, and the size of the jobStatuses cache is 2

To explain its impact, `FlinkJobStatusWatcher.touchApplicationJob()`  When jobDetails.isEmpty || jobDetails.get.jobs.isEmpty condition is true, inferApplicationFlinkJobStateFromK8sEvent() will change from 
`val latest: JobStatusCV = trackController.jobStatuses.get(trackId)`
... 

Screenshots

图片

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

  • I agree to follow this project's Code of Conduct

Activity

wolfboys

wolfboys commented on Sep 23, 2022

@wolfboys
Member

Thanks for your feedback, really a bug. welcome to fix it.

changed the title [Bug] FlinkTrackController's trackIds, canceling, endpoints, jobStatuses uniformly use appId as Key [Bug] FlinkTrackController's trackIds, canceling, jobStatuses uniformly use appId as Key on Sep 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

bugSomething isn't workingfeature/wipThis feature is in WIP

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Participants

@xinzhuxiansheng@wolfboys

Issue actions

    [Bug] FlinkTrackController's trackIds, canceling, jobStatuses uniformly use appId as Key · Issue #1666 · apache/streampark