roryqi
roryqi
@jerryshao Could you review again?
You need to restart the Trino if you rename the metalake and modify the configuration, so this isn't an issue. We should close it.
@zuston Have you committed this to branch 0.9?
@zuston When you merge a pull request, you should say which branch is merged. like ``` merged to master & branch 0.9. ```
> > @zuston When you merge a pull request, you should say which branch is merged. like > > ``` > > merged to master & branch 0.9. > >...
@dingshun3016 Could you create an issue for this?
You can also use API `/configs`. The API won't need authentication.
> As stated in the issue description, readiness check is expected to indicate that the service can serve requests. So I don't think `/configs` is a proper endpoint for this...
> Not sure I'm understanding this correctly, but I won't recommend a new repo for authz plugins. A separate repo takes a huge effort to set up and to review...
@mchades Should we include this fix in branch 0.9?