Hey Dominic,

At a high level, that change seems useful to me. I cannot comment on the detail 
of the implementation, but I’m confident in those that have approved it. 

I think that the CI tests should pass before merging, though.

Regards,

Rob

> On 3 Aug 2022, at 13:21, Dominic Kim <style9...@gmail.com> wrote:
> 
> Dear whiskers.
> 
> I am about to merge this PR [1].
> This is to specify a different limit with the system default for a certain
> namespace.
> With this change, we can configure a bigger limit for a certain namespace
> without altering the system-wide configuration that impacts all systems.
> It is backward-compatible.
> 
> Since it has been open for a long time, I would merge it based on silent
> consent.
> Please feel free to share your opinions and suggestions.
> 
> [1] https://github.com/apache/openwhisk/pull/5229

Reply via email to