Skip to content

Conversation

@cloud-fan
Copy link
Contributor

For inner primitive type(e.g. inside Product), we use schemaFor to get the catalyst type for it, https://github.com/apache/spark/blob/master/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ScalaReflection.scala#L403.

However, for top level primitive type, we use dataTypeFor, which is wrong.

@cloud-fan
Copy link
Contributor Author

cc @marmbrus

@cloud-fan
Copy link
Contributor Author

retest this please.

@SparkQA
Copy link

SparkQA commented Oct 29, 2015

Test build #44553 has finished for PR 9337 at commit 8b98d03.

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

@marmbrus
Copy link
Contributor

Thanks, merging to master.

@asfgit asfgit closed this in 87f28fc Oct 29, 2015
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