Skip to content

[MINOR][SQL] Pattern match using Singleton to check if object is a scala object#13080

Closed
techaddict wants to merge 1 commit into
apache:masterfrom
techaddict:use_singleton
Closed

[MINOR][SQL] Pattern match using Singleton to check if object is a scala object#13080
techaddict wants to merge 1 commit into
apache:masterfrom
techaddict:use_singleton

Conversation

@techaddict

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

To Check if obj is an Scala Object, Instead of doing

case obj if obj.getClass.getName.endsWith("$")

we do

case obj: Singleton

How was this patch tested?

Existing tests

@techaddict

Copy link
Copy Markdown
Contributor Author

cc: @cloud-fan

@techaddict techaddict closed this May 12, 2016
@SparkQA

SparkQA commented May 12, 2016

Copy link
Copy Markdown

Test build #58501 has finished for PR 13080 at commit b9e6eab.

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

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.

2 participants