Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-2729][SQL] Added test case for SPARK-2729 #1738

Closed
wants to merge 1 commit into from

Conversation

liancheng
Copy link
Contributor

This is a follow up of #1636.

@SparkQA
Copy link

SparkQA commented Aug 2, 2014

QA tests have started for PR 1738. This patch merges cleanly.
View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17770/consoleFull

@SparkQA
Copy link

SparkQA commented Aug 2, 2014

QA results for PR 1738:
- This patch PASSES unit tests.
- This patch merges cleanly
- This patch adds the following public classes (experimental):
case class TimestampField(time: Timestamp)

For more information see test ouptut:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17770/consoleFull

@marmbrus
Copy link
Contributor

marmbrus commented Aug 3, 2014

Thanks! I've merged this into master and 1.1

@asfgit asfgit closed this in 866cf1f Aug 3, 2014
asfgit pushed a commit that referenced this pull request Aug 3, 2014
This is a follow up of #1636.

Author: Cheng Lian <lian.cs.zju@gmail.com>

Closes #1738 from liancheng/test-for-spark-2729 and squashes the following commits:

b13692a [Cheng Lian] Added test case for SPARK-2729

(cherry picked from commit 866cf1f)
Signed-off-by: Michael Armbrust <michael@databricks.com>
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
This is a follow up of apache#1636.

Author: Cheng Lian <lian.cs.zju@gmail.com>

Closes apache#1738 from liancheng/test-for-spark-2729 and squashes the following commits:

b13692a [Cheng Lian] Added test case for SPARK-2729
@liancheng liancheng deleted the test-for-spark-2729 branch September 24, 2014 00:19
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