You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by ce...@apache.org on 2017/08/24 19:46:39 UTC

svn commit: r1806077 - /poi/trunk/src/ooxml/java/org/apache/poi/POIXMLTypeLoader.java

Author: centic
Date: Thu Aug 24 19:46:39 2017
New Revision: 1806077

URL: http://svn.apache.org/viewvc?rev=1806077&view=rev
Log:
Revert performance tuning done via Bug 61350 as it lead to problems when running multi-threaded large regression tests.

Modified:
    poi/trunk/src/ooxml/java/org/apache/poi/POIXMLTypeLoader.java

Modified: poi/trunk/src/ooxml/java/org/apache/poi/POIXMLTypeLoader.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/POIXMLTypeLoader.java?rev=1806077&r1=1806076&r2=1806077&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/POIXMLTypeLoader.java (original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/POIXMLTypeLoader.java Thu Aug 24 19:46:39 2017
@@ -71,7 +71,9 @@ public class POIXMLTypeLoader {
         // DEFAULT_XML_OPTIONS.setLoadEntityBytesLimit(4096);
         
         // POI is not thread-safe - so we can switch to unsynchronized xmlbeans mode - see #61350
-        DEFAULT_XML_OPTIONS.setUnsynchronized();
+        // Update: disabled again for now as it caused strange NPEs and other problems
+        // when reading properties in separate workbooks in multiple threads
+        // DEFAULT_XML_OPTIONS.setUnsynchronized();
 
         Map<String, String> map = new HashMap<String, String>();
         map.put("http://schemas.openxmlformats.org/drawingml/2006/main", "a");



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org