Skip to content

Conversation

@Fokko
Copy link
Contributor

@Fokko Fokko commented Jan 3, 2019

The patched version provides some hotfixes:
https://zookeeper.apache.org/doc/r3.4.13/releasenotes.html

What is the purpose of the change

(For example: This pull request makes task deployment go through the blob server, rather than through RPC. That way we avoid re-transferring them on each deployment (during recovery).)

Brief change log

Bump Zookeeper dependency

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.

  • Dependencies (does it add or upgrade a dependency): yes
  • 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: yes
  • 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

@Fokko
Copy link
Contributor Author

Fokko commented Jan 4, 2019

The error looks related, let me dive into it.

@Fokko
Copy link
Contributor Author

Fokko commented Jan 15, 2019

Kafka09SecuredRunITCase is failing. Let's see which minor patch of Zookeeper fails the build.

@zentol
Copy link
Contributor

zentol commented Jan 16, 2019

Please don't use the Apache Travis for debugging. I will close this PR; feel free to re-open once you got a working solution.

@thomaslee
Copy link

Late to the party, but pointing out the test failure in question appears to be a ZK auth issue:

https://travis-ci.org/apache/flink/jobs/480142816#L9784

Possible something subtle changed between .10 and .13 but I'm not sure what at this point.

@thomaslee
Copy link

Comments on https://issues.apache.org/jira/browse/HADOOP-15974 sound awfully similar to the issue we encountered here, but sounds more involved than a simple auth issue.

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.

4 participants