Skip to content

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Apr 23, 2015

It was over 1000 lines of code, making it harder to find all the types. Only moved code around, and didn't change any.

It was over 1000 lines of code, making it harder to find all the types.
@SparkQA
Copy link

SparkQA commented Apr 23, 2015

Test build #30872 has finished for PR 5670 at commit dcd5193.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds no public classes.
  • This patch does not change any dependencies.

@SparkQA
Copy link

SparkQA commented Apr 23, 2015

Test build #30873 has finished for PR 5670 at commit 8c59023.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.
  • This patch does not change any dependencies.

@asfgit asfgit closed this in 6220d93 Apr 23, 2015
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request May 14, 2015
It was over 1000 lines of code, making it harder to find all the types. Only moved code around, and didn't change any.

Author: Reynold Xin <[email protected]>

Closes apache#5670 from rxin/break-types and squashes the following commits:

8c59023 [Reynold Xin] Check in missing files.
dcd5193 [Reynold Xin] [SQL] Break dataTypes.scala into multiple files.
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
It was over 1000 lines of code, making it harder to find all the types. Only moved code around, and didn't change any.

Author: Reynold Xin <[email protected]>

Closes apache#5670 from rxin/break-types and squashes the following commits:

8c59023 [Reynold Xin] Check in missing files.
dcd5193 [Reynold Xin] [SQL] Break dataTypes.scala into multiple files.
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