You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spark.apache.org by Jose Soltren <jo...@cloudera.com> on 2016/11/22 03:31:45 UTC
[SPARK-16654][CORE][WIP] Add UI coverage for Application Level Blacklisting
Hi - I'm proposing a patch set for UI coverage of Application Level
Blacklisting:
https://github.com/jsoltren/spark/pull/1
This patch set builds on top of Imran Rashid's pending pull request,
[SPARK-8425][CORE] Application Level Blacklisting #14079
https://github.com/apache/spark/pull/14079/commits
The best way I could find to send this for review was to fork and
clone apache/spark, pull PR 14079, branch, apply my UI changes, and
issue a pull request of the UI changes into PR 14079. If there is a
better way, forgive me, I would love to hear it.
Attached is a screen shot of the updated UI.
I would appreciate feedback on this WIP patch. I will issue a formal
pull request once PR 14079 is merged.
Cheers,
--José
Re: [SPARK-16654][CORE][WIP] Add UI coverage for Application Level Blacklisting
Posted by Jose Soltren <jo...@cloudera.com>.
Hi Reynold - Alright, if that makes things easier, here is the same
patch set against Imran's squito:blacklist-SPARK-8425 branch.
https://github.com/squito/spark/pull/6
Cheers,
--José
On Mon, Nov 21, 2016 at 10:22 PM, Reynold Xin <rx...@databricks.com> wrote:
> You can submit a pull request against Imran's branch for the pull request.
>
> On Mon, Nov 21, 2016 at 7:33 PM Jose Soltren <jo...@cloudera.com> wrote:
>>
>> Hi - I'm proposing a patch set for UI coverage of Application Level
>> Blacklisting:
>>
>> https://github.com/jsoltren/spark/pull/1
>>
>> This patch set builds on top of Imran Rashid's pending pull request,
>>
>> [SPARK-8425][CORE] Application Level Blacklisting #14079
>> https://github.com/apache/spark/pull/14079/commits
>>
>> The best way I could find to send this for review was to fork and
>> clone apache/spark, pull PR 14079, branch, apply my UI changes, and
>> issue a pull request of the UI changes into PR 14079. If there is a
>> better way, forgive me, I would love to hear it.
>>
>> Attached is a screen shot of the updated UI.
>>
>> I would appreciate feedback on this WIP patch. I will issue a formal
>> pull request once PR 14079 is merged.
>>
>> Cheers,
>> --José
>>
>> ---------------------------------------------------------------------
>> To unsubscribe e-mail: dev-unsubscribe@spark.apache.org
---------------------------------------------------------------------
To unsubscribe e-mail: dev-unsubscribe@spark.apache.org
Re: [SPARK-16654][CORE][WIP] Add UI coverage for Application Level Blacklisting
Posted by Reynold Xin <rx...@databricks.com>.
You can submit a pull request against Imran's branch for the pull request.
On Mon, Nov 21, 2016 at 7:33 PM Jose Soltren <jo...@cloudera.com> wrote:
> Hi - I'm proposing a patch set for UI coverage of Application Level
> Blacklisting:
>
> https://github.com/jsoltren/spark/pull/1
>
> This patch set builds on top of Imran Rashid's pending pull request,
>
> [SPARK-8425][CORE] Application Level Blacklisting #14079
> https://github.com/apache/spark/pull/14079/commits
>
> The best way I could find to send this for review was to fork and
> clone apache/spark, pull PR 14079, branch, apply my UI changes, and
> issue a pull request of the UI changes into PR 14079. If there is a
> better way, forgive me, I would love to hear it.
>
> Attached is a screen shot of the updated UI.
>
> I would appreciate feedback on this WIP patch. I will issue a formal
> pull request once PR 14079 is merged.
>
> Cheers,
> --José
>
> ---------------------------------------------------------------------
> To unsubscribe e-mail: dev-unsubscribe@spark.apache.org