You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/04/10 12:16:40 UTC

[GitHub] [pulsar] eolivelli opened a new pull request, #15105: Tiered Storage: add JClouds HttpClient driver

eolivelli opened a new pull request, #15105:
URL: https://github.com/apache/pulsar/pull/15105

   ### Motivation
   
   Apache JClouds supports Apache HttpClient as library to perform HTTP class, that is better of the standard JDK HttpUrlConnection based provider.
   
   See https://github.com/jclouds/jclouds/tree/master/drivers/apachehc
   
   ### Modifications
   
   Modifications:
   - add ApacheHCHttpCommandExecutorServiceModule module
   - add jclouds driver based on Apache HTTP Client
   - add a new ShadedJCloudsUtils that is needed to workaround some problems with the shading of Guice (AbstractModule class is problematic)
   - add system property pulsar.jclouds.use_apache_hc to disable the new driver in case of problems
   
   
   ### Verifying this change
   
   I have tested this manually.
   
   ### Does this pull request potentially affect one of the following parts:
   
   Tiered Storage
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] lhotari commented on pull request #15105: Tiered Storage: add JClouds HttpClient driver

Posted by GitBox <gi...@apache.org>.
lhotari commented on PR #15105:
URL: https://github.com/apache/pulsar/pull/15105#issuecomment-1094328505

   @eolivelli please share the stack trace of the blocked thread which is the original motivation to switch to use Apace HttpComponents/HttpClient for JClouds.
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] HQebupt commented on pull request #15105: Tiered Storage: add JClouds HttpClient driver

Posted by GitBox <gi...@apache.org>.
HQebupt commented on PR #15105:
URL: https://github.com/apache/pulsar/pull/15105#issuecomment-1094514340

   /pulsarbot run-failure-checks


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] eolivelli merged pull request #15105: Tiered Storage: add JClouds HttpClient driver

Posted by GitBox <gi...@apache.org>.
eolivelli merged PR #15105:
URL: https://github.com/apache/pulsar/pull/15105


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

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