Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/19954
Thanks! merging to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user liyinan926 commented on the issue:
https://github.com/apache/spark/pull/19954
@vanzin @jiangxb1987 @ueshin Can this PR be merged now?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85398/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85398 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85398/testReport)**
for PR 19954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85398 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85398/testReport)**
for PR 19954 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85384/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85384 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85384/testReport)**
for PR 19954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85384 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85384/testReport)**
for PR 19954 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85352/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85352 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85352/testReport)**
for PR 19954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85352 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85352/testReport)**
for PR 19954 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85327/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85327 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85327/testReport)**
for PR 19954 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85324/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85324 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85324/testReport)**
for PR 19954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85327 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85327/testReport)**
for PR 19954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85324 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85324/testReport)**
for PR 19954 at commit
Github user foxish commented on the issue:
https://github.com/apache/spark/pull/19954
@mridulm, can you PTAL? This is ready for another round of review, and is
the last large feature we're planning to do in 2.3.
---
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/19954
I'll finish reading this by Friday, thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85219/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85219 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85219/testReport)**
for PR 19954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85219 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85219/testReport)**
for PR 19954 at commit
Github user liyinan926 commented on the issue:
https://github.com/apache/spark/pull/19954
@vanzin Thanks for the review! Addressed your comments in
https://github.com/apache/spark/pull/19954/commits/be5de00c9a75ef583759b931e3f8597b9f426f6c.
---
Github user foxish commented on the issue:
https://github.com/apache/spark/pull/19954
> I don't think they are independent as architecturally they make sense
together and represent a single concern: enabling use of remote dependencies
through init-containers. Missing any one of the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85142/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85142 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85142/testReport)**
for PR 19954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85142 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85142/testReport)**
for PR 19954 at commit
Github user liyinan926 commented on the issue:
https://github.com/apache/spark/pull/19954
@vanzin Addressed your comments in
https://github.com/apache/spark/pull/19954/commits/3407d7af68c44b100558e49e4012a27e41b29dda.
PTAL. Thanks a lot!
---
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/19954
> I don't think they are independent as architecturally they make sense
together and represent a single concern: enabling use of remote dependencies
through init-containers. Missing any one of the
Github user liyinan926 commented on the issue:
https://github.com/apache/spark/pull/19954
> @liyinan926 the code structure is the main issue, yes, but we can tackle
the code structure more effectively by having a better decomposition of the
review process as well. We have these three
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/19954
> I disagree. For example, just to pick one example I spotted while
glancing at the code quickly. There are many steps that take a
kubernetesResourceNamePrefix as a constructor argument. Then the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85126/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85126 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85126/testReport)**
for PR 19954 at commit
Github user liyinan926 commented on the issue:
https://github.com/apache/spark/pull/19954
> @liyinan926 we might be able to make the review smaller, though this
makes some of the code we contribute on each push unusable on its own.
@mccheah I think the biggest thing that
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19954
> I think regardless of the abstractions we choose, we're eventually going
to end up with many classes with unique constructors simply due to the
complexity of the submission client itself.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85122/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85122 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85122/testReport)**
for PR 19954 at commit
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/19954
@liyinan926 we might be able to make the review smaller, though this makes
some of the code we contribute on each push unusable on its own. If we're ok
with that, how about we create these commits:
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/19954
> It's nice that there is some documentation somewhere, but that
documentation doesn't really seem to address my comments. For one example, it
only explicitly talks about the driver - which sort of
Github user liyinan926 commented on the issue:
https://github.com/apache/spark/pull/19954
@vanzin additionally to shortening names and removing unnecessary traits,
we have further improved the code by removing unused/unnecessary class
arguments and enhancing the Scaladocs on core
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85126 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85126/testReport)**
for PR 19954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85122 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85122/testReport)**
for PR 19954 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85105/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85105 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85105/testReport)**
for PR 19954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85105 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85105/testReport)**
for PR 19954 at commit
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/19954
Jenkins, retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85094/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85094 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85094/testReport)**
for PR 19954 at commit
Github user liyinan926 commented on the issue:
https://github.com/apache/spark/pull/19954
@vanzin @mccheah @foxish Variables names have been shortened in
https://github.com/apache/spark/pull/19954/commits/d50c61e46c43049b9ede615ffabd89a55c4adb0c,
and traits with only a single
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85094 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85094/testReport)**
for PR 19954 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19954
> We discuss this a bit more in here
It's nice that there is some documentation somewhere, but that
documentation doesn't really seem to address my comments. For one example, it
only
Github user liyinan926 commented on the issue:
https://github.com/apache/spark/pull/19954
@mccheah @foxish I gave you push access to the fork I used for this PR.
Feel free to push commits if you want. Please do let me know if you plan to
address comments from @vanzin and make
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/19954
> I also don't fully understand all the abstractions being created here. It
seems there are three separate concepts (a "bootstrap", a "configuration step",
and an "orchestrator") and they're not
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85069/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85069 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85069/testReport)**
for PR 19954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85069 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85069/testReport)**
for PR 19954 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85066/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85066 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85066/testReport)**
for PR 19954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85066 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85066/testReport)**
for PR 19954 at commit
Github user liyinan926 commented on the issue:
https://github.com/apache/spark/pull/19954
@ueshin addressed your comments. PTAL. Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85056/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85056 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85056/testReport)**
for PR 19954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85056 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85056/testReport)**
for PR 19954 at commit
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/19954
Jenkins, retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85029/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85029 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85029/testReport)**
for PR 19954 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85028/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85028 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85028/testReport)**
for PR 19954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85029 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85029/testReport)**
for PR 19954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85028 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85028/testReport)**
for PR 19954 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85013/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85013 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85013/testReport)**
for PR 19954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #85013 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85013/testReport)**
for PR 19954 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84985/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #84985 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84985/testReport)**
for PR 19954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #84985 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84985/testReport)**
for PR 19954 at commit
Github user liyinan926 commented on the issue:
https://github.com/apache/spark/pull/19954
@vanzin Addressed your comments so far. PTAL. Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19954
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84966/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #84966 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84966/testReport)**
for PR 19954 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19954
**[Test build #84966 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84966/testReport)**
for PR 19954 at commit
Github user foxish commented on the issue:
https://github.com/apache/spark/pull/19954
@mridulm We did have some discussions on resource localization at some
point. This is a powerful mechanism when coupled with a resource staging server
(in the future). There is a cost per-pod when
1 - 100 of 104 matches
Mail list logo