Skip to content

[SPARK-14456][SQL][MINOR] Remove unused variables and logics in DataSource#12237

Closed
sarutak wants to merge 1 commit intoapache:masterfrom
sarutak:SPARK-14456
Closed

[SPARK-14456][SQL][MINOR] Remove unused variables and logics in DataSource#12237
sarutak wants to merge 1 commit intoapache:masterfrom
sarutak:SPARK-14456

Conversation

@sarutak
Copy link
Member

@sarutak sarutak commented Apr 7, 2016

What changes were proposed in this pull request?

In DataSource#write method, the variables dataSchema and equality, and related logics are no longer used. Let's remove them.

How was this patch tested?

Existing tests.

@SparkQA
Copy link

SparkQA commented Apr 7, 2016

Test build #55212 has finished for PR 12237 at commit 6382368.

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

@sarutak
Copy link
Member Author

sarutak commented Apr 7, 2016

retest this please.

@SparkQA
Copy link

SparkQA commented Apr 7, 2016

Test build #55215 has finished for PR 12237 at commit 6382368.

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

@marmbrus
Copy link
Contributor

marmbrus commented Apr 7, 2016

Thanks, merging to master.

@asfgit asfgit closed this in 8dcb0c7 Apr 7, 2016
@sarutak sarutak deleted the SPARK-14456 branch April 12, 2016 13:58
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