Skip to content

Conversation

@zsxwing
Copy link
Member

@zsxwing zsxwing commented Jan 13, 2016

This PR added instructions to get flume assembly jar for Python users in the flume integration page like Kafka doc.

@zsxwing
Copy link
Member Author

zsxwing commented Jan 13, 2016

CC @tdas

@SparkQA
Copy link

SparkQA commented Jan 14, 2016

Test build #49350 has finished for PR 10746 at commit 6761947.

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

@zsxwing
Copy link
Member Author

zsxwing commented Jan 14, 2016

This PR mostly copied the content from Kafka integration page and changed kafka to flume.

Copy link
Contributor

Choose a reason for hiding this comment

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

approach.

Copy link
Member Author

Choose a reason for hiding this comment

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

Did you mean , -> .?

Copy link
Contributor

Choose a reason for hiding this comment

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

yes, and remove Scala JAva Python

Copy link
Member Author

Choose a reason for hiding this comment

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

Done.

@tdas
Copy link
Contributor

tdas commented Jan 18, 2016

LGTM. Just couple of nits. Thanks @zsxwing

@SparkQA
Copy link

SparkQA commented Jan 18, 2016

Test build #49617 has finished for PR 10746 at commit 895eb4f.

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

@tdas
Copy link
Contributor

tdas commented Jan 18, 2016

LGTM. Merging.

asfgit pushed a commit that referenced this pull request Jan 18, 2016
…ntegration doc

This PR added instructions to get flume assembly jar for Python users in the flume integration page like Kafka doc.

Author: Shixiong Zhu <[email protected]>

Closes #10746 from zsxwing/flume-doc.

(cherry picked from commit a973f48)
Signed-off-by: Tathagata Das <[email protected]>
@asfgit asfgit closed this in a973f48 Jan 18, 2016
@zsxwing zsxwing deleted the flume-doc branch January 18, 2016 23:43
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