Skip to content

KAFKA-SITE DOCS 8227 - Add missing links Core Concepts duality of streams tables#204

Merged
bbejeck merged 8 commits into
apache:asf-sitefrom
londoncalling:KAFKA-8227-missing-links-duality-of-streams-tables
Apr 24, 2019
Merged

KAFKA-SITE DOCS 8227 - Add missing links Core Concepts duality of streams tables#204
bbejeck merged 8 commits into
apache:asf-sitefrom
londoncalling:KAFKA-8227-missing-links-duality-of-streams-tables

Conversation

@londoncalling

@londoncalling londoncalling commented Apr 23, 2019

Copy link
Copy Markdown

Description

Review Requests

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

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

Signed-off-by: Victoria Bialas <vicky@confluent.io>
Signed-off-by: Victoria Bialas <vicky@confluent.io>
Signed-off-by: Victoria Bialas <vicky@confluent.io>
Signed-off-by: Victoria Bialas <vicky@confluent.io>
Signed-off-by: Victoria Bialas <vicky@confluent.io>
Signed-off-by: Victoria Bialas <vicky@confluent.io>
Signed-off-by: Victoria Bialas <vicky@confluent.io>
@londoncalling londoncalling changed the title KAFKA-8227 - Add missing links Core Concepts duality of streams tables (WIP) KAFKA-SITE DOCS 8227 - Add missing links Core Concepts duality of streams tables (WIP) Apr 24, 2019
@londoncalling londoncalling changed the title KAFKA-SITE DOCS 8227 - Add missing links Core Concepts duality of streams tables (WIP) KAFKA-SITE DOCS 8227 - Add missing links Core Concepts duality of streams tables Apr 24, 2019

@bbejeck bbejeck left a comment

Copy link
Copy Markdown
Member

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 20/streams/core-concepts.html
Comment thread 21/streams/core-concepts.html Outdated
Comment thread 22/streams/core-concepts.html Outdated
Signed-off-by: Victoria Bialas <vicky@confluent.io>
@londoncalling

Copy link
Copy Markdown
Author

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

@joel-hamill

Copy link
Copy Markdown

I spot checked a handful of the links locally and they looked good.

@bbejeck
bbejeck merged commit 00e2637 into apache:asf-site Apr 24, 2019
@bbejeck

bbejeck commented Apr 24, 2019

Copy link
Copy Markdown
Member

Merged #204 into asf-site

@bbejeck

bbejeck commented Apr 24, 2019

Copy link
Copy Markdown
Member

Thanks for all the work @londoncalling!

@londoncalling
londoncalling deleted the KAFKA-8227-missing-links-duality-of-streams-tables branch June 4, 2019 01:15
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