[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[GitHub] brooklyn-dist pull request #121: [BROOKLYN-579] Add DNS TTL configuration vi...

Github user tbouron commented on a diff in the pull request:

    --- Diff: karaf/apache-brooklyn/src/main/resources/bin/setenv ---
    @@ -85,6 +89,9 @@ export EXTRA_JAVA_OPTS="-Dbrooklyn.location.localhost.address= ${EXTRA_
     # Increase garbage collection, see https://issues.apache.org/jira/browse/BROOKLYN-375
     export EXTRA_JAVA_OPTS="-XX:SoftRefLRUPolicyMSPerMB=1 ${EXTRA_JAVA_OPTS}"
    +# Set the DNS TTL for the JVM
    +export EXTRA_JAVA_OPTS="-Dsun.net.inetaddr.ttl=${DNS_TTL} ${EXTRA_JAVA_OPTS}"
    --- End diff --
    @geomacy Indeed, it is annoying that we cannot control the `networkaddress.cache.ttl` directly hence why I went for the old `sun.net.inetaddr.ttl` property.
    Good idea though, I'll add a comment