Rafał Leszko
Rafał Leszko
> sel.warmPool.selector.Remove(params.l Good point. I've fired: https://github.com/livepeer/go-livepeer/pull/3640 Then, we can check if we still see `try=2` in the logs. Thanks @ad-astra-video
> > sel.warmPool.selector.Remove(params.l > > Good point. I've fired: #3640 > > Then, we can check if we still see `try=2` in the logs. Thanks @ad-astra-video For some reason, I...
> Maybe we can just keep the loop and use a lower count. 5 maybe? Will allow for more time to see it work in the wild and what causes...
@ad-astra-video Updated with your suggestions. PTAL.
Ok, thanks @j0sh . Then, closing this one
> The PR looks fine but a little confused about why we need it, since I thought we were doing time based payments now instead of pixel based Well... it's...
> Alternative PR that makes things more deterministic and avoids the need for a sleep: #3663 Thanks @j0sh . Then, closing this one!
Thanks for the comments Josh! > Thanks for this PR - in addition to the comment about reordering the `conn`, is this PR still needed now that we have #3613...
Closing for now, since there are other PRs from @j0sh that covers this.
> > So what will happen now? The frontend broadcast compoenent will retry it, right? > > If the app is still there, yes it should - which happens rarely....