Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-105741535
it works, thanks @squito i will file a PR
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project do
Github user squito commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-105736242
can you change that line to the equivalent in GMT:
`applications?maxDate=2015-02-03T16:42:40.000GMT`?
sigh, I never realized the problems CST would cause :P
sorry
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-105707437
@squito
https://github.com/apache/spark/blob/master/core/src/test/scala/org/apache/spark/deploy/history/HistoryServerSuite.scala#L85
also failed due to the time zone issue.
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/6377
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enab
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/6377#discussion_r31033487
--- Diff:
core/src/test/scala/org/apache/spark/status/api/v1/SimpleDateParamSuite.scala
---
@@ -18,12 +18,12 @@ package org.apache.spark.status.api.v1
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/6377#discussion_r31007855
--- Diff:
core/src/test/scala/org/apache/spark/status/api/v1/SimpleDateParamSuite.scala
---
@@ -18,12 +18,12 @@ package org.apache.spark.status.api.v1
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-105358297
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-105358294
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-105358259
[Test build #33485 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33485/consoleFull)
for PR 6377 at commit
[`b8df1e5`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-105339197
[Test build #33485 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33485/consoleFull)
for PR 6377 at commit
[`b8df1e5`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-105339091
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-105339082
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/6377#discussion_r30982314
--- Diff:
core/src/test/scala/org/apache/spark/status/api/v1/SimpleDateParamSuite.scala
---
@@ -18,12 +18,13 @@ package org.apache.spark.status.api.v1
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-105133553
[Test build #33460 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33460/consoleFull)
for PR 6377 at commit
[`8bb74f0`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-105133582
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-105133583
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-105117429
[Test build #33460 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33460/consoleFull)
for PR 6377 at commit
[`8bb74f0`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-105116761
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-105116732
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-105116013
@squito updated, tested it and passed on my local machine.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. I
Github user squito commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-105041178
Hi @scwf thanks for finding this. So I dug into this a little bit -- looks
like this is just a little bit of ignorance on my part when creating the test.
I used "CST" t
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-104889569
/cc @squito
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enab
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-104884996
[Test build #33408 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33408/consoleFull)
for PR 6377 at commit
[`34bca75`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-104885010
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-104885008
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-104882417
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-104882412
[Test build #33405 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33405/consoleFull)
for PR 6377 at commit
[`0588fba`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-104882418
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33
Github user scwf commented on a diff in the pull request:
https://github.com/apache/spark/pull/6377#discussion_r30943885
--- Diff:
core/src/test/scala/org/apache/spark/status/api/v1/SimpleDateParamTest.scala ---
@@ -21,6 +21,11 @@ import org.scalatest.{Matchers, FunSuite}
clas
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-104877637
[Test build #33408 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33408/consoleFull)
for PR 6377 at commit
[`34bca75`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-104877263
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-104877210
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/6377#discussion_r30943672
--- Diff:
core/src/test/scala/org/apache/spark/status/api/v1/SimpleDateParamTest.scala ---
@@ -21,6 +21,11 @@ import org.scalatest.{Matchers, FunSuite}
cl
Github user scwf commented on a diff in the pull request:
https://github.com/apache/spark/pull/6377#discussion_r30943635
--- Diff:
core/src/test/scala/org/apache/spark/status/api/v1/SimpleDateParamTest.scala ---
@@ -21,6 +21,11 @@ import org.scalatest.{Matchers, FunSuite}
clas
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/6377#discussion_r30943538
--- Diff:
core/src/test/scala/org/apache/spark/status/api/v1/SimpleDateParamTest.scala ---
@@ -21,6 +21,11 @@ import org.scalatest.{Matchers, FunSuite}
cl
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-104872631
[Test build #33405 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33405/consoleFull)
for PR 6377 at commit
[`0588fba`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-104872607
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6377#issuecomment-104872585
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
GitHub user scwf opened a pull request:
https://github.com/apache/spark/pull/6377
[CORE] [TEST] Fix SimpleDateParamTest
```
sbt.ForkMain$ForkError: 1424424077190 was not equal to 1424474477190
at
org.scalatest.MatchersHelper$.newTestFailedException(MatchersHelper.scala:1
39 matches
Mail list logo