You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by bi...@apache.org on 2017/02/13 11:20:14 UTC

kylin git commit: KYLIN-2428 cleanup reflection

Repository: kylin
Updated Branches:
  refs/heads/KYLIN-2428 c26f6fd5f -> 401c38a23


KYLIN-2428 cleanup reflection


Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/401c38a2
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/401c38a2
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/401c38a2

Branch: refs/heads/KYLIN-2428
Commit: 401c38a2331c8b47ef42f7a8eec7a0e44979d3cd
Parents: c26f6fd
Author: Billy Liu <bi...@apache.org>
Authored: Mon Feb 13 19:20:06 2017 +0800
Committer: Billy Liu <bi...@apache.org>
Committed: Mon Feb 13 19:20:06 2017 +0800

----------------------------------------------------------------------
 engine-spark/pom.xml                            | 42 --------------------
 .../engine/spark/KylinKryoRegistrator.java      |  9 -----
 pom.xml                                         |  8 ----
 3 files changed, 59 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/401c38a2/engine-spark/pom.xml
----------------------------------------------------------------------
diff --git a/engine-spark/pom.xml b/engine-spark/pom.xml
index a279279..258232d 100644
--- a/engine-spark/pom.xml
+++ b/engine-spark/pom.xml
@@ -67,14 +67,6 @@
             <scope>provided</scope>
         </dependency>
 
-        <!--
-        <dependency>
-            <groupId>org.reflections</groupId>
-            <artifactId>reflections</artifactId>
-            <scope>provided</scope>
-        </dependency>
-        -->
-
         <!-- Hadoop dependency -->
         <dependency>
             <groupId>org.apache.hadoop</groupId>
@@ -119,40 +111,6 @@
             <artifactId>maven-model</artifactId>
             <scope>test</scope>
         </dependency>
-
     </dependencies>
 
-    <build>
-        <plugins>
-            <plugin>
-                <groupId>org.apache.maven.plugins</groupId>
-                <artifactId>maven-shade-plugin</artifactId>
-                <executions>
-                    <execution>
-                        <phase>package</phase>
-                        <goals>
-                            <goal>shade</goal>
-                        </goals>
-                        <configuration>
-                            <minimizeJar>false</minimizeJar>
-                            <shadedArtifactAttached>true</shadedArtifactAttached>
-                            <shadedClassifierName>job</shadedClassifierName>
-                            <filters>
-                                <filter>
-                                    <artifact>*:*</artifact>
-                                    <excludes>
-                                        <exclude>META-INF/*.SF</exclude>
-                                        <exclude>META-INF/*.DSA</exclude>
-                                        <exclude>META-INF/*.RSA</exclude>
-                                    </excludes>
-                                </filter>
-                            </filters>
-                        </configuration>
-                    </execution>
-                </executions>
-            </plugin>
-        </plugins>
-    </build>
-
-
 </project>
\ No newline at end of file

http://git-wip-us.apache.org/repos/asf/kylin/blob/401c38a2/engine-spark/src/main/java/org/apache/kylin/engine/spark/KylinKryoRegistrator.java
----------------------------------------------------------------------
diff --git a/engine-spark/src/main/java/org/apache/kylin/engine/spark/KylinKryoRegistrator.java b/engine-spark/src/main/java/org/apache/kylin/engine/spark/KylinKryoRegistrator.java
index 5a6eee1..9168c37 100644
--- a/engine-spark/src/main/java/org/apache/kylin/engine/spark/KylinKryoRegistrator.java
+++ b/engine-spark/src/main/java/org/apache/kylin/engine/spark/KylinKryoRegistrator.java
@@ -51,15 +51,6 @@ public class KylinKryoRegistrator implements KryoRegistrator {
         kyroClasses.add(java.util.HashMap.class);
         kyroClasses.add(java.util.TreeMap.class);
         kyroClasses.add(java.util.Properties.class);
-//        kyroClasses.addAll(new Reflections("org.apache.kylin").getSubTypesOf(Serializable.class));
-//        kyroClasses.addAll(new Reflections("org.apache.kylin.dimension").getSubTypesOf(Serializable.class));
-//        kyroClasses.addAll(new Reflections("org.apache.kylin.cube").getSubTypesOf(Serializable.class));
-//        kyroClasses.addAll(new Reflections("org.apache.kylin.cube.model").getSubTypesOf(Object.class));
-//        kyroClasses.addAll(new Reflections("org.apache.kylin.metadata").getSubTypesOf(Object.class));
-//        kyroClasses.addAll(new Reflections("org.apache.kylin.metadata.model").getSubTypesOf(Object.class));
-//        kyroClasses.addAll(new Reflections("org.apache.kylin.metadata.measure").getSubTypesOf(Object.class));
-//        kyroClasses.addAll(new Reflections("org.apache.kylin.metadata.datatype").getSubTypesOf(org.apache.kylin.common.util.BytesSerializer.class));
-//        kyroClasses.addAll(new Reflections("org.apache.kylin.measure").getSubTypesOf(MeasureIngester.class));
 
         kyroClasses.add(org.apache.spark.sql.Row[].class);
         kyroClasses.add(org.apache.spark.sql.Row.class);

http://git-wip-us.apache.org/repos/asf/kylin/blob/401c38a2/pom.xml
----------------------------------------------------------------------
diff --git a/pom.xml b/pom.xml
index 72ef9ec..a103ae0 100644
--- a/pom.xml
+++ b/pom.xml
@@ -582,14 +582,6 @@
                 <version>${kryo.version}</version>
                 <scope>provided</scope>
             </dependency>
-            <!--
-            <dependency>
-                <groupId>org.reflections</groupId>
-                <artifactId>reflections</artifactId>
-                <version>${reflections.version}</version>
-                <scope>provided</scope>
-            </dependency>
-            -->
             <!-- Kafka dependency -->
             <dependency>
                 <groupId>org.apache.kafka</groupId>