Skip to content

Update StructType.scala#10141

Closed
swkimme wants to merge 3 commits into
apache:masterfrom
swkimme:patch-1
Closed

Update StructType.scala#10141
swkimme wants to merge 3 commits into
apache:masterfrom
swkimme:patch-1

Conversation

@swkimme

@swkimme swkimme commented Dec 4, 2015

Copy link
Copy Markdown
Contributor

The example will throw error like
:20: error: not found: value StructType

Need to add this line:
import org.apache.spark.sql.types._

The example will throw error like 
<console>:20: error: not found: value StructType

Need to add this line:
import org.apache.spark.sql.types._
@SparkQA

SparkQA commented Dec 4, 2015

Copy link
Copy Markdown

Test build #2171 has finished for PR 10141 at commit 64d1650.

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you need to remove the trailing space here to pass the style checker

@JoshRosen

Copy link
Copy Markdown
Contributor

Ping @swkimme, can you update this so we can merge it?

@swkimme

swkimme commented Jan 27, 2016

Copy link
Copy Markdown
Contributor Author

@JoshRosen @rxin Oh I've been missed this PR! I'll update!

remove trailing whitespace
@marmbrus

marmbrus commented Feb 2, 2016

Copy link
Copy Markdown
Contributor

ok to test

@SparkQA

SparkQA commented Feb 2, 2016

Copy link
Copy Markdown

Test build #50580 has finished for PR 10141 at commit 058019d.

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

asfgit pushed a commit that referenced this pull request Feb 2, 2016
The example will throw error like
<console>:20: error: not found: value StructType

Need to add this line:
import org.apache.spark.sql.types._

Author: Kevin (Sangwoo) Kim <sangwookim.me@gmail.com>

Closes #10141 from swkimme/patch-1.

(cherry picked from commit b377b03)
Signed-off-by: Michael Armbrust <michael@databricks.com>
@marmbrus

marmbrus commented Feb 2, 2016

Copy link
Copy Markdown
Contributor

Merged to master and 1.6

@asfgit asfgit closed this in b377b03 Feb 2, 2016
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