LGTM in principle, but I'd definitely like to see a bit more thought on error handling before landing this, since at present it could be difficult to debug what's going on if there's some kind of outage and I'm not sure the implemented behaviour in that case is necessarily what we want.
LGTM in principle, but I'd definitely like to see a bit more thought on error handling before landing this, since at present it could be difficult to debug what's going on if there's some kind of outage and I'm not sure the implemented behaviour in that case is necessarily what we want.