orangeCatDeveloper commented on code in PR #3618:
URL: https://github.com/apache/fluss/pull/3618#discussion_r3607505510
##########
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:
Good catch on the script behavior. These flags are injected by
`bin/config.sh` unconditionally (regardless of whether `env.java.opts.all` is
set), so they are not really a *default value* of this option — keeping the
Default column as `(None)`, consistent with how Flink documents
`env.java.opts.all`. Added a note to the description instead: the startup
scripts always prepend `-XX:+IgnoreUnrecognizedVMOptions` (and
`-Djava.security.manager=allow` on JDK 18+).
--
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]