yifan-c commented on code in PR #126: URL: https://github.com/apache/cassandra-sidecar/pull/126#discussion_r1635597079
########## gradlew: ########## @@ -105,8 +105,10 @@ if [ "$cygwin" = "false" -a "$darwin" = "false" -a "$nonstop" = "false" ] ; then fi # For Darwin, add options to specify how the application appears in the dock +# In addition, we want to disable the MaxFDLimit for MacOS, which is set to a lower limit than the actual system maximum. Review Comment: To clarify, the JVM option `-XX:-MaxFDLimit` is to _disable_ the low `MaxFDLimit` set in JDK. By disabling, it can pick up the system FD limit, which is larger and proper. The suggested sentence looks incorrect. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
