AK to CCS merge 10-08-2020#425
Merged
Matthew Wong (mattwong949) merged 7 commits intoOct 13, 2020
Merged
Conversation
Implements KIP-617 for `SessionStore` Reviewers: A. Sophie Blee-Goldman <sophie@confluent.io>, John Roesler <vvcephei@apache.org>
Reviewers: Mickael Maison <mickael.maison@gmail.com>, Lee Dongjin <dongjin@apache.org>
Reviewers: Mickael Maison <mickael.maison@gmail.com>, Lee Dongjin <dongjin@apache.org>
…9393) In this PR, I have addressed the review comments from @chia7712 in #9001 which were provided after #9001 was merged. The changes are made mainly to KafkaAdminClient: Improve error message in updateFeatures api when feature name is empty. Propagate top-level error message in updateFeatures api. Add an empty-parameter variety for describeFeatures api. Minor documentation updates to @param and @return to make these resemble other apis. Reviewers: Chia-Ping Tsai chia7712@gmail.com, Jun Rao junrao@gmail.com
…e partition (#9020) StreamThreadStateStoreProvider excessive loop over calling internalTopologyBuilder.topicGroups(), which is synchronized, thus causing significant performance degradation to the caller, especially when store has many partitions. Reviewers: John Roesler <vvcephei@apache.org>, Guozhang Wang <wangguoz@gmail.com>
Author
|
ping one of Bob Barrett (@bob-barrett) Ismael Juma (@ijuma) for review. |
Bill Bejeck (bbejeck)
left a comment
Author
There was a problem hiding this comment.
I've added comments on the files that are related to the 2.7 release. Everything else is just a merge from AK and probably doesn't require reviewing.
| "version": "27", | ||
| "dotVersion": "2.7", | ||
| "fullDotVersion": "2.7.0", | ||
| "version": "28", |
Author
There was a problem hiding this comment.
updates for the new release
| # - tests/kafkatest/version.py (variable DEV_VERSION) | ||
| # - kafka-merge-pr.py | ||
| version=6.1.0-0-ccs | ||
|
|
Author
There was a problem hiding this comment.
updates for the 2.7 release
| DEV_BRANCH_NAME = "trunk" | ||
|
|
||
| DEFAULT_FIX_VERSION = os.environ.get("DEFAULT_FIX_VERSION", "2.7.0") | ||
| DEFAULT_FIX_VERSION = os.environ.get("DEFAULT_FIX_VERSION", "2.8.0") |
| <groupId>org.apache.kafka</groupId> | ||
| <artifactId>streams-quickstart</artifactId> | ||
| <version>6.1.0-0-ccs</version> | ||
| <version>6.2.0-0-ccs</version> |
| <properties> | ||
| <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding> | ||
| <kafka.version>6.1.0-0-ccs</kafka.version> | ||
| <kafka.version>6.2.0-0-ccs</kafka.version> |
| <artifactId>streams-quickstart</artifactId> | ||
| <packaging>pom</packaging> | ||
| <version>6.1.0-0-ccs</version> | ||
| <version>6.2.0-0-ccs</version> |
|
|
||
| # For example, when Kafka is at version 1.0.0-0, this should be something like "1.0.0-0.dev0" | ||
| __version__ = '6.1.0-0.dev0' | ||
| __version__ = '6.2.0-0.dev0' |
Author
There was a problem hiding this comment.
more updates for the new release
|
|
||
| DEV_BRANCH = KafkaVersion("dev") | ||
| DEV_VERSION = KafkaVersion("6.1.0-0") | ||
| DEV_VERSION = KafkaVersion("6.2.0-0") |
Author
There was a problem hiding this comment.
release related update
Bob Barrett (bob-barrett)
approved these changes
Oct 13, 2020
Bob Barrett (bob-barrett)
left a comment
Member
There was a problem hiding this comment.
Checked that the version bump changes LGTM. As you suggested, I didn't review the rest of the changes since they're an upstream merge.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR includes the commits related to AK 2.7 branch cutting and subsequent commits afterward
Committer Checklist (excluded from commit message)