You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by Adrian Crum <ad...@sandglass-software.com> on 2013/05/14 09:08:41 UTC

Re: svn commit: r1482000 - /ofbiz/trunk/framework/base/src/org/ofbiz/base/container/ContainerLoader.java

Thank you for fixing this Adam. I've always wanted to fix that but never 
had the time.

-Adrian

On 5/13/2013 7:01 PM, doogie@apache.org wrote:
> Author: doogie
> Date: Mon May 13 18:01:47 2013
> New Revision: 1482000
>
> URL: http://svn.apache.org/r1482000
> Log:
> FIX/FEATURE: When shutting down, make certain to close down log4j as
> well.  This will ensure that any asynchronous logs have finished
> writing to disk, or to the console.
>
> Modified:
>      ofbiz/trunk/framework/base/src/org/ofbiz/base/container/ContainerLoader.java
>
> Modified: ofbiz/trunk/framework/base/src/org/ofbiz/base/container/ContainerLoader.java
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/container/ContainerLoader.java?rev=1482000&r1=1481999&r2=1482000&view=diff
> ==============================================================================
> --- ofbiz/trunk/framework/base/src/org/ofbiz/base/container/ContainerLoader.java (original)
> +++ ofbiz/trunk/framework/base/src/org/ofbiz/base/container/ContainerLoader.java Mon May 13 18:01:47 2013
> @@ -27,6 +27,9 @@ import java.util.LinkedList;
>   import java.util.List;
>   import java.util.Map;
>   
> +import org.apache.log4j.Logger;
> +import org.apache.log4j.Appender;
> +
>   import org.ofbiz.base.component.ComponentConfig;
>   import org.ofbiz.base.start.Config;
>   import org.ofbiz.base.start.StartupException;
> @@ -265,6 +268,11 @@ public class ContainerLoader implements
>                       Debug.logInfo("Stopped container " + container.getName(), module);
>                   }
>               }
> +            Enumeration<Appender> en = Logger.getRootLogger().getAllAppenders();
> +            while (en.hasMoreElements()) {
> +                Appender appender = en.nextElement();
> +                appender.close();
> +            }
>           }
>       }
>   }
>
>