Revert KYLIN-2361, Downgrade to Tomcat 7.0.69
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/51c65710 Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/51c65710 Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/51c65710 Branch: refs/heads/master-hbase1.x Commit: 51c657108d4dae221a1d4d090ba68a1d0948e8f7 Parents: 131a3f3 Author: Billy Liu <billy...@apache.org> Authored: Wed Jan 18 17:15:11 2017 +0800 Committer: Billy Liu <billy...@apache.org> Committed: Wed Jan 18 17:15:11 2017 +0800 ---------------------------------------------------------------------- build/script/download-tomcat.sh | 12 ++++++------ pom.xml | 2 +- .../java/org/apache/kylin/rest/DebugTomcat.java | 16 +++++++--------- .../kylin/ext/CustomizedWebappClassloader.java | 6 +++--- 4 files changed, 17 insertions(+), 19 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/kylin/blob/51c65710/build/script/download-tomcat.sh ---------------------------------------------------------------------- diff --git a/build/script/download-tomcat.sh b/build/script/download-tomcat.sh index 403d87b..b3aa509 100755 --- a/build/script/download-tomcat.sh +++ b/build/script/download-tomcat.sh @@ -27,25 +27,25 @@ if [[ `uname -a` =~ "Darwin" ]]; then alias md5cmd="md5 -q" fi -tomcat_pkg_version="8.5.9" -tomcat_pkg_md5="b41270a64b7774c964e4bec813eea2ed" +tomcat_pkg_version="7.0.69" +tomcat_pkg_md5="10a071e5169a1a8b14ff35a0ad181052" if [ ! -f "build/apache-tomcat-${tomcat_pkg_version}.tar.gz" ] then echo "no binary file found" - wget --directory-prefix=build/ http://archive.apache.org/dist/tomcat/tomcat-8/v${tomcat_pkg_version}/bin/apache-tomcat-${tomcat_pkg_version}.tar.gz || echo "Download tomcat failed" + wget --directory-prefix=build/ http://archive.apache.org/dist/tomcat/tomcat-7/v${tomcat_pkg_version}/bin/apache-tomcat-${tomcat_pkg_version}.tar.gz || echo "Download tomcat failed" else if [ `md5cmd build/apache-tomcat-${tomcat_pkg_version}.tar.gz | awk '{print $1}'` != "${tomcat_pkg_md5}" ] then echo "md5 check failed" rm build/apache-tomcat-${tomcat_pkg_version}.tar.gz - wget --directory-prefix=build/ http://archive.apache.org/dist/tomcat/tomcat-8/v${tomcat_pkg_version}/bin/apache-tomcat-${tomcat_pkg_version}.tar.gz || echo "download tomcat failed" + wget --directory-prefix=build/ http://archive.apache.org/dist/tomcat/tomcat-7/v${tomcat_pkg_version}/bin/apache-tomcat-${tomcat_pkg_version}.tar.gz || echo "download tomcat failed" fi fi unalias md5cmd -tar -zxvf build/apache-tomcat-8.5.9.tar.gz -C build/ -mv build/apache-tomcat-8.5.9 build/tomcat +tar -zxvf build/apache-tomcat-${tomcat_pkg_version}.tar.gz -C build/ +mv build/apache-tomcat-${tomcat_pkg_version} build/tomcat rm -rf build/tomcat/webapps/* mv build/tomcat/conf/server.xml build/tomcat/conf/server.xml.bak http://git-wip-us.apache.org/repos/asf/kylin/blob/51c65710/pom.xml ---------------------------------------------------------------------- diff --git a/pom.xml b/pom.xml index e8ccb6c..ff4c4e8 100644 --- a/pom.xml +++ b/pom.xml @@ -101,7 +101,7 @@ <cglib.version>3.2.4</cglib.version> <supercsv.version>2.4.0</supercsv.version> <cors.version>2.5</cors.version> - <tomcat.version>8.5.9</tomcat.version> + <tomcat.version>7.0.69</tomcat.version> <t-digest.version>3.1</t-digest.version> <!-- REST Service --> http://git-wip-us.apache.org/repos/asf/kylin/blob/51c65710/server/src/main/java/org/apache/kylin/rest/DebugTomcat.java ---------------------------------------------------------------------- diff --git a/server/src/main/java/org/apache/kylin/rest/DebugTomcat.java b/server/src/main/java/org/apache/kylin/rest/DebugTomcat.java index 2cac82e..59b7a28 100644 --- a/server/src/main/java/org/apache/kylin/rest/DebugTomcat.java +++ b/server/src/main/java/org/apache/kylin/rest/DebugTomcat.java @@ -21,13 +21,11 @@ package org.apache.kylin.rest; import org.apache.catalina.Context; import org.apache.catalina.core.AprLifecycleListener; import org.apache.catalina.core.StandardServer; +import org.apache.catalina.deploy.ErrorPage; import org.apache.catalina.startup.Tomcat; import org.apache.commons.lang3.StringUtils; import org.apache.hadoop.util.Shell; import org.apache.kylin.common.KylinConfig; -import org.apache.tomcat.JarScanFilter; -import org.apache.tomcat.JarScanType; -import org.apache.tomcat.util.descriptor.web.ErrorPage; import java.io.File; import java.lang.reflect.Field; @@ -129,12 +127,12 @@ public class DebugTomcat { notFound.setLocation("/index.html"); webContext.addErrorPage(notFound); webContext.addWelcomeFile("index.html"); - webContext.getJarScanner().setJarScanFilter(new JarScanFilter() { - @Override - public boolean check(JarScanType arg0, String arg1) { - return false; - } - }); +// webContext.getJarScanner().setJarScanFilter(new JarScanFilter() { +// @Override +// public boolean check(JarScanType arg0, String arg1) { +// return false; +// } +// }); // tomcat start tomcat.start(); http://git-wip-us.apache.org/repos/asf/kylin/blob/51c65710/tomcat-ext/src/main/java/org/apache/kylin/ext/CustomizedWebappClassloader.java ---------------------------------------------------------------------- diff --git a/tomcat-ext/src/main/java/org/apache/kylin/ext/CustomizedWebappClassloader.java b/tomcat-ext/src/main/java/org/apache/kylin/ext/CustomizedWebappClassloader.java index 23bde2c..816601f 100644 --- a/tomcat-ext/src/main/java/org/apache/kylin/ext/CustomizedWebappClassloader.java +++ b/tomcat-ext/src/main/java/org/apache/kylin/ext/CustomizedWebappClassloader.java @@ -45,7 +45,7 @@ public class CustomizedWebappClassloader extends ParallelWebappClassLoader { * @param name class name * @return true if the class should be filtered */ - protected boolean filter(String name, boolean isClassName) { + protected boolean filter(String name) { if (name == null) return false; @@ -62,7 +62,7 @@ public class CustomizedWebappClassloader extends ParallelWebappClassLoader { return true; } - return super.filter(name, isClassName); - + //return super.filter(name, isClassName); + return false; } }
