Skip to content

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Apr 24, 2015

Also took the chance to improve documentation for various types.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a bug introduced in #5350

@SparkQA
Copy link

SparkQA commented Apr 24, 2015

Test build #30902 has finished for PR 5675 at commit 0f31856.

  • 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 d3a302d Apr 24, 2015
Copy link
Contributor

Choose a reason for hiding this comment

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

typo? Seems it should be DataType.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ah yes - I will fix that.

jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request May 14, 2015
Also took the chance to improve documentation for various types.

Author: Reynold Xin <[email protected]>

Closes apache#5675 from rxin/data-type-matching-expr and squashes the following commits:

0f31856 [Reynold Xin] One more function documentation.
27c1973 [Reynold Xin] Added more documentation.
336a36d [Reynold Xin] [SQL] Fixed expression data type matching.
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
Also took the chance to improve documentation for various types.

Author: Reynold Xin <[email protected]>

Closes apache#5675 from rxin/data-type-matching-expr and squashes the following commits:

0f31856 [Reynold Xin] One more function documentation.
27c1973 [Reynold Xin] Added more documentation.
336a36d [Reynold Xin] [SQL] Fixed expression data type matching.
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