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


##########
docs/iceberg-rest-service.md:
##########
@@ -447,6 +447,14 @@ Gravitino provides the build-in 
`org.apache.gravitino.iceberg.common.cache.Local
 
|---------------------------------------------|--------------------------------------------------------------|---------------|----------|------------------|
 | `gravitino.iceberg-rest.extension-packages` | Comma-separated list of 
Iceberg REST API packages to expand. | (none)        | No       | 
0.7.0-incubating |
 
+### Memory settings
+
+The Iceberg REST server uses `GRAVITINO_MEM` for JVM heap/metaspace flags. 
Default: `-Xms1024m -Xmx1024m -XX:MaxMetaspaceSize=512m`. The launch script 
appends `GRAVITINO_MEM` to `JAVA_OPTS`; set `GRAVITINO_MEM` to change the heap 
size used at runtime.
+Example tuning:
+- Development: `GRAVITINO_MEM="-Xms1g -Xmx1g -XX:MaxMetaspaceSize=512m"`

Review Comment:
   Inconsistent formatting: This line uses a colon after "Development" while 
line 455 uses a colon after "Medium workloads", but line 456 lacks a colon 
after "Heavier concurrency/catalog count". For consistency with the list 
format, all items should use consistent punctuation.



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