[GitHub] [pulsar] Nicklee007 commented on a diff in pull request #16143: [improve][broker]Specify default nar tmp dir

[GitBox <gi...@apache.org>]

Nicklee007 commented on code in PR #16143:
URL: https://github.com/apache/pulsar/pull/16143#discussion_r1066774533


##########
bin/pulsar:
##########
@@ -307,6 +307,9 @@ if [[ -z "$IS_JAVA_8" ]]; then
   OPTS="$OPTS --add-opens jdk.management/com.sun.management.internal=ALL-UNNAMED"
 fi
 
+# Specify default nar tmp dir
+OPTS="$OPTS -Dnar.extraction.tmpdir=$PULSAR_HOME/tmp"

Review Comment:
   @eolivelli Relate to PR https://github.com/apache/pulsar/pull/15979, we define `nar.extraction.tmpdir` ， the dir only contain some class file which extraction from nar package, use the dir in pulsar can avoid those .class file be clean by os in some linux os;
   Diff linux os has diff behaviours for the /tmp dir clean, Debian will clean when the boot , but as Centos or RedHat will auto clean the file in the /tmp dir when 7 days or 30 days the file not be used, In this way, the .class file will loss.
   In our case, we add the KOP protocol, after few days when we want use the KOP protocol, we find some .class have been cleaned by os.



-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org