Github user sarutak commented on the issue:
https://github.com/apache/spark/pull/12257
Sorry for my late reply. I'll close this PR for now and might open another
PR in the near future. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/12257
Really appreciate your contribution! Sorry, based on the comment, we might
need to close this PR, but please submit more PRs in the future. Thanks again!
---
If your project is set up for it,
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/12257
FYI #18331 has made some updates that affect this
---
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
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/12257
gentle ping @sarutak on ^
---
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
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/12257
I'm still worried about clutter like I mentioned before, but the additional
table column is what I'm most worried about especially on the AllJobsPage.
@sarutak how important is the user column on
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/12257
gentle ping @ajbozarth @tdas
---
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 HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/12257
Hi all, where are we on this?
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70958/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #70958 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70958/testReport)**
for PR 12257 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #70958 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70958/testReport)**
for PR 12257 at commit
Github user sarutak commented on the issue:
https://github.com/apache/spark/pull/12257
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 feature
enabled and wishes so,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #70949 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70949/testReport)**
for PR 12257 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70949/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #70949 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70949/testReport)**
for PR 12257 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69971/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #69971 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69971/consoleFull)**
for PR 12257 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #69971 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/69971/consoleFull)**
for PR 12257 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68758/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #68758 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68758/consoleFull)**
for PR 12257 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #68758 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68758/consoleFull)**
for PR 12257 at commit
Github user sarutak commented on the issue:
https://github.com/apache/spark/pull/12257
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 feature
enabled and wishes so,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68748/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #68748 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68748/consoleFull)**
for PR 12257 at commit
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/12257
The screen shots look great, and you're probably right about a better
solution. I'm not sure if a checkbox is much better though, especially since
theres no precedent for it on these pages.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68242/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #68242 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68242/consoleFull)**
for PR 12257 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #68242 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68242/consoleFull)**
for PR 12257 at commit
Github user sarutak commented on the issue:
https://github.com/apache/spark/pull/12257
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 feature
enabled and wishes so,
Github user sarutak commented on the issue:
https://github.com/apache/spark/pull/12257
Not my day...
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 feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #68240 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68240/consoleFull)**
for PR 12257 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68240/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #68240 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68240/consoleFull)**
for PR 12257 at commit
Github user sarutak commented on the issue:
https://github.com/apache/spark/pull/12257
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 feature
enabled and wishes so,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68235/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #68235 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68235/consoleFull)**
for PR 12257 at commit
Github user sarutak commented on the issue:
https://github.com/apache/spark/pull/12257
@ajbozarth Your concern seems to be reasonable but I think it might be
better way than you suggested. One possible way is toggle the appearance of
User column by checkbox on demand. I think it's
Github user sarutak commented on the issue:
https://github.com/apache/spark/pull/12257
@tdas @ajbozarth Sorry for the late response. I'm attaching new screen
captures as follows.
* AllJobsPage
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #68235 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68235/consoleFull)**
for PR 12257 at commit
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/12257
I only skimmed the code, but it seems good. I agree with @tdas about moving
the user column to the right of the description column though. Also I believe
your screen shot for JobPage(After) is
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67770/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #67770 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67770/consoleFull)**
for PR 12257 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #67770 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67770/consoleFull)**
for PR 12257 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67764/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #67764 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67764/consoleFull)**
for PR 12257 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #67764 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67764/consoleFull)**
for PR 12257 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67752/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #67752 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67752/consoleFull)**
for PR 12257 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #67752 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67752/consoleFull)**
for PR 12257 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67748/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #67748 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67748/consoleFull)**
for PR 12257 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #67748 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67748/consoleFull)**
for PR 12257 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12257
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67745/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #67745 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67745/consoleFull)**
for PR 12257 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12257
**[Test build #67745 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67745/consoleFull)**
for PR 12257 at commit
Github user sarutak commented on the issue:
https://github.com/apache/spark/pull/12257
@ajbozarth @tdas Thanks for your interest. I'll rebase this to master and
reflect the comments.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/12257
@sarutak this is a good PR have. could you please resolve the conflicts
with master?
---
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 tdas commented on the issue:
https://github.com/apache/spark/pull/12257
@sarutak this looks like a useful PR, mind rebasing to master?
Also a high level comment. The 'user' is hardly the second thing that most
users want to see. Description is most important. So its
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/12257
Looking through some stale prs to review and try to push through. This
still looks like a good add, @sarutak could you rebase?
---
If your project is set up for it, you can reply to this email
70 matches
Mail list logo