[GitHub] spark pull request: [SPARK-13481] Desc order of appID by default f...

2016-03-09 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/11357#issuecomment-194406185 fyi https://issues.apache.org/jira/browse/SPARK-13775 to sort by completed time. --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] spark pull request: [SPARK-13481] Desc order of appID by default f...

2016-02-29 Thread zjffdu
Github user zjffdu commented on the pull request: https://github.com/apache/spark/pull/11357#issuecomment-190245248 agree with sorting on start time. different mode has different format of app_id. sorting by start time is more reasonable. --- If your project is set up for it, you ca

[GitHub] spark pull request: [SPARK-13481] Desc order of appID by default f...

2016-02-29 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11357#issuecomment-190240878 I don't mind the change if @zhuoliu doesn't since the original idea was to sort by start time. Meh, maybe the JIRA title could be updated but not a big deal. --- If yo

[GitHub] spark pull request: [SPARK-13481] Desc order of appID by default f...

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

[GitHub] spark pull request: [SPARK-13481] Desc order of appID by default f...

2016-02-29 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/11357#issuecomment-190234286 since this is just changing the default order to descending I'm going to check this in, if we want to change which column its sorted by lets file a separate jira.

[GitHub] spark pull request: [SPARK-13481] Desc order of appID by default f...

2016-02-26 Thread zhuoliu
Github user zhuoliu commented on the pull request: https://github.com/apache/spark/pull/11357#issuecomment-189516371 This basically provides a default view of sorting when the page is loaded. User can easily sort by start time or other columns by clicking the column name. --- If your

[GitHub] spark pull request: [SPARK-13481] Desc order of appID by default f...

2016-02-25 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/11357#issuecomment-19246 So we could. I was hoping for it to be similar to the YARN RM ui where it sorted by app id. To me its also most intuitive to have first column sorted. The

[GitHub] spark pull request: [SPARK-13481] Desc order of appID by default f...

2016-02-25 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11357#issuecomment-15864 Could we just sort by start time? that sounds like the goal. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] spark pull request: [SPARK-13481] Desc order of appID by default f...

2016-02-25 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/11357#issuecomment-11477 +1 on this. @srowen let me know if you have concerns still or if we didn't describe well enough. --- If your project is set up for it, you can reply to this email a

[GitHub] spark pull request: [SPARK-13481] Desc order of appID by default f...

2016-02-25 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/11357#issuecomment-188799267 descending puts all the applications with application_ in the order with most recent on top. Since its still sorting alphabetically it puts the local- before the appl

[GitHub] spark pull request: [SPARK-13481] Desc order of appID by default f...

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

[GitHub] spark pull request: [SPARK-13481] Desc order of appID by default f...

2016-02-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11357#issuecomment-188795346 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-13481] Desc order of appID by default f...

2016-02-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11357#issuecomment-188794748 **[Test build #51965 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51965/consoleFull)** for PR 11357 at commit [`dfe86bc`](https://g

[GitHub] spark pull request: [SPARK-13481] Desc order of appID by default f...

2016-02-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11357#issuecomment-188747186 **[Test build #51965 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51965/consoleFull)** for PR 11357 at commit [`dfe86bc`](https://gi

[GitHub] spark pull request: [SPARK-13481] Desc order of appID by default f...

2016-02-25 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11357#issuecomment-188745917 Descending by app ID != most recent first, right? I could see an argument for the latter but I'm not sure that's what this does. In your example, you even see it shows "

[GitHub] spark pull request: [SPARK-13481] Desc order of appID by default f...

2016-02-25 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/11357#issuecomment-188745439 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 have t

[GitHub] spark pull request: [SPARK-13481] Desc order of appID by default f...

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

[GitHub] spark pull request: [SPARK-13481] Desc order of appID by default f...

2016-02-24 Thread zhuoliu
GitHub user zhuoliu opened a pull request: https://github.com/apache/spark/pull/11357 [SPARK-13481] Desc order of appID by default for history server page. ## What changes were proposed in this pull request? Now by default, it shows as ascending order of appId. We might pref