[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11259 --- 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

[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-23 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/11259#issuecomment-187796242 +1 --- 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

[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11259#issuecomment-187775837 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

[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11259#issuecomment-187775843 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11259#issuecomment-187775216 **[Test build #51769 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51769/consoleFull)** for PR 11259 at commit

[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11259#issuecomment-187718853 **[Test build #51769 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51769/consoleFull)** for PR 11259 at commit

[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-23 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/11259#issuecomment-187714258 Jenkins, test this please --- 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

[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-22 Thread zhuoliu
Github user zhuoliu commented on the pull request: https://github.com/apache/spark/pull/11259#issuecomment-187487562 Jenkins, test this please --- 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

[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11259#issuecomment-187470800 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11259#issuecomment-187470796 Merged build finished. Test FAILed. --- 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

[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11259#issuecomment-187470566 **[Test build #51673 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51673/consoleFull)** for PR 11259 at commit

[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11259#issuecomment-187397171 **[Test build #51673 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51673/consoleFull)** for PR 11259 at commit

[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-22 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/11259#issuecomment-187214518 Jenkins, test this please --- 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

[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-19 Thread zhuoliu
Github user zhuoliu commented on the pull request: https://github.com/apache/spark/pull/11259#issuecomment-186360098 Hi @rxin , yes, I see the pull request template and now updated the description. @tgravescs , I addressed your comment. Not find a javascript function I can

[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-19 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/11259#discussion_r53471628 --- Diff: core/src/main/resources/org/apache/spark/ui/static/historypage.js --- @@ -37,6 +37,22 @@ function formatDuration(milliseconds) { return

[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-19 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/11259#discussion_r53471523 --- Diff: core/src/main/resources/org/apache/spark/ui/static/historypage.js --- @@ -37,6 +37,22 @@ function formatDuration(milliseconds) { return

[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-18 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11259#issuecomment-186051360 Question: do you see the pull request template? --- 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

[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11259#issuecomment-186049612 Can one of the admins verify this patch? --- 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

[GitHub] spark pull request: [SPARK-13364] Sort appId as num rather than st...

2016-02-18 Thread zhuoliu
GitHub user zhuoliu opened a pull request: https://github.com/apache/spark/pull/11259 [SPARK-13364] Sort appId as num rather than str in history page. ## What changes were proposed in this pull request? History page now sorts the appID as a string, which can lead to unexpected