This is an automated email from the ASF dual-hosted git repository.

penghui pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git


The following commit(s) were added to refs/heads/master by this push:
     new 2dc2470  Comment lo4j2 script in order to save resources in 
pulsar-admin (#9309)
2dc2470 is described below

commit 2dc24705efe61fdfca3315edb29946d3a0a9e599
Author: Enrico Olivelli <[email protected]>
AuthorDate: Wed Jan 27 01:40:10 2021 +0100

    Comment lo4j2 script in order to save resources in pulsar-admin (#9309)
    
    Our default configuration file for log4j includes and example about how to 
use Javascript based filters.
    
    Enabling that lo4j feature forces log4j runtime to perform Nashorn JS 
engine initialisation, this basically slows down pulsar-admin startup.
    
    When you run on JDK11+ you also see a warning "Warning: Nashorn engine is 
planned to be removed from a future JDK release" that is very annoying for 
users.
    
    This change is part of a little series of patches about improving 
pulsar-admin startup speed
---
 conf/log4j2.yaml | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/conf/log4j2.yaml b/conf/log4j2.yaml
index 0ca3966..fea8476 100644
--- a/conf/log4j2.yaml
+++ b/conf/log4j2.yaml
@@ -40,12 +40,12 @@ Configuration:
         value: "Console"
 
   # Example: logger-filter script
-  Scripts:
-    ScriptFile:
-      name: filter.js
-      language: JavaScript
-      path: ./conf/log4j2-scripts/filter.js
-      charset: UTF-8
+#  Scripts:
+#    ScriptFile:
+#      name: filter.js
+#      language: JavaScript
+#      path: ./conf/log4j2-scripts/filter.js
+#      charset: UTF-8
 
   Appenders:
 

Reply via email to