Skip to content

[SPARK-17798][SQL] Remove redundant Experimental annotations in sql.streaming#15373

Closed
rxin wants to merge 1 commit into
apache:masterfrom
rxin:SPARK-17798
Closed

[SPARK-17798][SQL] Remove redundant Experimental annotations in sql.streaming#15373
rxin wants to merge 1 commit into
apache:masterfrom
rxin:SPARK-17798

Conversation

@rxin

@rxin rxin commented Oct 6, 2016

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

I was looking through API annotations to catch mislabeled APIs, and realized DataStreamReader and DataStreamWriter classes are already annotated as Experimental, and as a result there is no need to annotate each method within them.

How was this patch tested?

N/A

@rxin

rxin commented Oct 6, 2016

Copy link
Copy Markdown
Contributor Author

cc @zsxwing

@zsxwing

zsxwing commented Oct 6, 2016

Copy link
Copy Markdown
Member

LGTM

@SparkQA

SparkQA commented Oct 6, 2016

Copy link
Copy Markdown

Test build #66429 has finished for PR 15373 at commit 3013513.

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

@zsxwing

zsxwing commented Oct 6, 2016

Copy link
Copy Markdown
Member

Thanks! Merging to master and 2.0.

@asfgit asfgit closed this in 79accf4 Oct 6, 2016
asfgit pushed a commit that referenced this pull request Oct 6, 2016
…treaming

## What changes were proposed in this pull request?
I was looking through API annotations to catch mislabeled APIs, and realized DataStreamReader and DataStreamWriter classes are already annotated as Experimental, and as a result there is no need to annotate each method within them.

## How was this patch tested?
N/A

Author: Reynold Xin <rxin@databricks.com>

Closes #15373 from rxin/SPARK-17798.

(cherry picked from commit 79accf4)
Signed-off-by: Shixiong Zhu <shixiong@databricks.com>
uzadude pushed a commit to uzadude/spark that referenced this pull request Jan 27, 2017
…treaming

## What changes were proposed in this pull request?
I was looking through API annotations to catch mislabeled APIs, and realized DataStreamReader and DataStreamWriter classes are already annotated as Experimental, and as a result there is no need to annotate each method within them.

## How was this patch tested?
N/A

Author: Reynold Xin <rxin@databricks.com>

Closes apache#15373 from rxin/SPARK-17798.
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