Skip to content

KAFKA-8227 DOCS Fixed missing links duality of streams tables#6625

Merged
bbejeck merged 2 commits intoapache:trunkfrom
londoncalling:kafka-docs-8227-missing-links-duality-of-streams-tables
Apr 24, 2019
Merged

KAFKA-8227 DOCS Fixed missing links duality of streams tables#6625
bbejeck merged 2 commits intoapache:trunkfrom
londoncalling:kafka-docs-8227-missing-links-duality-of-streams-tables

Conversation

@londoncalling
Copy link
Copy Markdown
Contributor

@londoncalling londoncalling commented Apr 24, 2019

Description

Review Requests

@bbejeck , @mjsax , @JimGalasyn , @joel-hamill

Signed-off-by: Victoria Bialas vicky@confluent.io

Signed-off-by: Victoria Bialas <vicky@confluent.io>
Copy link
Copy Markdown
Member

@bbejeck bbejeck left a comment

Choose a reason for hiding this comment

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

@londoncalling thanks for the PR. I just have some minor comments, but otherwise this PR LGTM.

Comment thread docs/streams/core-concepts.html Outdated
Signed-off-by: Victoria Bialas <vicky@confluent.io>
@londoncalling
Copy link
Copy Markdown
Contributor Author

@mjsax , @joel-hamill , @JimGalasyn this should be good to go after build completes, can someone merge for me who has permissions?

Copy link
Copy Markdown
Member

@bbejeck bbejeck left a comment

Choose a reason for hiding this comment

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

Thanks for the cleanup @londoncalling! LGTM

@bbejeck bbejeck merged commit dd81314 into apache:trunk Apr 24, 2019
@bbejeck
Copy link
Copy Markdown
Member

bbejeck commented Apr 24, 2019

Merged #6625 into trunk

@bbejeck
Copy link
Copy Markdown
Member

bbejeck commented Apr 24, 2019

Thanks for fixing the links @londoncalling!

bbejeck pushed a commit that referenced this pull request Apr 24, 2019
Fixed missing links duality of streams tables

Reviewers: Jim Galasyn <jim.galasyn@confluent.io> Bill Bejeck <bbejeck@gmail.com>
bbejeck pushed a commit that referenced this pull request Apr 24, 2019
Fixed missing links duality of streams tables

Reviewers: Jim Galasyn <jim.galasyn@confluent.io> Bill Bejeck <bbejeck@gmail.com>
bbejeck pushed a commit that referenced this pull request Apr 24, 2019
Fixed missing links duality of streams tables

Reviewers: Jim Galasyn <jim.galasyn@confluent.io> Bill Bejeck <bbejeck@gmail.com>
@bbejeck
Copy link
Copy Markdown
Member

bbejeck commented Apr 24, 2019

cherry-picked to 2.2, 2.1, and 2.0

dhruvilshah3 added a commit to confluentinc/kafka that referenced this pull request Apr 29, 2019
* ak/trunk: (42 commits)
  KAFKA-8134: `linger.ms` must be a long
  KAFKA-7779; Avoid unnecessary loop iteration in leastLoadedNode (apache#6081)
  MINOR: Update Gradle to 5.4.1 and update its plugins  (apache#6436)
  MINOR: improve Session expiration notice (apache#6618)
  KAFKA-8029: In memory session store (apache#6525)
  MINOR: In-memory stores cleanup (apache#6595)
  KAFKA-7862 & KIP-345 part-one: Add static membership logic to JoinGroup protocol (apache#6177)
  KAFKA-8254: Pass Changelog as Topic in Suppress Serdes (apache#6602)
  KAFKA-7903: automatically generate OffsetCommitRequest (apache#6583)
  KAFKA-8291 : System test fix (apache#6637)
  MINOR: Do not log retriable offset commit exceptions as errors (apache#5904)
  MINOR: Fix log message error of loadTransactionMetadata (apache#6571)
  MINOR: Fix 404 security features links (apache#6634)
  MINOR: Remove an unnecessary character from broker's startup log
  MINOR: Make LogCleaner.shouldRetainRecord more readable (apache#6590)
  MINOR: Remove implicit return statement (apache#6629)
  KAFKA-8237; Untangle TopicDeleteManager and add test cases (apache#6588)
  KAFKA-8227 DOCS Fixed missing links duality of streams tables (apache#6625)
  MINOR: reformat settings.gradle to be more readable (apache#6621)
  MINOR: Correct RestServerTest formatting
  ...

 Conflicts:
	build.gradle
	settings.gradle
pengxiaolong pushed a commit to pengxiaolong/kafka that referenced this pull request Jun 14, 2019
…#6625)

Fixed missing links duality of streams tables

Reviewers: Jim Galasyn <jim.galasyn@confluent.io> Bill Bejeck <bbejeck@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants