Skip to content

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