You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by rm...@apache.org on 2012/09/14 00:37:46 UTC
svn commit: r1384581 - in /openejb/trunk/openejb:
container/openejb-core/src/main/java/org/apache/openejb/core/timer/
maven/tomee-maven-plugin/src/main/java/org/apache/openejb/maven/plugin/
tomee/tomee-util/src/main/java/org/apache/tomee/util/
Author: rmannibucau
Date: Thu Sep 13 22:37:45 2012
New Revision: 1384581
URL: http://svn.apache.org/viewvc?rev=1384581&view=rev
Log:
bad key in quick server.xml parsing
Modified:
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/core/timer/DefaultTimerThreadPoolAdapter.java
openejb/trunk/openejb/maven/tomee-maven-plugin/src/main/java/org/apache/openejb/maven/plugin/AbstractTomEEMojo.java
openejb/trunk/openejb/tomee/tomee-util/src/main/java/org/apache/tomee/util/QuickServerXmlParser.java
Modified: openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/core/timer/DefaultTimerThreadPoolAdapter.java
URL: http://svn.apache.org/viewvc/openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/core/timer/DefaultTimerThreadPoolAdapter.java?rev=1384581&r1=1384580&r2=1384581&view=diff
==============================================================================
--- openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/core/timer/DefaultTimerThreadPoolAdapter.java (original)
+++ openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/core/timer/DefaultTimerThreadPoolAdapter.java Thu Sep 13 22:37:45 2012
@@ -17,21 +17,17 @@
package org.apache.openejb.core.timer;
-import org.apache.openejb.loader.Options;
import org.apache.openejb.loader.SystemInstance;
-import org.apache.openejb.util.DaemonThreadFactory;
import org.apache.openejb.util.ExecutorBuilder;
import org.apache.openejb.util.LogCategory;
import org.apache.openejb.util.Logger;
-import org.apache.openejb.util.executor.OfferRejectedExecutionHandler;
import org.quartz.SchedulerConfigException;
import org.quartz.spi.ThreadPool;
import java.util.concurrent.Executor;
import java.util.concurrent.RejectedExecutionException;
-import java.util.concurrent.RejectedExecutionHandler;
-import java.util.concurrent.ThreadFactory;
import java.util.concurrent.ThreadPoolExecutor;
+import java.util.concurrent.TimeUnit;
/**
* @version $Rev$ $Date$
@@ -161,7 +157,17 @@ public class DefaultTimerThreadPoolAdapt
@Override
public void shutdown(final boolean arg0) {
- //TODO Seems we should never try to shutdown the thread pool, as it is shared in global scope
+ if (threadPoolExecutorUsed) {
+ final ThreadPoolExecutor tpe = (ThreadPoolExecutor) executor;
+ tpe.shutdown();
+ if (arg0) {
+ try {
+ tpe.awaitTermination(1, TimeUnit.HOURS);
+ } catch (InterruptedException e) {
+ logger.error(e.getMessage(), e);
+ }
+ }
+ }
}
public int getThreadCount() {
Modified: openejb/trunk/openejb/maven/tomee-maven-plugin/src/main/java/org/apache/openejb/maven/plugin/AbstractTomEEMojo.java
URL: http://svn.apache.org/viewvc/openejb/trunk/openejb/maven/tomee-maven-plugin/src/main/java/org/apache/openejb/maven/plugin/AbstractTomEEMojo.java?rev=1384581&r1=1384580&r2=1384581&view=diff
==============================================================================
--- openejb/trunk/openejb/maven/tomee-maven-plugin/src/main/java/org/apache/openejb/maven/plugin/AbstractTomEEMojo.java (original)
+++ openejb/trunk/openejb/maven/tomee-maven-plugin/src/main/java/org/apache/openejb/maven/plugin/AbstractTomEEMojo.java Thu Sep 13 22:37:45 2012
@@ -271,7 +271,7 @@ public abstract class AbstractTomEEMojo
}
private void removeDefaultWebapps() {
- final File webapps = new File(catalinaBase, "webapps");
+ final File webapps = new File(catalinaBase, webappDir);
if (webapps.isDirectory()) {
for (File webapp : webapps.listFiles()) {
final String name = webapp.getName();
Modified: openejb/trunk/openejb/tomee/tomee-util/src/main/java/org/apache/tomee/util/QuickServerXmlParser.java
URL: http://svn.apache.org/viewvc/openejb/trunk/openejb/tomee/tomee-util/src/main/java/org/apache/tomee/util/QuickServerXmlParser.java?rev=1384581&r1=1384580&r2=1384581&view=diff
==============================================================================
--- openejb/trunk/openejb/tomee/tomee-util/src/main/java/org/apache/tomee/util/QuickServerXmlParser.java (original)
+++ openejb/trunk/openejb/tomee/tomee-util/src/main/java/org/apache/tomee/util/QuickServerXmlParser.java Thu Sep 13 22:37:45 2012
@@ -37,7 +37,7 @@ public class QuickServerXmlParser extend
private static final String HTTP_KEY = "HTTP";
private static final String AJP_KEY = "AJP";
private static final String HOST_KEY = "host";
- private static final String APP_BASE_KEY = "host";
+ private static final String APP_BASE_KEY = "app-base";
private static final String DEFAULT_CONNECTOR_KEY = HTTP_KEY;
public static final String DEFAULT_HTTP_PORT = "8080";