Copilot commented on code in PR #3618:
URL: https://github.com/apache/fluss/pull/3618#discussion_r3607492297
##########
website/docs/maintenance/configuration.md:
##########
@@ -54,6 +54,22 @@ during the Fluss cluster working.
| server.io-pool.size | Integer |
10
| The size of the IO thread pool to run blocking operations for both
coordinator and tablet servers. This includes discard unnecessary snapshot
files, transfer kv snapshot files, and transfer remote log files. Increase this
value if you experience slow IO operations. The default value is 10.
|
+## Environment
+
+The following options control the launch environment of the Fluss server
processes, such as the JVM options and the directory for process id files. They
are set in `conf/server.yaml` and are applied by the startup scripts under
`bin/` when starting the CoordinatorServer and TabletServer.
+
+| Option | Type | Default | Description
|
+|----------------------------------|--------|---------|---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
+| env.java.home | String | (None) | Path to the JDK/JRE
used to run the Fluss processes. If not set, the system `JAVA_HOME` or the
`java` binary found on the `PATH` is used.
|
+| env.java.opts.all | String | (None) | JVM options applied to
all Fluss server processes, i.e. both the CoordinatorServer and the
TabletServer, for example `-Xmx4g -XX:+UseG1GC`.
|
Review Comment:
The table lists `env.java.opts.all` default as “(None)”, but `bin/config.sh`
always prepends `-XX:+IgnoreUnrecognizedVMOptions` to `FLUSS_ENV_JAVA_OPTS`
even when the config key is unset (and may also add
`-Djava.security.manager=allow` on newer JVMs). This makes the documented
default misleading; consider reflecting the actual default behavior either in
the Default column or in the description.
--
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]
