Github user tgravescs commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-182895149
+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 user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11038
---
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 user tgravescs commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-182583973
Ok, I'll try it again on clean apache build also.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user tgravescs commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-182580830
Can you clarify when you see the exception with inspecting element?
Which browser and plugin, etc?
When I inspect it with firefox I don't see any
Github user ajbozarth commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-182582778
Last I checked I was seeing it on Safari, Firefox and Chrome, no plug-ins.
I'll build master and spin up a Web UI and double check right now though.
---
If your
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-182599890
**[Test build #51059 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51059/consoleFull)**
for PR 11038 at commit
Github user ajbozarth commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-182597718
Ok so I'm only seeing it in Safari now, and I did a couple searches based
on only seeing it in Safari and I found a better fix, I'll push the change
shortly
---
If
Github user ajbozarth commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-182599218
It seems that when a expected map file is missing Firefox and Chrome just
ignore it, Safari on the other hand throws a 404 error. Given the map file
doesn't exist to
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-182642556
**[Test build #51059 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51059/consoleFull)**
for PR 11038 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-182642729
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-182642727
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 user ajbozarth commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-181521289
@tgravescs since you did a lot of review on the DataTables addition do you
want to take a look at this?
---
If your project is set up for it, you can reply to this
Github user holdenk commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-179418027
So looking at the code we can't mix and match the old style and new style
tables on the same page correct?
---
If your project is set up for it, you can reply to this
Github user ajbozarth commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-179425258
@holdenk not without changing a large amount of css or html. Both styles
use common table classes in their basic css. This does seem to be one way
though, only the
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-178948854
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-178948851
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 user ajbozarth opened a pull request:
https://github.com/apache/spark/pull/11038
[SPARK-13124] [Web UI] Fixed CSS and JS issues caused by addition of JQuery
DataTables
Made sure the old tables continue to use the old css and the new DataTables
use the new css. Also added a
Github user ajbozarth commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-178951294
retest 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
Github user ajbozarth commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-178953512
@zhuoliu want to take a look since you added the DataTables
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user ajbozarth commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-178941357
For reference here's what the issues look like, I returned them to their
original look
Github user zhuoliu commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-178984171
Tested also for history page. Looks good to me.
---
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-178989963
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-178989817
**[Test build #50630 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50630/consoleFull)**
for PR 11038 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-178989966
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11038#issuecomment-178956015
**[Test build #50630 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50630/consoleFull)**
for PR 11038 at commit
25 matches
Mail list logo