On Thursday, July 21, 2016 at 6:15:46 AM UTC+2, James Reeves wrote:
>
> On 21 July 2016 at 05:05, Ashish Negi <thisismy...@gmail.com <javascript:>
> > wrote:
>
>> with core async `go` you can not do blocking IO or any time consuming 
>> work.
>> `go` uses fixed threadpool (no of cpus + 2 or something).
>> and in your async-handler you are using `<!! c` which is blocking . 
>> https://clojure.github.io/core.async/
>>
>> Try with `future`. https://clojuredocs.org/clojure.core/future
>>
>
> While futures will work, it's worth pointing out you'd get the same result 
> just by increasing the thread count on the adapter. Either way you're using 
> up a thread per request.
>
> Since the database functions are not asynchronous, core.async isn't really 
> useful in this context. My suggestion would be to drop it altogether.
>
> - James
>

http-kit threadpool was definitely bottleneck, and probably hakiri-cp 
connection pool.
I reverted to initial version without core.async and bumped both pools to 
128 and now CPU is fully saturated, and performance is much better. 
Comparable to Go version I wrote, maybe even better.
Thanks you both for your time :)

-- 
You received this message because you are subscribed to the Google
Groups "Clojure" group.
To post to this group, send email to clojure@googlegroups.com
Note that posts from new members are moderated - please be patient with your 
first post.
To unsubscribe from this group, send email to
clojure+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/clojure?hl=en
--- 
You received this message because you are subscribed to the Google Groups 
"Clojure" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to clojure+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to