Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Feb 05 18:40
    kennykong starred http4s/http4s
  • Feb 05 14:43
    danicheg review_requested #6937
  • Feb 05 14:43
    danicheg commented #6937
  • Feb 05 09:34
    danicheg synchronize #6957
  • Feb 05 09:22
    mergify[bot] labeled #6957
  • Feb 05 09:21
    danicheg opened #6957
  • Feb 04 18:43
    rossabaker commented #789
  • Feb 04 09:21

    github-actions[bot] on gh-pages

    deploy: 0e253e30bf983545cb0ebdd… (compare)

  • Feb 04 09:14

    armanbilge on series

    (compare)

  • Feb 04 09:14

    armanbilge on 0.23

    Update cats-effect, cats-effect… Merge pull request #6956 from h… (compare)

  • Feb 04 09:14
    armanbilge closed #6956
  • Feb 04 09:12
    plokhotnyuk commented #789
  • Feb 04 08:28
    mergify[bot] labeled #6956
  • Feb 04 08:28
    mergify[bot] labeled #6956
  • Feb 04 08:27
    http4s-steward[bot] opened #6956
  • Feb 04 08:27

    http4s-steward[bot] on series

    Update cats-effect, cats-effect… (compare)

  • Feb 04 04:52
    rossabaker commented #789
  • Feb 04 00:29
    bryce-anderson commented #789
  • Feb 03 21:05
    danicheg commented #789
  • Feb 03 19:09
    SethTisue commented #789
Rob Norris
@tpolecat
almost have it working with Client[IO] + LiftIO
Christopher Davenport
@ChristopherDavenport
@Daenyth I have some internal work on rewriting scalaxb to be on cats-effec.t
Ross had a better PR a long time ago.
(By internal I mean not released, not closed source)
Ross A. Baker
@rossabaker
My PR to scalaxb is unpolished and probably of the cats-effect-0.10 era, but it did the hello-world thing.
Niklas Klein
@Taig
I'm trying to send an HTTP request with blaze client that contains a timestamp. The server invalidates the request if the timestamp is a couple seconds off. Unfortunately (especially during testing) the request seems occasionally to be queued for a little while so that the server will reject it. I was hoping the fetch[A](req: F[Request[F]]) method would only evaluate the request when it is ready to be sent, but that does not appear to be the case. Is that correct / expected? Do I have any other options to solve that?
Christopher Davenport
@ChristopherDavenport
The client should get around to any request is less than a couple seconds unless you are horrifically slow in some area.
Ross A. Baker
@rossabaker
@mpilquist How does the prioritization of fromConcurrentEffect and fromAsyncAndContextShift work? Does it look for the unary before the binary?
Christopher Davenport
@ChristopherDavenport
In tests specifically though, you can likely just increase the pool size.
Ross A. Baker
@rossabaker
I don't think we could defer creating the request until it's ready to be sent, because we need to see the request's scheme and authority to know whether there is a connection available with which to send it.
Well, I guess we could theoretically enforce global constraints first and then local on demand, but that would make a hard problem harder.
Christopher Davenport
@ChristopherDavenport
Ross wants linear types again. Waits for Linear Scala plugin to be released.
Niklas Klein
@Taig
Alright, I'll increase the pool then. I'm already at 128 max connections and am starting to feel a little uneasy about going higher. But I guess it's not that much of a problem in tests.
Christopher Davenport
@ChristopherDavenport
Could create a client specifically for the time dependent calls you have in there.
Niklas Klein
@Taig
Yes, I think that's a better idea. I'll need some kind of priority handling anyway so that is probably the way to go. Thank you (:
Gavin Bisesi
@Daenyth
@Taig You could also make sure that if you are passing a blockingEC that it's an unbounded cached thread pool, and perhaps sprinkle some ContextShift[F] calls around
to hopefully help improve any fairness issues
Ross A. Baker
@rossabaker
Maybe a semaphore in front could limit the requests being made to a client such that they spend less time queued up in the client.
Christopher Davenport
@ChristopherDavenport
I like Ross’s, thats nice.
Niklas Klein
@Taig
It's already backed by such a thread pool. I'm not entirely sure how shifting may help though? The app itself is already highly concurrent.
Gavin Bisesi
@Daenyth
then it probably wouldn't
What are you doing with the responses? My first guess would be that you're spending cpu pool time handling responses and that's why things are queueing up
Niklas Klein
@Taig
@rossabaker I like that. Going down that route (:
Ross A. Baker
@rossabaker
If requests are sitting in the wait queue on the blaze client pool, I think my idea might help. If they're not, I think it won't.
Instead of semaphores, making a Stream of requests and using its concurrency controls may be worth exploring as well.
While people are awake, we have a couple open PRs that would be nice to get an 0.20.2 out.
Gavin Bisesi
@Daenyth
^ I'm a big fan of that approach, it makes it quite easy to handle rate limiting requests, retries, etc
Niklas Klein
@Taig
@Daenyth That's an interesting catch. I wasn't aware that might be blocking the queue. I'm basically just parsing json, but I might wanna step into the flamegrapghs to find out more.
Gavin Bisesi
@Daenyth
oof that reminds me, I need to get us up from 0.20.0-M5
@Taig json parsing is heavily cpu bound IME
Christopher Davenport
@ChristopherDavenport
I’ve been on vacation, my notification list is 29 http4s issues, with many of them PR’s what is most pressing?
Gavin Bisesi
@Daenyth
I'd definitely do something like fs2.Stream.emits(requests).mapAsyncUnordered(cpuCount)(req => send(req).flatMap(handleResp)) or something
Ross A. Baker
@rossabaker
I just milestoned the 0.20.2 ones
Christopher Davenport
@ChristopherDavenport
Where doews #2604 invoke getDefault? Are we worried this will bite us?
i.e. behavior will change with default behavior having an SSL context or having no ssl context.
As if we were deferring creation, I’d expect I’d see where we called that later with the delayed call.
Ross A. Baker
@rossabaker
getDefault
I don't really like it. If you're on a system where getDefault throws, it just defers the explosion until the first https call. But the reason it's there is that it gives people who know that getDefault throws a chance to override it. The crucial point is that getDefault not appear as a default argument. That's why we've gotten that bug filed ... twice.
Christopher Davenport
@ChristopherDavenport
Alright. Lets go with it, but lets make it big in the release notes.
We
*We’ll still get the issue
Ross A. Baker
@rossabaker
I would like to do better there, but it's the best I could figure out to do without breaking bincompat.
Christopher Davenport
@ChristopherDavenport
I believe that unblocks 0.20.2
Ross A. Baker
@rossabaker
There's absolutely no way to create an SSLContext that's guaranteed to work, which suggests maybe it should be an Option.
Christopher Davenport
@ChristopherDavenport
Yeah, I think it will still blow up though.
So…
Ross A. Baker
@rossabaker
If it were optional, it would fail with a message of our choosing if someone attempts to make an https call and doesn't have it configured.
And it still needs to be lazy, because Some(sslContext.getDefault) is the right choice for most people.
Christopher Davenport
@ChristopherDavenport
lazy should not be our tool to defer.
but, thats for a binary-breaking change.