Skip to content

[SPARK-16312][STREAMING][KAFKA][DOC] Doc for Kafka 0.10 integration #14385

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

Closed
wants to merge 1 commit into from

Conversation

koeninger
Copy link
Contributor

What changes were proposed in this pull request?

Doc for the Kafka 0.10 integration

How was this patch tested?

Scala code examples were taken from my example repo, so hopefully they compile.

@SparkQA
Copy link

SparkQA commented Jul 28, 2016

Test build #62953 has finished for PR 14385 at commit c44611d.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@koeninger
Copy link
Contributor Author

@tdas @srowen @zsxwing Anyone have time to look at this? Because of the issues building scaladoc for the 0.10 integration, it seems like it'd be good to have at least some kind of documentation available.

@srowen
Copy link
Member

srowen commented Aug 1, 2016

Purely because some doc is better than none, I'd probably merge this. I also imagine you're pretty expert on this part so think the docs look OK to me even with a quick look.

@srowen
Copy link
Member

srowen commented Aug 5, 2016

Merged to master/2.0

@asfgit asfgit closed this in c9f2501 Aug 5, 2016
asfgit pushed a commit that referenced this pull request Aug 5, 2016
## What changes were proposed in this pull request?
Doc for the Kafka 0.10 integration

## How was this patch tested?
Scala code examples were taken from my example repo, so hopefully they compile.

Author: cody koeninger <cody@koeninger.org>

Closes #14385 from koeninger/SPARK-16312.

(cherry picked from commit c9f2501)
Signed-off-by: Sean Owen <sowen@cloudera.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