Skip to content

[FLINK-7653] Properly implement Dispatcher#requestClusterOverview#4793

Closed
tillrohrmann wants to merge 1 commit intoapache:masterfrom
tillrohrmann:fixRequestStatusOverview
Closed

[FLINK-7653] Properly implement Dispatcher#requestClusterOverview#4793
tillrohrmann wants to merge 1 commit intoapache:masterfrom
tillrohrmann:fixRequestStatusOverview

Conversation

@tillrohrmann
Copy link
Copy Markdown
Contributor

What is the purpose of the change

This commit implements the ClusterOverview generation on the Dispatcher. In
order to do this, the Dispatcher requests the ResourceOverview from the
ResourceManager and the job status from all JobMasters. After receiving all
information, it is compiled into the ClusterOverview.

Note: StatusOverview has been renamed to ClusterOverview

Verifying this change

Tested the changes manually.

Does this pull request potentially affect one of the following parts:

  • Dependencies (does it add or upgrade a dependency): (no)
  • The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
  • The serializers: (no)
  • The runtime per-record code paths (performance sensitive): (no)
  • Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)

Documentation

  • Does this pull request introduce a new feature? (no)
  • If yes, how is the feature documented? (not applicable)

Copy link
Copy Markdown
Contributor

@zentol zentol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@tillrohrmann
Copy link
Copy Markdown
Contributor Author

Thanks for the review @zentol. Rebasing onto the latest master and merging after Travis gives green light.

@tillrohrmann tillrohrmann force-pushed the fixRequestStatusOverview branch from 6b7aec2 to 881ea78 Compare October 11, 2017 16:08
This commit implements the ClusterOverview generation on the Dispatcher. In
order to do this, the Dispatcher requests the ResourceOverview from the
ResourceManager and the job status from all JobMasters. After receiving all
information, it is compiled into the ClusterOverview.

Note: StatusOverview has been renamed to ClusterOverview
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants