Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19640
**[Test build #83727 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83727/testReport)**
for PR 19640 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19640
I think it makes sense, but the implementation is too hacky. We should
extract the time epoch and convert it to timestamp string with local timezone.
---
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/19640
@cloud-fan For the UI part, how about this PR:
https://github.com/apache/spark/pull/14577
---
-
To unsubscribe, e-mail:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19640
Since a spark cluster may be used by people from different timezones, I
think at server side we should just store seconds from unix epoch(time epoch),
which is comparable though different
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19640
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19640
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83601/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19640
**[Test build #83601 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83601/testReport)**
for PR 19640 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19640
I'm not a fan of this change; this seems like it should be done
client-side, so that the browser shows timestamps in its own time zone, and the
server should only know about UTC. The API probably
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19640
**[Test build #83601 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83601/testReport)**
for PR 19640 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19640
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83550/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19640
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19640
**[Test build #83550 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83550/testReport)**
for PR 19640 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19640
**[Test build #83550 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83550/testReport)**
for PR 19640 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19640
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83480/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19640
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19640
**[Test build #83480 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83480/testReport)**
for PR 19640 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19640
**[Test build #83480 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/83480/testReport)**
for PR 19640 at commit
17 matches
Mail list logo