Re: Spark 0.9.0 and log4j

2014-03-08 Thread Patrick Wendell
Evan I actually remembered that Paul Brown (who also reported this
issue) tested it and found that it worked. I'm going to merge this
into master and branch 0.9, so please give it a spin when you have a
chance.

- Patrick

On Sat, Mar 8, 2014 at 2:00 PM, Patrick Wendell  wrote:
> Hey Evan,
>
> This is being tracked here:
> https://spark-project.atlassian.net/browse/SPARK-1190
>
> That patch didn't get merged but I've just opened a new one here:
> https://github.com/apache/spark/pull/107/files
>
> Would you have any interest in testing this? I want to make sure it
> works for users who are using logback.
>
> I'd like to get this merged quickly since it's one of the only
> remaining blockers for Spark 0.9.1.
>
> - Patrick
>
>
>
> On Fri, Mar 7, 2014 at 11:04 AM, Evan Chan  wrote:
>> Hey guys,
>>
>> This is a follow-up to this semi-recent thread:
>> http://apache-spark-developers-list.1001551.n3.nabble.com/0-9-0-forces-log4j-usage-td532.html
>>
>> 0.9.0 final is causing issues for us as well because we use Logback as
>> our backend and Spark requires Log4j now.
>>
>> I see Patrick has a PR #560 to incubator-spark, was that merged in or
>> left out?
>>
>> Also I see references to a new PR that might fix this, but I can't
>> seem to find it in the github open PR page.   Anybody have a link?
>>
>> As a last resort we can switch to Log4j, but would rather not have to
>> do that if possible.
>>
>> thanks,
>> Evan
>>
>> --
>> --
>> Evan Chan
>> Staff Engineer
>> e...@ooyala.com  |


Re: Spark 0.9.0 and log4j

2014-03-08 Thread Patrick Wendell
Hey Evan,

This is being tracked here:
https://spark-project.atlassian.net/browse/SPARK-1190

That patch didn't get merged but I've just opened a new one here:
https://github.com/apache/spark/pull/107/files

Would you have any interest in testing this? I want to make sure it
works for users who are using logback.

I'd like to get this merged quickly since it's one of the only
remaining blockers for Spark 0.9.1.

- Patrick



On Fri, Mar 7, 2014 at 11:04 AM, Evan Chan  wrote:
> Hey guys,
>
> This is a follow-up to this semi-recent thread:
> http://apache-spark-developers-list.1001551.n3.nabble.com/0-9-0-forces-log4j-usage-td532.html
>
> 0.9.0 final is causing issues for us as well because we use Logback as
> our backend and Spark requires Log4j now.
>
> I see Patrick has a PR #560 to incubator-spark, was that merged in or
> left out?
>
> Also I see references to a new PR that might fix this, but I can't
> seem to find it in the github open PR page.   Anybody have a link?
>
> As a last resort we can switch to Log4j, but would rather not have to
> do that if possible.
>
> thanks,
> Evan
>
> --
> --
> Evan Chan
> Staff Engineer
> e...@ooyala.com  |


Spark 0.9.0 and log4j

2014-03-07 Thread Evan Chan
Hey guys,

This is a follow-up to this semi-recent thread:
http://apache-spark-developers-list.1001551.n3.nabble.com/0-9-0-forces-log4j-usage-td532.html

0.9.0 final is causing issues for us as well because we use Logback as
our backend and Spark requires Log4j now.

I see Patrick has a PR #560 to incubator-spark, was that merged in or
left out?

Also I see references to a new PR that might fix this, but I can't
seem to find it in the github open PR page.   Anybody have a link?

As a last resort we can switch to Log4j, but would rather not have to
do that if possible.

thanks,
Evan

-- 
--
Evan Chan
Staff Engineer
e...@ooyala.com  |