Skip to content

Conversation

@scwf
Copy link
Contributor

@scwf scwf commented Dec 27, 2014

HiveInspectorSuite test failure:
[info] - wrap / unwrap null, constant null and writables *** FAILED *** (21 milliseconds)
[info] 1 did not equal 0 (HiveInspectorSuite.scala:136)
this is because the origin date(is 3914-10-23) not equals the date returned by unwrap(is 3914-10-22).

Setting TimeZone and Locale fix this.
Another minor change here is rename def checkValues(v1: Any, v2: Any): Unit to def checkValue(v1: Any, v2: Any): Unit to make the code more clear

@SparkQA
Copy link

SparkQA commented Dec 27, 2014

Test build #24843 has started for PR 3814 at commit 72b19a9.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Dec 27, 2014

Test build #24844 has started for PR 3814 at commit d8531ef.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Dec 27, 2014

Test build #24844 has finished for PR 3814 at commit d8531ef.

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

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24844/
Test PASSed.

@SparkQA
Copy link

SparkQA commented Dec 27, 2014

Test build #24843 has finished for PR 3814 at commit 72b19a9.

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

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24843/
Test PASSed.

@marmbrus
Copy link
Contributor

Thanks! Merged to master.

@asfgit asfgit closed this in 65357f1 Dec 30, 2014
@scwf scwf deleted the fix-inspectorsuite branch January 7, 2015 09:49
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.

4 participants