Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FLINK-9121] [flip6] Remove Flip6 prefixes and other references #5801

Closed
wants to merge 2 commits into from

Conversation

tillrohrmann
Copy link
Contributor

What is the purpose of the change

Remove Flip6 prefixes and references to make Flip-6 the proper default:

Rename categories Flip6 -> New and OldAndFlip6 -> LegacyAndNew

Remove Flip-6 from documentation

Remove Flip-6 from start up scripts

Verifying this change

  • Covered by existing tests

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): (yes)
  • 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)
  • The S3 file system connector: (no)

Documentation

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

Remove Flip6 prefixes and references to make Flip-6 the proper default:

Rename categories Flip6 -> New and OldAndFlip6 -> LegacyAndNew

Remove Flip-6 from documentation

Remove Flip-6 from start up scripts
asfgit pushed a commit that referenced this pull request Apr 2, 2018
Remove Flip6 prefixes and references to make Flip-6 the proper default:

Rename categories Flip6 -> New and OldAndFlip6 -> LegacyAndNew

Remove Flip-6 from documentation

Remove Flip-6 from start up scripts

This closes #5801.
@asfgit asfgit closed this in af5279e Apr 2, 2018
@tillrohrmann tillrohrmann deleted the removeFlip6 branch May 29, 2018 10:21
ApplicationReport report,
Configuration flinkConfiguration,
boolean perJobCluster) throws Exception {
return new RestClusterClient<>(
Copy link
Contributor

Choose a reason for hiding this comment

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

why don't return a YarnClusterClient here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because the new Flink architecture will use REST calls for the client-server communication. This is unlike how it was done in the legacy architecture (pre Flip-6).

sampathBhat pushed a commit to sampathBhat/flink that referenced this pull request Jul 26, 2018
Remove Flip6 prefixes and references to make Flip-6 the proper default:

Rename categories Flip6 -> New and OldAndFlip6 -> LegacyAndNew

Remove Flip-6 from documentation

Remove Flip-6 from start up scripts

This closes apache#5801.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants