andymck
andymck
putting to draft until i get tests regressions addressed
@tsloughter I am curious to get your views on this change, mainly in relation to handling msgs in the stream context as opposed to spawning a new process. As noted...
@tsloughter We were seeing the grpcbox_socket process go down sporadically, and during init it would fail to restart as it hit eaddrinuse. The only scenario in which I can reproduce...
> Why not return an application level error since it’s the right thing to do? If we need it out to fix an ongoing issue this is the quickest thing...
> Looks like `blockchain_ledger_gateway_v2:gain/1` can also return `undefined`; should we also handle that and return an empty binary in that case as well? why would we want to return an...
> > > Looks like `blockchain_ledger_gateway_v2:gain/1` can also return `undefined`; should we also handle that and return an empty binary in that case as well? > > > > >...
This will obviously need updated to account for any changes to the associated proto PR but getting https://github.com/helium/oracles/pull/115/ merged first would also be recommended and apply the handling of the...