zachjsh

Results 6 comments of zachjsh

Looks like it is failing code coverage, weird, no code was added / removed. Tried restarting those jobs, lets see.

@somu-imply , thanks for adding this new monitoring. One suggestion; would you add more details to the git message with more explanation of what was implemented here and why, and...

> Here's how we propose to revise the REST API. Comments or suggestions? > > Use a form of `{topic}/{path}`, where the topic is the kind of action. This resolves...

> We really have to resist the temptation to add a config for every corner case we run into. Is it absolutely necessary to have this config? Even if you...

> I am not entirely sure we need a config here either. Irrespective of what we name the config, it is going to cause confusion with the existing `timeoutMs`, especially...