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-5488] stop YarnClient before exception is thrown #3805

Closed
wants to merge 258 commits into from

Conversation

fanyon
Copy link

@fanyon fanyon commented May 2, 2017

No description provided.

StephanEwen and others added 30 commits April 20, 2017 10:52
  - Move new 'Types' class to 'common' scope, not 'java api'.
  - Make the method names the same in Scala and Java
…tingSettings

Cleanup to consistently use
  - checkpoint for the overall fault tolerance mechanism and procedure
  - snapshot for an operators state snapshot that is part of a checkpoint
…r is single use.

This reduces the public API of the StreamingJobGraphGenerator to a static createJobGraph() method,
which internally instantiates a generator, creates the job, and drops the generator.
This more intuitivce name clarifies the role of the class.
This also marks the class as PublicEvolving, as it is not actually used as an internal class.
Properly reset the "pipeline breaker" upon closing.

This closes apache#3747
…stically

Sorts files read from Flink's lib directory and places the distribution
JAR to the end of the CLASSPATH.

This closes apache#3632
This will be re-added when we have a coherent plan for Docker support
and after the official Docker images are available.

This closes apache#3658
… start/stop scripts

This way, if all slaves refer to "localhost", we do not require ssh at all.

This closes apache#3298
Evaluate user-defined Java options immediately before starting services
and rotate all log files.

This closes apache#3701
greghogan and others added 24 commits May 12, 2017 10:11
…e if previous serializer is unavailable

This closes apache#3882.
…Writable hooks

This commit makes the method hooks for defining compatibile
serialization versions of VersionedIOReadableWritables more restricted.

Functionally everything remains the same, but with lesser space for
error-prone user implementations. It also allows for a better error
message to indicate version mismatch.

This closes apache#3883.
…defined

Previously, if a serializer determines that state migration needs to be
performed but could not provide a fallback convert deserializer, it
would use CompatibilityResult.requiresMigration(null).

This commit makes this option more explicit by having a
CompatibilityResult.requiresMigration() option that takes no parameters.
This improves how the user perceives the API without having to rely on
the Javadoc that it is allowed to have no fallback convert deserializer.

Consequently, when using
CompatibilityResult.requiresMigration(TypeDeserializer), the provided
argument cannot be null.

This closes apache#3886.
@tzulitai
Copy link
Contributor

tzulitai commented May 24, 2017

@fanyon could you rebase the actual commit changes of this PR on the latest master?

@FangYongs
Copy link
Contributor

@tzulitai I have make a new PR #4022 for this issue and close this PR soon, thanks :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet