provide way to clear the state on the ThreadLocal used byby ExtractorFactory

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1907581 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 16 insertions, 0 deletions

diff --git a/poi/src/main/java/org/apache/poi/extractor/ExtractorFactory.java b/poi/src/main/java/org/apache/poi/extractor/ExtractorFactory.java
index 28b81e9cc4..3e8f046112 100644
--- a/poi/src/main/java/org/apache/poi/extractor/ExtractorFactory.java
+++ b/poi/src/main/java/org/apache/poi/extractor/ExtractorFactory.java
@@ -111,6 +111,12 @@ public final class ExtractorFactory {
      * Should this thread prefer event based over usermodel based extractors?
      * Will only be used if the All Threads setting is null.
      *
+     * <p>
+     *     This uses ThreadLocals and these can leak resources when you have a lot of threads.
+     * </p>
+     *
+     * You should always try to call {@link #removeThreadPrefersEventExtractorsSetting()}.
+     *
      * @param preferEventExtractors If this threads should prefer event based extractors.
      */
     public static void setThreadPrefersEventExtractors(boolean preferEventExtractors) {
@@ -118,6 +124,16 @@ public final class ExtractorFactory {
     }
 
     /**
+     * Clears the setting for this thread made by {@link #setThreadPrefersEventExtractors(boolean) }
+     *
+     * @see #setThreadPrefersEventExtractors(boolean)
+     * @since POI 5.2.4
+     */
+    public static void removeThreadPrefersEventExtractorsSetting() {
+        threadPreferEventExtractors.remove();
+    }
+
+    /**
      * Should all threads prefer event based over usermodel based extractors?
      * If set, will take preference over the Thread level setting.
      *