You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@felix.apache.org by GitBox <gi...@apache.org> on 2020/06/11 07:39:59 UTC

[GitHub] [felix-dev] pderop opened a new pull request #28: [FELIX-6286] Concurrency issue in Gogo runtime Activator

pderop opened a new pull request #28:
URL: https://github.com/apache/felix-dev/pull/28


   Could someone please review this PR, I think it fixes the concurrency issue reported in FELIX-6286.
   The only concerns is about the newProcessor(ThreadIO tio, BundleContext context) from runtime/src/main/java/org/apache/felix/gogo/runtime/activator/Activator.java:  
   
   Before this method was returning a ServiceRegistration, now it is reterning a CommandProcessorImpl object. Will this break something ? I see that the org.apache.felix.gogo.runtime.activator is exported ...
   
   thanks.
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org