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]

Reply via email to