Copilot commented on code in PR #11669:
URL: https://github.com/apache/gravitino/pull/11669#discussion_r3418656492


##########
docs/manage-relational-metadata-using-gravitino.md:
##########
@@ -140,6 +140,14 @@ Gravitino supports the following catalog providers:
 | `jdbc-hologres`     | [Hologres catalog 
property](./jdbc-hologres-catalog.md#catalog-properties)              |
 | `lakehouse-generic` | [Lakehouse generic catalog 
property](./lakehouse-generic-catalog.md#catalog-properties) |
 
+:::caution
+Properties passed via the `gravitino.bypass.` prefix are **not hidden** and 
will be returned in
+plaintext by the Gravitino REST API (e.g. `GET 
/api/metalakes/{metalake}/catalogs/{catalog}`).

Review Comment:
   The caution block uses placeholder names `{metalake}` / `{catalog}` in the 
example endpoint, but the rest of this doc (and other REST references) uses 
`{metalake_name}` / `{catalog_name}`. Using consistent placeholders avoids 
confusion when copy/pasting examples.



-- 
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]

Reply via email to