steveloughran commented on PR #7295:
URL: https://github.com/apache/hadoop/pull/7295#issuecomment-2609548734

   > Do you mean create a new JIRA to add in CRT creation into ClientManager 
and S3ClientFactory? This can be done now, independent of this PR.
   
   exactly! It can go into trunk. 
   
   In #7214 There's a callback to ask for the async client, where the stream 
can explicitly request the CRT client. This will raise an exception if not 
present. we could tune the behaviour, but I do want to push the checks into 
that ClientManager class for any required use elsewhere.
   
   > Do you mean in the feature branch:
   > 1/ first merge in your changes from #7214
   > 2/ Then merge changes to add in analytics stream support (last couple of 
commits from this PR)
   
   yes -and make sure that #7214 satisfies all your needs
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to