plow-technologies / servant-streaming

Support for servant requests and responses via the 'streaming' library
13 stars 3 forks source link

Transient error with a test #12

Open domenkozar opened 6 years ago

domenkozar commented 6 years ago

https://nix-cache.s3.amazonaws.com/log/bhwgyywz2a4lslwk1n7xkf49yp19id51-servant-streaming-server-0.3.0.0.drv


Failures:

  test/Servant/Streaming/ServerSpec.hs:53:3: 
  1) Servant.Streaming.Server, StreamBody instance, does not keep the request in memory
       uncaught exception: HttpException
       HttpExceptionRequest Request {
         host                 = "localhost"
         port                 = 45355
         secure               = False
         requestHeaders       = [("Content-Type","application/json")]
         path                 = "length"
         queryString          = ""
         method               = "POST"
         proxy                = Nothing
         rawBody              = False
         redirectCount        = 10
         responseTimeout      = ResponseTimeoutNone
         requestVersion       = HTTP/1.1
       }
        NoResponseDataReceived

  To rerun use: --match "/Servant.Streaming.Server/StreamBody instance/does not keep the request in memory/"

Randomized with seed 73381978

Nix tests can run under quite heavy load, so probably a quick workaround would be to just increase default timeout.

jkarni commented 6 years ago

We can bump up the memory it checks against to maybe 500mb? Hard to say how much other components are using...

domenkozar commented 6 years ago

It fails with NoResponseDataReceived, so you mean to reduce 1000M stream into 500M?

jkarni commented 6 years ago

Oh, I didn't actually read the error! Sorry! I guess I have no clue what's going on...

On Mon, Jul 23, 2018 at 4:19 PM, Domen Kožar notifications@github.com wrote:

It fails with NoResponseDataReceived, so you mean to reduce 1000M stream into 500M?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/plow-technologies/servant-streaming/issues/12#issuecomment-407074284, or mute the thread https://github.com/notifications/unsubscribe-auth/ABlKmrplrXRT3voUUzxJUnj7js-5j7KUks5uJdtUgaJpZM4Va4wF .