Skip to content

Conversation

@zsxwing
Copy link
Member

@zsxwing zsxwing commented Oct 25, 2016

What changes were proposed in this pull request?

Don't need to build doc for KafkaSource because the user should use the data source APIs to use KafkaSource. All KafkaSource APIs are internal.

How was this patch tested?

Verified manually.

@zsxwing
Copy link
Member Author

zsxwing commented Oct 26, 2016

cc @tdas

@SparkQA
Copy link

SparkQA commented Oct 26, 2016

Test build #67537 has finished for PR 15630 at commit 65e6cd4.

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

@zsxwing
Copy link
Member Author

zsxwing commented Oct 26, 2016

Discussed with @tdas offline. I'm going to merge to master and 2.0.

asfgit pushed a commit that referenced this pull request Oct 26, 2016
## What changes were proposed in this pull request?

Don't need to build doc for KafkaSource because the user should use the data source APIs to use KafkaSource. All KafkaSource APIs are internal.

## How was this patch tested?

Verified manually.

Author: Shixiong Zhu <[email protected]>

Closes #15630 from zsxwing/kafka-unidoc.

(cherry picked from commit 7d10631)
Signed-off-by: Shixiong Zhu <[email protected]>
@asfgit asfgit closed this in 7d10631 Oct 26, 2016
robert3005 pushed a commit to palantir/spark that referenced this pull request Nov 1, 2016
## What changes were proposed in this pull request?

Don't need to build doc for KafkaSource because the user should use the data source APIs to use KafkaSource. All KafkaSource APIs are internal.

## How was this patch tested?

Verified manually.

Author: Shixiong Zhu <[email protected]>

Closes apache#15630 from zsxwing/kafka-unidoc.
@zsxwing zsxwing deleted the kafka-unidoc branch November 17, 2016 22:31
uzadude pushed a commit to uzadude/spark that referenced this pull request Jan 27, 2017
## What changes were proposed in this pull request?

Don't need to build doc for KafkaSource because the user should use the data source APIs to use KafkaSource. All KafkaSource APIs are internal.

## How was this patch tested?

Verified manually.

Author: Shixiong Zhu <[email protected]>

Closes apache#15630 from zsxwing/kafka-unidoc.
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.

2 participants