Skip to content

[SPARK-16705] Kafka Direct Stream is not experimental anymore #14343

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

BertrandDechoux
Copy link
Contributor

What changes were proposed in this pull request?

Fix documentation. Direct Stream for Kafka is not experimental anymore. People should use it and not be afraid that it is not stable or likely to change or to be removed.

How was this patch tested?

Visual test. It is only documentation.

See https://issues.apache.org/jira/browse/SPARK-16705

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@srowen
Copy link
Member

srowen commented Jul 25, 2016

CC @koeninger I think we could just remove the text about anything being experimental here, if this is the case.

@koeninger
Copy link
Contributor

Agreed that only removing the word experimental should be necessary. I thought there were more mentions, but can double check.

@jerryshao
Copy link
Contributor

Several annotations is kafka.py are still Experimental, we should also remove them.

@srowen
Copy link
Member

srowen commented Jul 29, 2016

@BertrandDechoux can you update?

@koeninger
Copy link
Contributor

I've got a separate PR for 0.10 docs that also fixes all the experimental
mentions I could find, SPARK-16312
#14385
On Jul 29, 2016 6:42 AM, "Sean Owen" notifications@github.com wrote:

@BertrandDechoux https://github.com/BertrandDechoux can you update?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#14343 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAGAB55EmgS3Sc5XToiI4hXcuhzrCRRYks5qaechgaJpZM4JT4GH
.

@srowen
Copy link
Member

srowen commented Jul 29, 2016

OK maybe we should close this as a duplicate of Cody's larger doc change.

@asfgit asfgit closed this in 53e766c Aug 4, 2016
@BertrandDechoux BertrandDechoux deleted the patch-1 branch August 10, 2016 07:47
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.

5 participants