Unnecessary and thread-blocking "synchronized" in MVELSafeHelper.java --------------------------------------------------------------------- Key: DROOLS-745 URL: https://issues.jboss.org/browse/DROOLS-745 Project: Drools Issue Type: Bug Components: core engine Affects Versions: 6.2.0.Final Reporter: Andrey Bichkevskiy Assignee: Mario Fusco Labels: drools-core Fix For: 6.3.0.Beta1 Original Estimate: 5 minutes Remaining Estimate: 5 minutes Class org.drools.core.util.MVELSafeHelper, obsolete and thread-blocking "synchronized" keyword in static getter of a static final field: public class MVELSafeHelper { private static final MVELEvaluator evaluator; static { evaluator = KiePolicyHelper.isPolicyEnabled() ? new SafeMVELEvaluator() : new RawMVELEvaluator(); } public static synchronized MVELEvaluator getEvaluator() { return evaluator; } // }