Skip to content

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Jul 29, 2016

What changes were proposed in this pull request?

It is useful to log the timezone when query result does not match, especially on build machines that have different timezone from AMPLab Jenkins.

How was this patch tested?

This is a test-only change.

@SparkQA
Copy link

SparkQA commented Jul 29, 2016

Test build #63020 has finished for PR 14413 at commit 0abbd91.

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

@rxin
Copy link
Contributor Author

rxin commented Aug 1, 2016

cc @yhuai

@yhuai
Copy link
Contributor

yhuai commented Aug 1, 2016

LGTM. Merging to master and branch 2.0.

asfgit pushed a commit that referenced this pull request Aug 1, 2016
## What changes were proposed in this pull request?
It is useful to log the timezone when query result does not match, especially on build machines that have different timezone from AMPLab Jenkins.

## How was this patch tested?
This is a test-only change.

Author: Reynold Xin <rxin@databricks.com>

Closes #14413 from rxin/SPARK-16805.

(cherry picked from commit 579fbcf)
Signed-off-by: Yin Huai <yhuai@databricks.com>
@asfgit asfgit closed this in 579fbcf Aug 1, 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.

3 participants