You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by ch...@apache.org on 2019/05/03 17:54:50 UTC
[flink] branch master updated: [FLINK-12390][build] Fully migrate
to flink-shaded-asm-6
This is an automated email from the ASF dual-hosted git repository.
chesnay pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/flink.git
The following commit(s) were added to refs/heads/master by this push:
new 9975e03 [FLINK-12390][build] Fully migrate to flink-shaded-asm-6
9975e03 is described below
commit 9975e0393a9e09dbde21ca61f1a5751cc5934411
Author: Chesnay Schepler <ch...@apache.org>
AuthorDate: Wed Apr 3 11:31:32 2019 +0200
[FLINK-12390][build] Fully migrate to flink-shaded-asm-6
---
flink-core/pom.xml | 2 +-
.../api/java/typeutils/TypeExtractionUtils.java | 4 +--
flink-java/pom.xml | 5 ---
.../org/apache/flink/api/java/ClosureCleaner.java | 4 +--
flink-runtime/pom.xml | 2 +-
.../flink/runtime/util/DependencyVisitor.java | 38 +++++++++++-----------
flink-scala/pom.xml | 5 ---
pom.xml | 6 ----
8 files changed, 25 insertions(+), 41 deletions(-)
diff --git a/flink-core/pom.xml b/flink-core/pom.xml
index dd503fb..0768e81 100644
--- a/flink-core/pom.xml
+++ b/flink-core/pom.xml
@@ -49,7 +49,7 @@ under the License.
<dependency>
<groupId>org.apache.flink</groupId>
- <artifactId>flink-shaded-asm</artifactId>
+ <artifactId>flink-shaded-asm-6</artifactId>
</dependency>
<!-- standard utilities -->
diff --git a/flink-core/src/main/java/org/apache/flink/api/java/typeutils/TypeExtractionUtils.java b/flink-core/src/main/java/org/apache/flink/api/java/typeutils/TypeExtractionUtils.java
index 07f1e1e..a49e10a 100644
--- a/flink-core/src/main/java/org/apache/flink/api/java/typeutils/TypeExtractionUtils.java
+++ b/flink-core/src/main/java/org/apache/flink/api/java/typeutils/TypeExtractionUtils.java
@@ -35,8 +35,8 @@ import java.util.ArrayList;
import java.util.Collections;
import java.util.List;
-import static org.apache.flink.shaded.asm5.org.objectweb.asm.Type.getConstructorDescriptor;
-import static org.apache.flink.shaded.asm5.org.objectweb.asm.Type.getMethodDescriptor;
+import static org.apache.flink.shaded.asm6.org.objectweb.asm.Type.getConstructorDescriptor;
+import static org.apache.flink.shaded.asm6.org.objectweb.asm.Type.getMethodDescriptor;
@Internal
public class TypeExtractionUtils {
diff --git a/flink-java/pom.xml b/flink-java/pom.xml
index f91a5a9..3d633f4 100644
--- a/flink-java/pom.xml
+++ b/flink-java/pom.xml
@@ -43,11 +43,6 @@ under the License.
<dependency>
<groupId>org.apache.flink</groupId>
- <artifactId>flink-shaded-asm</artifactId>
- </dependency>
-
- <dependency>
- <groupId>org.apache.flink</groupId>
<artifactId>flink-shaded-asm-6</artifactId>
</dependency>
diff --git a/flink-java/src/main/java/org/apache/flink/api/java/ClosureCleaner.java b/flink-java/src/main/java/org/apache/flink/api/java/ClosureCleaner.java
index d4c619d..a7f3276 100644
--- a/flink-java/src/main/java/org/apache/flink/api/java/ClosureCleaner.java
+++ b/flink-java/src/main/java/org/apache/flink/api/java/ClosureCleaner.java
@@ -176,7 +176,7 @@ class This0AccessFinder extends ClassVisitor {
private boolean isThis0Accessed;
public This0AccessFinder(String this0Name) {
- super(Opcodes.ASM5);
+ super(Opcodes.ASM6);
this.this0Name = this0Name;
}
@@ -186,7 +186,7 @@ class This0AccessFinder extends ClassVisitor {
@Override
public MethodVisitor visitMethod(int access, String name, String desc, String sig, String[] exceptions) {
- return new MethodVisitor(Opcodes.ASM5) {
+ return new MethodVisitor(Opcodes.ASM6) {
@Override
public void visitFieldInsn(int op, String owner, String name, String desc) {
diff --git a/flink-runtime/pom.xml b/flink-runtime/pom.xml
index e46e703..6e7b5a3 100644
--- a/flink-runtime/pom.xml
+++ b/flink-runtime/pom.xml
@@ -90,7 +90,7 @@ under the License.
<dependency>
<groupId>org.apache.flink</groupId>
- <artifactId>flink-shaded-asm</artifactId>
+ <artifactId>flink-shaded-asm-6</artifactId>
</dependency>
<dependency>
diff --git a/flink-runtime/src/main/java/org/apache/flink/runtime/util/DependencyVisitor.java b/flink-runtime/src/main/java/org/apache/flink/runtime/util/DependencyVisitor.java
index 3a521a8..193ab02 100644
--- a/flink-runtime/src/main/java/org/apache/flink/runtime/util/DependencyVisitor.java
+++ b/flink-runtime/src/main/java/org/apache/flink/runtime/util/DependencyVisitor.java
@@ -18,16 +18,16 @@
package org.apache.flink.runtime.util;
-import org.apache.flink.shaded.asm5.org.objectweb.asm.AnnotationVisitor;
-import org.apache.flink.shaded.asm5.org.objectweb.asm.ClassVisitor;
-import org.apache.flink.shaded.asm5.org.objectweb.asm.Opcodes;
-import org.apache.flink.shaded.asm5.org.objectweb.asm.FieldVisitor;
-import org.apache.flink.shaded.asm5.org.objectweb.asm.MethodVisitor;
-import org.apache.flink.shaded.asm5.org.objectweb.asm.Type;
-import org.apache.flink.shaded.asm5.org.objectweb.asm.TypePath;
-import org.apache.flink.shaded.asm5.org.objectweb.asm.Label;
-import org.apache.flink.shaded.asm5.org.objectweb.asm.signature.SignatureReader;
-import org.apache.flink.shaded.asm5.org.objectweb.asm.signature.SignatureVisitor;
+import org.apache.flink.shaded.asm6.org.objectweb.asm.AnnotationVisitor;
+import org.apache.flink.shaded.asm6.org.objectweb.asm.ClassVisitor;
+import org.apache.flink.shaded.asm6.org.objectweb.asm.Opcodes;
+import org.apache.flink.shaded.asm6.org.objectweb.asm.FieldVisitor;
+import org.apache.flink.shaded.asm6.org.objectweb.asm.MethodVisitor;
+import org.apache.flink.shaded.asm6.org.objectweb.asm.Type;
+import org.apache.flink.shaded.asm6.org.objectweb.asm.TypePath;
+import org.apache.flink.shaded.asm6.org.objectweb.asm.Label;
+import org.apache.flink.shaded.asm6.org.objectweb.asm.signature.SignatureReader;
+import org.apache.flink.shaded.asm6.org.objectweb.asm.signature.SignatureVisitor;
import java.util.HashSet;
import java.util.Set;
@@ -66,7 +66,7 @@ public class DependencyVisitor extends ClassVisitor {
@Override
public AnnotationVisitor visitAnnotation(String desc, boolean visible) {
addDesc(desc);
- return new AnnotationVisitorImpl(Opcodes.ASM5);
+ return new AnnotationVisitorImpl(Opcodes.ASM6);
}
@Override
@@ -79,7 +79,7 @@ public class DependencyVisitor extends ClassVisitor {
if (value instanceof Type) {
addType((Type) value);
}
- return new FieldVisitorImpl(Opcodes.ASM5);
+ return new FieldVisitorImpl(Opcodes.ASM6);
}
@Override
@@ -90,7 +90,7 @@ public class DependencyVisitor extends ClassVisitor {
addSignature(signature);
}
addInternalNames(exceptions);
- return new MethodVisitorImpl(Opcodes.ASM5);
+ return new MethodVisitorImpl(Opcodes.ASM6);
}
// ---------------------------------------------------------------------------------------------
@@ -154,13 +154,13 @@ public class DependencyVisitor extends ClassVisitor {
private void addSignature(String signature) {
if (signature != null) {
- new SignatureReader(signature).accept(new SignatureVisitorImpl(Opcodes.ASM5));
+ new SignatureReader(signature).accept(new SignatureVisitorImpl(Opcodes.ASM6));
}
}
private void addTypeSignature(String signature) {
if (signature != null) {
- new SignatureReader(signature).acceptType(new SignatureVisitorImpl(Opcodes.ASM5));
+ new SignatureReader(signature).acceptType(new SignatureVisitorImpl(Opcodes.ASM6));
}
}
@@ -173,7 +173,7 @@ public class DependencyVisitor extends ClassVisitor {
@Override
public AnnotationVisitor visitParameterAnnotation(int parameter, String desc, boolean visible) {
addDesc(desc);
- return new AnnotationVisitorImpl(Opcodes.ASM5);
+ return new AnnotationVisitorImpl(Opcodes.ASM6);
}
@Override
@@ -222,13 +222,13 @@ public class DependencyVisitor extends ClassVisitor {
@Override
public AnnotationVisitor visitAnnotationDefault() {
- return new AnnotationVisitorImpl(Opcodes.ASM5);
+ return new AnnotationVisitorImpl(Opcodes.ASM6);
}
@Override
public AnnotationVisitor visitAnnotation(String desc, boolean visible) {
addDesc(desc);
- return new AnnotationVisitorImpl(Opcodes.ASM5);
+ return new AnnotationVisitorImpl(Opcodes.ASM6);
}
@Override
@@ -277,7 +277,7 @@ public class DependencyVisitor extends ClassVisitor {
@Override
public AnnotationVisitor visitTypeAnnotation(int typeRef, TypePath typePath, String desc, boolean visible) {
addDesc(desc);
- return new AnnotationVisitorImpl(Opcodes.ASM5);
+ return new AnnotationVisitorImpl(Opcodes.ASM6);
}
}
diff --git a/flink-scala/pom.xml b/flink-scala/pom.xml
index 02b359c..b4502df 100644
--- a/flink-scala/pom.xml
+++ b/flink-scala/pom.xml
@@ -47,11 +47,6 @@ under the License.
<dependency>
<groupId>org.apache.flink</groupId>
- <artifactId>flink-shaded-asm</artifactId>
- </dependency>
-
- <dependency>
- <groupId>org.apache.flink</groupId>
<artifactId>flink-shaded-asm-6</artifactId>
</dependency>
diff --git a/pom.xml b/pom.xml
index b3cbcf2..ede22a7 100644
--- a/pom.xml
+++ b/pom.xml
@@ -241,12 +241,6 @@ under the License.
<dependency>
<groupId>org.apache.flink</groupId>
- <artifactId>flink-shaded-asm</artifactId>
- <version>5.0.4-${flink.shaded.version}</version>
- </dependency>
-
- <dependency>
- <groupId>org.apache.flink</groupId>
<artifactId>flink-shaded-asm-6</artifactId>
<version>6.2.1-${flink.shaded.version}</version>
</dependency>