Skip to content

Conversation

@zsxwing
Copy link
Member

@zsxwing zsxwing commented Sep 4, 2018

What changes were proposed in this pull request?

Revert SPARK-24863 (#21819) and SPARK-24748 (#21721) as per discussion in #21721. We will revisit them when the data source v2 APIs are out.

How was this patch tested?

Jenkins

@zsxwing zsxwing changed the title [SPARK-25336][SS]Revert SPARK-24863 and SPARK 24748 [SPARK-25336][SS]Revert SPARK-24863 and SPARK-24748 Sep 4, 2018
@SparkQA
Copy link

SparkQA commented Sep 4, 2018

Test build #95684 has finished for PR 22334 at commit 3d59df1.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@zsxwing
Copy link
Member Author

zsxwing commented Sep 4, 2018

retest this please

@SparkQA
Copy link

SparkQA commented Sep 5, 2018

Test build #95687 has finished for PR 22334 at commit 3d59df1.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@cloud-fan
Copy link
Contributor

retest this please

@SparkQA
Copy link

SparkQA commented Sep 5, 2018

Test build #95689 has finished for PR 22334 at commit 3d59df1.

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

@cloud-fan
Copy link
Contributor

thanks, merging to master!

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