continuum-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Emmanuel Venisse <emmanuel.veni...@gmail.com>
Subject Re: svn commit: r787953 - in /continuum/trunk: continuum-api/src/main/java/org/apache/continuum/release/distributed/manager/ continuum-api/src/main/java/org/apache/maven/continuum/release/ continuum-buildagent/continuum-buildagent-api/src/main/java/o
Date Fri, 26 Jun 2009 06:23:28 GMT
ok, no problem, we'll modify it with the global refactoring for Continuum2

Emmanuel

On Fri, Jun 26, 2009 at 7:36 AM, Jevica Arianne B. Zurbano <
jevica.arianne@gmail.com> wrote:

> Hi Emmanuel,
>
> I have tried using slf4j MDC/ThreadLocal but it can't retrieve the value
> (username) since it works if running on the same thread.
>
> And the distributed release processes run on different threads.
>
> The method signatures I have modified affects the distributed release
> methods.
>
> I guess I will be leaving the fix as is for now.
>
>
>
> Emmanuel Venisse wrote:
>
>> Wouldn't it be better to use a aspect and/or a ThreadLocal/slf4j MDC to
>> get/set the username instead of adding the username param on all methods?
>> I'll find it more elegant/intrusive.
>>
>> Emmanuel
>>
>> On Wed, Jun 24, 2009 at 11:20 AM, <jzurbano@apache.org> wrote:
>>
>>
>>
> --
>
> Thanks,
>
> Jev
>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message