[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-12-06 Thread merlintang
Github user merlintang commented on the issue: https://github.com/apache/spark/pull/13670 @kishorvpatil you provided the function allexecutors, which is used to return the dead and active executor information. For the document http://spark.apache.org/docs/latest/moni

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-21 Thread ajbozarth
Github user ajbozarth commented on the issue: https://github.com/apache/spark/pull/13670 I opened SPARK-16673 for the issue since it also apply to the Thread Dump column which the History Server shouldn't display. --- If your project is set up for it, you can reply to this email and

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-21 Thread tgravescs
Github user tgravescs commented on the issue: https://github.com/apache/spark/pull/13670 personally I don't see this as a big deal. As long as it doesn't show the links, the column being there doesn't hurt anything and it keeps things more consistent. Perhaps change it to say Not Av

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-21 Thread ajbozarth
Github user ajbozarth commented on the issue: https://github.com/apache/spark/pull/13670 @kishorvpatil I can't believe I missed that. The logs only exist if you have logging turned on in your confs, if it's off there won't be any logs and no reason for the column. @nblintao if you wan

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-21 Thread kishorvpatil
Github user kishorvpatil commented on the issue: https://github.com/apache/spark/pull/13670 @nblintao , I am not sure why we had the check of existence of logs - specially when the reference to logs is static link for `stdout` and `stderr`. Unless there is scenario where there coul

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-21 Thread nblintao
Github user nblintao commented on the issue: https://github.com/apache/spark/pull/13670 Hi, @kishorvpatil. Cool improvement! I'm developing the Executors Page at #14204 based on your changes. But I have a small question about your new implementation of the tables. In the implement

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-20 Thread tgravescs
Github user tgravescs commented on the issue: https://github.com/apache/spark/pull/13670 +1, thanks @kishorvpatil @ajbozarth --- 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] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 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 does not have this feature e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62607/ Test PASSed. ---

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-20 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62607 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62607/consoleFull)** for PR 13670 at commit [`87301c1`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-20 Thread kishorvpatil
Github user kishorvpatil commented on the issue: https://github.com/apache/spark/pull/13670 @ajbozarth Thank you for review. I removed unused methods from the the javascript. I agree converting pages to `DataTables` is tricky and listing some standard way might be a huge hel

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-20 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62607 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62607/consoleFull)** for PR 13670 at commit [`87301c1`](https://github.com/apache/spark/commit/8

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-19 Thread ajbozarth
Github user ajbozarth commented on the issue: https://github.com/apache/spark/pull/13670 I checked out out the latest code and ran it and it looks good, I also looked through the code and outside of the two unused functions I found the code looks good too. Im comfortable givi

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-18 Thread ajbozarth
Github user ajbozarth commented on the issue: https://github.com/apache/spark/pull/13670 Sorry I couldn't get to this today, I'll do my best to take a look in the morning --- 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] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62472/ Test PASSed. ---

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 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 does not have this feature e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-18 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62472 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62472/consoleFull)** for PR 13670 at commit [`e81d45b`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-18 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62472 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62472/consoleFull)** for PR 13670 at commit [`e81d45b`](https://github.com/apache/spark/commit/e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-18 Thread kishorvpatil
Github user kishorvpatil commented on the issue: https://github.com/apache/spark/pull/13670 @tgravescs, I removed unwanted code/comment from the javascript. --- 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] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-15 Thread ajbozarth
Github user ajbozarth commented on the issue: https://github.com/apache/spark/pull/13670 Been busy the last couple days, I'll take a look at this Monday --- 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

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-15 Thread tgravescs
Github user tgravescs commented on the issue: https://github.com/apache/spark/pull/13670 @ajbozarth were you able to try the latest version? --- 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 issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62346/ Test PASSed. ---

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 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 does not have this feature e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-14 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62346 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62346/consoleFull)** for PR 13670 at commit [`66df19f`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 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 does not have this feature e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62343/ Test PASSed. ---

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-14 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62343 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62343/consoleFull)** for PR 13670 at commit [`bc132b1`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-14 Thread kishorvpatil
Github user kishorvpatil commented on the issue: https://github.com/apache/spark/pull/13670 @tgravescs , moved `formatBytes` to `utils.js`. --- 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 thi

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-14 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62346 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62346/consoleFull)** for PR 13670 at commit [`66df19f`](https://github.com/apache/spark/commit/6

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-14 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62343 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62343/consoleFull)** for PR 13670 at commit [`bc132b1`](https://github.com/apache/spark/commit/b

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-14 Thread kishorvpatil
Github user kishorvpatil commented on the issue: https://github.com/apache/spark/pull/13670 @ajbozarth , I fixed the code for standalone mode as well. Sorry for not being fully aware of possible UI invocations. I will keep this in mind for future patches. --- If your project is set

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-14 Thread kishorvpatil
Github user kishorvpatil commented on the issue: https://github.com/apache/spark/pull/13670 @tgravescs, fixed documentation. --- 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 enabl

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread ajbozarth
Github user ajbozarth commented on the issue: https://github.com/apache/spark/pull/13670 Follow up to my previous post, this works on yarn and the history server but not on standalone, as a user I can't figure out what's broken. --- If your project is set up for it, you can reply to

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread ajbozarth
Github user ajbozarth commented on the issue: https://github.com/apache/spark/pull/13670 Thanks for all the fixes, I checked it out and am in the middle of looking at it. It looks great on the history server for me but the page just doesn't load or show and form of errors when i try t

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 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 does not have this feature e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62274/ Test PASSed. ---

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62274 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62274/consoleFull)** for PR 13670 at commit [`faf814f`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread kishorvpatil
Github user kishorvpatil commented on the issue: https://github.com/apache/spark/pull/13670 @tgravescs, I think addressed the comments mentioned above. --- 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

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62274 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62274/consoleFull)** for PR 13670 at commit [`faf814f`](https://github.com/apache/spark/commit/f

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62258/ Test PASSed. ---

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 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 does not have this feature e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62258 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62258/consoleFull)** for PR 13670 at commit [`5cf514d`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 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 project does not have this feature e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62269/ Test FAILed. ---

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62269 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62269/consoleFull)** for PR 13670 at commit [`286d6d8`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62269 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62269/consoleFull)** for PR 13670 at commit [`286d6d8`](https://github.com/apache/spark/commit/2

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 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 project does not have this feature e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62267 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62267/consoleFull)** for PR 13670 at commit [`fe12d59`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62267/ Test FAILed. ---

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62267 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62267/consoleFull)** for PR 13670 at commit [`fe12d59`](https://github.com/apache/spark/commit/f

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62263/ Test FAILed. ---

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62263 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62263/consoleFull)** for PR 13670 at commit [`eb96c5d`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 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 project does not have this feature e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62263 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62263/consoleFull)** for PR 13670 at commit [`eb96c5d`](https://github.com/apache/spark/commit/e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62258 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62258/consoleFull)** for PR 13670 at commit [`5cf514d`](https://github.com/apache/spark/commit/5

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62195/ Test PASSed. ---

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 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 does not have this feature e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62195 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62195/consoleFull)** for PR 13670 at commit [`3454193`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62194/ Test PASSed. ---

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 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 does not have this feature e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62194 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62194/consoleFull)** for PR 13670 at commit [`55419aa`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 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 does not have this feature e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62191/ Test PASSed. ---

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62191 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62191/consoleFull)** for PR 13670 at commit [`2453e21`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62195 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62195/consoleFull)** for PR 13670 at commit [`3454193`](https://github.com/apache/spark/commit/3

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-12 Thread ajbozarth
Github user ajbozarth commented on the issue: https://github.com/apache/spark/pull/13670 @kishorvpatil thanks, I'll check it out --- 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 e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62194 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62194/consoleFull)** for PR 13670 at commit [`55419aa`](https://github.com/apache/spark/commit/5

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-12 Thread kishorvpatil
Github user kishorvpatil commented on the issue: https://github.com/apache/spark/pull/13670 @ajbozarth, I have fixed the issue with safari exception. Its because `parser.baseURI` is `null`. I switched to `document.baseURI`. It looks fine on testing. Let me know if you still see this i

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #62191 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62191/consoleFull)** for PR 13670 at commit [`2453e21`](https://github.com/apache/spark/commit/2

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-07-12 Thread tgravescs
Github user tgravescs commented on the issue: https://github.com/apache/spark/pull/13670 Made a pass through and some mostly minor comments. --- 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 th

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-28 Thread ajbozarth
Github user ajbozarth commented on the issue: https://github.com/apache/spark/pull/13670 I get the error by just starting up a spark-shell, but I only get it on Safari, it works fine on Chrome and Firefox. I'll make a line note of what I see going wrong --- If your project is set u

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-28 Thread ajbozarth
Github user ajbozarth commented on the issue: https://github.com/apache/spark/pull/13670 I'll re-checkout later today and see if I can reproduce the error for you --- 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 issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-28 Thread kishorvpatil
Github user kishorvpatil commented on the issue: https://github.com/apache/spark/pull/13670 @ajbozarth, I do not see how I could produce the exception you see. Please help me reproduce this so that I can address it. --- If your project is set up for it, you can reply to this email an

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-28 Thread kishorvpatil
Github user kishorvpatil commented on the issue: https://github.com/apache/spark/pull/13670 @zhuoliu , i have removed unwanted, unused javascript functions in the executorspage.js. There are no duplicate functions now between historyserver.js and this script.. --- If your project is

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-28 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #61387 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61387/consoleFull)** for PR 13670 at commit [`c73e905`](https://github.com/apache/spark/commit/c

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 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 does not have this feature e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61070/ Test PASSed. ---

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-22 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #61070 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61070/consoleFull)** for PR 13670 at commit [`3b4962a`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-22 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #61070 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61070/consoleFull)** for PR 13670 at commit [`3b4962a`](https://github.com/apache/spark/commit/3

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-16 Thread ajbozarth
Github user ajbozarth commented on the issue: https://github.com/apache/spark/pull/13670 I agree with @zhuoliu we should move common/util functions into a datatables util js. For the above error, I pinpointed that it is baseURI that is null. I also never see the "Loading Exec

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-16 Thread ajbozarth
Github user ajbozarth commented on the issue: https://github.com/apache/spark/pull/13670 My comment didn't post last night: I haven't finished reading through your code yet, but after checking it out and starting up the web ui I get the following error on the executors page. ![

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-16 Thread zhuoliu
Github user zhuoliu commented on the issue: https://github.com/apache/spark/pull/13670 Generally looks great to me, I am good with it as long as @tgravescs verifies. One minor concern is that we may move the utility functions from executorspage.js and historypage.ps to a common locati

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 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 does not have this feature e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60612/ Test PASSed. ---

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-15 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #60612 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60612/consoleFull)** for PR 13670 at commit [`56f3901`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-15 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #60612 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60612/consoleFull)** for PR 13670 at commit [`56f3901`](https://github.com/apache/spark/commit/5

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-15 Thread ajbozarth
Github user ajbozarth commented on the issue: https://github.com/apache/spark/pull/13670 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 this feature enabled and wis

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60601/ Test FAILed. ---

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 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 project does not have this feature e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-15 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #60601 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60601/consoleFull)** for PR 13670 at commit [`56f3901`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-15 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #60601 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60601/consoleFull)** for PR 13670 at commit [`56f3901`](https://github.com/apache/spark/commit/5

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-15 Thread tgravescs
Github user tgravescs commented on the issue: https://github.com/apache/spark/pull/13670 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 this feature enabled and wis

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60584/ Test FAILed. ---

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 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 project does not have this feature e

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-15 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #60584 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60584/consoleFull)** for PR 13670 at commit [`56f3901`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-15 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13670 **[Test build #60584 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60584/consoleFull)** for PR 13670 at commit [`56f3901`](https://github.com/apache/spark/commit/5

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-15 Thread tgravescs
Github user tgravescs commented on the issue: https://github.com/apache/spark/pull/13670 Jenkins, ok to test --- 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

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-15 Thread kishorvpatil
Github user kishorvpatil commented on the issue: https://github.com/apache/spark/pull/13670 @ajbozarth , Thank you for loooking into this patch. I have added screenshot with annotation. I hope that helps. --- If your project is set up for it, you can reply to this email and have you

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-14 Thread ajbozarth
Github user ajbozarth commented on the issue: https://github.com/apache/spark/pull/13670 Can you post a screenshot of how it looks after the change? I'll take a look through the code when I have a chance. Also @zhuoliu want to take a look since you were the one who added JQuery DataTa

[GitHub] spark issue #13670: [SPARK-15951] Change Executors Page to use datatables to...

2016-06-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13670 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 project does not have this feat

  1   2   >