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

Thu, 25 Jun 2009 22:37:30 -0700 

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, <jzurb...@apache.org> wrote:


  

--

Thanks,

Jev























					Reply via email to