async-http-client icon indicating copy to clipboard operation
async-http-client copied to clipboard

Suggestion

Open armor009 opened this issue 5 years ago • 4 comments

Is it possible to have an async .execute without all the callback and function specification. It is very messy and dirty to have to define all the possible call back methods. See parseq http-client by LinkedIn built on top of this systems predecessor ning-http-client https://github.com/linkedin/parseq/blob/master/subprojects/parseq-http-client/src/main/java/com/linkedin/parseq/httpclient/HttpClient.java

armor009 avatar Nov 10 '20 18:11 armor009

@mayank-mundhra-2012 Don't see why not, but I'd need a better description than this to get going. How would the API look like from your perspective?

TomGranot avatar Nov 10 '20 21:11 TomGranot

@mayank-mundhra-2012 Ping on this when you get a moment:)

TomGranot avatar Nov 17 '20 08:11 TomGranot

  1. Maybe you could have a look at the parseq repo and readme. The base repo of parseq is https://github.com/linkedin/parseq It has parallel async calls

  2. parseq-http-client Uses ning client and has a wrapper around it to prevent the complicated functions and callables that compulsorily need to be developed and written everytime we do an async call. https://github.com/linkedin/parseq/tree/master/subprojects/parseq-http-client

  3. You could also look at optimisations and innovations made at Restli https://linkedin.github.io/rest.li/

Thank you.

I hope this is useful for enhancing the async-http-client. I am currently using async-http-client and hope it gets strengthened even further. Really like the positivity you bring to the project. Please keep that alive.

On Tue, 17 Nov 2020, 1:48 pm Tom Granot, [email protected] wrote:

@mayank-mundhra-2012 https://github.com/mayank-mundhra-2012 Ping on this when you get a moment:)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/AsyncHttpClient/async-http-client/issues/1744#issuecomment-728766177, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB2AE7DNJTHBUWZIAQSYIBDSQIWVVANCNFSM4TRA4UXA .

armor009 avatar Nov 17 '20 16:11 armor009

@mayank-mundhra-2012 Thank you very much for the comments! We do it for the fun of it, after all....

Will look into this once the final housekeeping (maven central, etc..) stuff is all done.

TomGranot avatar Nov 20 '20 05:11 TomGranot