Copilot commented on code in PR #9933: URL: https://github.com/apache/gravitino/pull/9933#discussion_r2785701210
########## conf/gravitino-env.sh.template: ########## @@ -35,4 +35,4 @@ GRAVITINO_VERSION=GRAVITINO_VERSION_PLACEHOLDER # UI selection behaviour: # - By default the template sets `GRAVITINO_USE_WEB_V2=false` which forces the V1 UI. # - To enable the v2 UI you can set it to `true` (for example: `export GRAVITINO_USE_WEB_V2=true`). Review Comment: The surrounding comment is now inaccurate: the template no longer sets `GRAVITINO_USE_WEB_V2=false`, but line 36 still says it does. Please update the UI selection behaviour comment to reflect that `GRAVITINO_USE_WEB_V2` is intentionally left unset by default (so container env can control it) and that users can optionally set it explicitly if desired. ```suggestion # - By default this template does not set `GRAVITINO_USE_WEB_V2`, so the container or environment can control which UI is used. # - To force the v1 UI, explicitly set it to `false`; to enable the v2 UI, set it to `true` (for example: `export GRAVITINO_USE_WEB_V2=true`). ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
