search

fabriziosestito / commanded-spear-adapter

Spear event store adapter for Commanded
MIT License
19 stars 5 forks source link

Use separate connection for read/write #57

Closed dvic closed 1 year ago

dvic commented 1 year ago

@fabriziosestito we found a bizar bug again: with many concurrent (large) writes it seems like the HTTP2 window size fills up and the persistent subscription stops sending events for ~10 seconds. Using two connections fixes this issue.

I have a reproducible script for this if you want poke around yourself but for now I think this is an acceptable solution.

fabriziosestito commented 1 year ago

@dvic hey yes could pleaase link me a gist with the script. I am curious to check it out!

dvic commented 1 year ago

@dvic hey yes could pleaase link me a gist with the script. I am curious to check it out!

Sure! https://gist.github.com/dvic/a2177efd0d95fc6ff755b2650928cbca

dvic commented 1 year ago

now that https://github.com/NFIBrokerage/spear/issues/87#issuecomment-1496430632 is fixed i think we can revert this one?