Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bump netty-all from 4.1.13.Final to 4.1.42.Final #1403

Merged
merged 1 commit into from
Mar 26, 2021
Merged

Conversation

javeme
Copy link
Contributor

@javeme javeme commented Mar 23, 2021

related: hugegraph@bd06f1a
related: hugegraph@accc27f

Change-Id: I1893ec67371712b091f35b085611959dd9d429cc

Change-Id: I1893ec67371712b091f35b085611959dd9d429cc
@javeme
Copy link
Contributor Author

javeme commented Mar 23, 2021

NOTE: hugegraph netty version now is 4.1.13.Final:

// org.apache.tinkerpop:gremlin-driver:jar:3.4.3:
io.netty:netty-all:jar:4.1.36.Final

// com.alipay.sofa:bolt:jar:1.6.2:
io.netty:netty-all:jar:4.1.42.Final

// com.datastax.cassandra:cassandra-driver-core:jar:3.6.0:
io.netty:netty-handler:jar:4.0.56.Final

// cassandra-driver:
io.netty:netty-all:jar:4.1.13.Final
io.netty:netty-transport-native-epoll:jar:4.1.13.Final

@javeme
Copy link
Contributor Author

javeme commented Mar 23, 2021

If we use 4.1.42.Final and keep netty-transport-native-epoll, gremlin-server can't start with error:

2021-03-23 12:28:07 2969  [main] [ERROR] com.baidu.hugegraph.dist.HugeGraphServer [] - HugeGremlinServer start error:
java.lang.ExceptionInInitializerError: null
        at java.lang.Class.forName0(Native Method) ~[?:1.8.0_111]
        at java.lang.Class.forName(Class.java:264) ~[?:1.8.0_111]
        at org.apache.tinkerpop.gremlin.groovy.engine.GremlinExecutor.initializeGremlinScriptEngineManager(GremlinExecutor.java:413) ~[gremlin-groovy-3.4.3.jar:3.4.3]
        at org.apache.tinkerpop.gremlin.groovy.engine.GremlinExecutor.<init>(GremlinExecutor.java:108) ~[gremlin-groovy-3.4.3.jar:3.4.3]
        at org.apache.tinkerpop.gremlin.groovy.engine.GremlinExecutor.<init>(GremlinExecutor.java:77) ~[gremlin-groovy-3.4.3.jar:3.4.3]
        at org.apache.tinkerpop.gremlin.groovy.engine.GremlinExecutor$Builder.create(GremlinExecutor.java:592) ~[gremlin-groovy-3.4.3.jar:3.4.3]
        at org.apache.tinkerpop.gremlin.server.util.ServerGremlinExecutor.<init>(ServerGremlinExecutor.java:128) ~[gremlin-server-3.4.3.jar:3.4.3]
        at org.apache.tinkerpop.gremlin.server.GremlinServer.<init>(GremlinServer.java:122) ~[gremlin-server-3.4.3.jar:3.4.3]
        at com.baidu.hugegraph.auth.ContextGremlinServer.<init>(ContextGremlinServer.java:46) ~[hugegraph-api-0.11.2.jar:0.59.0.0]
        at com.baidu.hugegraph.dist.HugeGremlinServer.startWithInjectTraversal(HugeGremlinServer.java:78) ~[hugegraph-dist-0.11.2.jar:?]
        at com.baidu.hugegraph.dist.HugeGremlinServer.start(HugeGremlinServer.java:61) ~[hugegraph-dist-0.11.2.jar:?]
        at com.baidu.hugegraph.dist.HugeGraphServer.<init>(HugeGraphServer.java:44) [hugegraph-dist-0.11.2.jar:?]
        at com.baidu.hugegraph.dist.HugeGraphServer.main(HugeGraphServer.java:100) [hugegraph-dist-0.11.2.jar:?]
Caused by: com.baidu.hugegraph.HugeException: Failed to scan classes under package com.baidu.hugegraph.type.define
        at com.baidu.hugegraph.plugin.HugeGraphGremlinPlugin.<clinit>(HugeGraphGremlinPlugin.java:51) ~[hugegraph-core-0.11.2.jar:0.11.2.0]
        ... 13 more
Caused by: java.util.zip.ZipException: invalid LOC header (bad signature)
        at java.util.zip.ZipFile.read(Native Method) ~[?:1.8.0_111]
        at java.util.zip.ZipFile.access$1400(ZipFile.java:60) ~[?:1.8.0_111]
        at java.util.zip.ZipFile$ZipFileInputStream.read(ZipFile.java:717) ~[?:1.8.0_111]
        at java.util.zip.ZipFile$ZipFileInflaterInputStream.fill(ZipFile.java:419) ~[?:1.8.0_111]
        at java.util.zip.InflaterInputStream.read(InflaterInputStream.java:158) ~[?:1.8.0_111]
        at sun.misc.IOUtils.readFully(IOUtils.java:65) ~[?:1.8.0_111]
        at java.util.jar.JarFile.getBytes(JarFile.java:425) ~[?:1.8.0_111]
        at java.util.jar.JarFile.getManifestFromReference(JarFile.java:193) ~[?:1.8.0_111]
        at java.util.jar.JarFile.getManifest(JarFile.java:180) ~[?:1.8.0_111]
        at com.google.common.reflect.ClassPath$Scanner.scanJar(ClassPath.java:397) ~[guava-25.1-jre.jar:?]
        at com.google.common.reflect.ClassPath$Scanner.scanFrom(ClassPath.java:384) ~[guava-25.1-jre.jar:?]
        at com.google.common.reflect.ClassPath$Scanner.scan(ClassPath.java:361) ~[guava-25.1-jre.jar:?]
        at com.google.common.reflect.ClassPath$Scanner.scan(ClassPath.java:354) ~[guava-25.1-jre.jar:?]
        at com.google.common.reflect.ClassPath.from(ClassPath.java:119) ~[guava-25.1-jre.jar:?]
        at com.baidu.hugegraph.util.ReflectionUtil.classes(ReflectionUtil.java:90) ~[hugegraph-common-1.8.1.jar:1.8.1.0]
        at com.baidu.hugegraph.plugin.HugeGraphGremlinPlugin.<clinit>(HugeGraphGremlinPlugin.java:49) ~[hugegraph-core-0.11.2.jar:0.11.2.0]
        ... 13 more

netty jars in lib:

$ ls -lh hugegraph-0.11.2/lib/ |grep netty
-rw-r--r--  1 user  INTERNAL\Domain Users   6.8M  3 23 11:27 grpc-netty-shaded-1.28.0.jar
-rw-r--r--  1 user  INTERNAL\Domain Users   3.8M  3 23 11:27 hbase-shaded-netty-2.1.0.jar
-rw-r--r--  1 user  INTERNAL\Domain Users   3.9M  3 23 11:27 netty-all-4.1.42.Final.jar
-rw-r--r--  1 user  INTERNAL\Domain Users   318K  3 23 11:27 netty-buffer-4.1.42.Final.jar
-rw-r--r--  1 user  INTERNAL\Domain Users   610K  3 23 11:27 netty-common-4.1.42.Final.jar
-rw-r--r--  1 user  INTERNAL\Domain Users    32K  3 23 11:27 netty-resolver-4.1.42.Final.jar
-rw-r--r--  1 user  INTERNAL\Domain Users   478K  3 23 11:27 netty-transport-4.1.42.Final.jar
-rw-r--r--  1 user  INTERNAL\Domain Users   189K  3 23 11:27 netty-transport-native-epoll-4.1.42.Final-linux-x86_64.jar
-rw-r--r--  1 user  INTERNAL\Domain Users    32K  3 23 11:27 netty-transport-native-unix-common-4.1.42.Final.jar

cassandra pom.xml:

        <dependency>
            <groupId>com.datastax.cassandra</groupId>
            <artifactId>cassandra-driver-core</artifactId>
            <version>3.6.0</version>
            <exclusions>
                <exclusion>
                    <groupId>io.netty</groupId>
                    <artifactId>netty-handler</artifactId>
                </exclusion>
            </exclusions>
        </dependency>
        <dependency>
            <!-- netty-all contain netty-transport-native-epoll, https://github.com/netty/netty/issues/8714 -->
            <groupId>io.netty</groupId>
            <artifactId>netty-all</artifactId>
            <version>4.1.42.Final</version>
        </dependency>
        <dependency>
            <groupId>io.netty</groupId>
            <artifactId>netty-transport-native-epoll</artifactId>
            <version>4.1.42.Final</version>
            <classifier>linux-x86_64</classifier>
        </dependency>

@codecov
Copy link

codecov bot commented Mar 23, 2021

Codecov Report

Merging #1403 (d8d1348) into master (4c4b770) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1403      +/-   ##
============================================
- Coverage     62.05%   62.04%   -0.02%     
- Complexity     5837     5838       +1     
============================================
  Files           386      386              
  Lines         32185    32192       +7     
  Branches       4494     4497       +3     
============================================
+ Hits          19974    19975       +1     
- Misses        10178    10185       +7     
+ Partials       2033     2032       -1     
Impacted Files Coverage Δ Complexity Δ
...ain/java/com/baidu/hugegraph/api/auth/UserAPI.java 0.00% <0.00%> (ø) 0.00% <0.00%> (ø%)
...om/baidu/hugegraph/auth/StandardAuthenticator.java 0.00% <0.00%> (ø) 0.00% <0.00%> (ø%)
...idu/hugegraph/api/filter/AuthenticationFilter.java 32.09% <0.00%> (ø) 0.00% <0.00%> (ø%)
...om/baidu/hugegraph/task/StandardTaskScheduler.java 76.35% <0.00%> (+0.24%) 86.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4c4b770...d8d1348. Read the comment docs.

@javeme
Copy link
Contributor Author

javeme commented Mar 23, 2021

If we use 4.1.42.Final and not exclude netty-handler:

2021-03-23 14:09:28 3431  [main] [ERROR] com.baidu.hugegraph.dist.HugeGraphServer [] - HugeGremlinServer start error: 
java.lang.ExceptionInInitializerError: null
        at java.lang.Class.forName0(Native Method) ~[?:1.8.0_111]
        at java.lang.Class.forName(Class.java:264) ~[?:1.8.0_111]
        at org.apache.tinkerpop.gremlin.groovy.engine.GremlinExecutor.initializeGremlinScriptEngineManager(GremlinExecutor.java:413) ~[gremlin-groovy-3.4.3.jar:3.4.3]
        at org.apache.tinkerpop.gremlin.groovy.engine.GremlinExecutor.<init>(GremlinExecutor.java:108) ~[gremlin-groovy-3.4.3.jar:3.4.3]
        at org.apache.tinkerpop.gremlin.groovy.engine.GremlinExecutor.<init>(GremlinExecutor.java:77) ~[gremlin-groovy-3.4.3.jar:3.4.3]
        at org.apache.tinkerpop.gremlin.groovy.engine.GremlinExecutor$Builder.create(GremlinExecutor.java:592) ~[gremlin-groovy-3.4.3.jar:3.4.3]
        at org.apache.tinkerpop.gremlin.server.util.ServerGremlinExecutor.<init>(ServerGremlinExecutor.java:128) ~[gremlin-server-3.4.3.jar:3.4.3]
        at org.apache.tinkerpop.gremlin.server.GremlinServer.<init>(GremlinServer.java:122) ~[gremlin-server-3.4.3.jar:3.4.3]
        at com.baidu.hugegraph.auth.ContextGremlinServer.<init>(ContextGremlinServer.java:46) ~[hugegraph-api-0.11.2.jar:0.59.0.0]
        at com.baidu.hugegraph.dist.HugeGremlinServer.startWithInjectTraversal(HugeGremlinServer.java:78) ~[hugegraph-dist-0.11.2.jar:?]
        at com.baidu.hugegraph.dist.HugeGremlinServer.start(HugeGremlinServer.java:61) ~[hugegraph-dist-0.11.2.jar:?]
        at com.baidu.hugegraph.dist.HugeGraphServer.<init>(HugeGraphServer.java:44) [hugegraph-dist-0.11.2.jar:?]
        at com.baidu.hugegraph.dist.HugeGraphServer.main(HugeGraphServer.java:100) [hugegraph-dist-0.11.2.jar:?]
Caused by: com.baidu.hugegraph.HugeException: Failed to scan classes under package com.baidu.hugegraph.type.define
        at com.baidu.hugegraph.plugin.HugeGraphGremlinPlugin.<clinit>(HugeGraphGremlinPlugin.java:51) ~[hugegraph-core-0.11.2.jar:0.11.2.0]
        ... 13 more
Caused by: java.util.zip.ZipException: invalid LOC header (bad signature)
        at java.util.zip.ZipFile.read(Native Method) ~[?:1.8.0_111]
        at java.util.zip.ZipFile.access$1400(ZipFile.java:60) ~[?:1.8.0_111]
        at java.util.zip.ZipFile$ZipFileInputStream.read(ZipFile.java:717) ~[?:1.8.0_111]
        at java.util.zip.ZipFile$ZipFileInflaterInputStream.fill(ZipFile.java:419) ~[?:1.8.0_111]
        at java.util.zip.InflaterInputStream.read(InflaterInputStream.java:158) ~[?:1.8.0_111]
        at sun.misc.IOUtils.readFully(IOUtils.java:65) ~[?:1.8.0_111]
        at java.util.jar.JarFile.getBytes(JarFile.java:425) ~[?:1.8.0_111]
        at java.util.jar.JarFile.getManifestFromReference(JarFile.java:193) ~[?:1.8.0_111]
        at java.util.jar.JarFile.getManifest(JarFile.java:180) ~[?:1.8.0_111]
        at com.google.common.reflect.ClassPath$Scanner.scanJar(ClassPath.java:397) ~[guava-25.1-jre.jar:?]
        at com.google.common.reflect.ClassPath$Scanner.scanFrom(ClassPath.java:384) ~[guava-25.1-jre.jar:?]
        at com.google.common.reflect.ClassPath$Scanner.scan(ClassPath.java:361) ~[guava-25.1-jre.jar:?]
        at com.google.common.reflect.ClassPath$Scanner.scan(ClassPath.java:354) ~[guava-25.1-jre.jar:?]
        at com.google.common.reflect.ClassPath.from(ClassPath.java:119) ~[guava-25.1-jre.jar:?]
        at com.baidu.hugegraph.util.ReflectionUtil.classes(ReflectionUtil.java:90) ~[hugegraph-common-1.8.1.jar:1.8.1.0]
        at com.baidu.hugegraph.plugin.HugeGraphGremlinPlugin.<clinit>(HugeGraphGremlinPlugin.java:49) ~[hugegraph-core-0.11.2.jar:0.11.2.0]
        ... 13 more
Exception in thread "main" java.lang.ExceptionInInitializerError
        at java.lang.Class.forName0(Native Method)
        at java.lang.Class.forName(Class.java:264)
        at org.apache.tinkerpop.gremlin.groovy.engine.GremlinExecutor.initializeGremlinScriptEngineManager(GremlinExecutor.java:413)
        at org.apache.tinkerpop.gremlin.groovy.engine.GremlinExecutor.<init>(GremlinExecutor.java:108)
        at org.apache.tinkerpop.gremlin.groovy.engine.GremlinExecutor.<init>(GremlinExecutor.java:77)
        at org.apache.tinkerpop.gremlin.groovy.engine.GremlinExecutor$Builder.create(GremlinExecutor.java:592)
        at org.apache.tinkerpop.gremlin.server.util.ServerGremlinExecutor.<init>(ServerGremlinExecutor.java:128)
        at org.apache.tinkerpop.gremlin.server.GremlinServer.<init>(GremlinServer.java:122)
        at com.baidu.hugegraph.auth.ContextGremlinServer.<init>(ContextGremlinServer.java:46)
        at com.baidu.hugegraph.dist.HugeGremlinServer.startWithInjectTraversal(HugeGremlinServer.java:78)
        at com.baidu.hugegraph.dist.HugeGremlinServer.start(HugeGremlinServer.java:61)
        at com.baidu.hugegraph.dist.HugeGraphServer.<init>(HugeGraphServer.java:44)
        at com.baidu.hugegraph.dist.HugeGraphServer.main(HugeGraphServer.java:100)
Caused by: com.baidu.hugegraph.HugeException: Failed to scan classes under package com.baidu.hugegraph.type.define
        at com.baidu.hugegraph.plugin.HugeGraphGremlinPlugin.<clinit>(HugeGraphGremlinPlugin.java:51)
        ... 13 more
Caused by: java.util.zip.ZipException: invalid LOC header (bad signature)
        at java.util.zip.ZipFile.read(Native Method)
        at java.util.zip.ZipFile.access$1400(ZipFile.java:60)
        at java.util.zip.ZipFile$ZipFileInputStream.read(ZipFile.java:717)
        at java.util.zip.ZipFile$ZipFileInflaterInputStream.fill(ZipFile.java:419)
        at java.util.zip.InflaterInputStream.read(InflaterInputStream.java:158)
        at sun.misc.IOUtils.readFully(IOUtils.java:65)
        at java.util.jar.JarFile.getBytes(JarFile.java:425)
        at java.util.jar.JarFile.getManifestFromReference(JarFile.java:193)
        at java.util.jar.JarFile.getManifest(JarFile.java:180)
        at com.google.common.reflect.ClassPath$Scanner.scanJar(ClassPath.java:397)
        at com.google.common.reflect.ClassPath$Scanner.scanFrom(ClassPath.java:384)
        at com.google.common.reflect.ClassPath$Scanner.scan(ClassPath.java:361)
        at com.google.common.reflect.ClassPath$Scanner.scan(ClassPath.java:354)
        at com.google.common.reflect.ClassPath.from(ClassPath.java:119)
        at com.baidu.hugegraph.util.ReflectionUtil.classes(ReflectionUtil.java:90)
        at com.baidu.hugegraph.plugin.HugeGraphGremlinPlugin.<clinit>(HugeGraphGremlinPlugin.java:49)
        ... 13 more

netty jars in lib:

$ ls -lh hugegraph-0.11.2/lib/ |grep netty
-rw-r--r--  1 user  INTERNAL\Domain Users   6.8M  3 23 14:02 grpc-netty-shaded-1.28.0.jar
-rw-r--r--  1 user  INTERNAL\Domain Users   3.8M  3 23 14:02 hbase-shaded-netty-2.1.0.jar
-rw-r--r--  1 user  INTERNAL\Domain Users   3.9M  3 23 14:02 netty-all-4.1.42.Final.jar
-rw-r--r--  1 user  INTERNAL\Domain Users   229K  3 23 14:02 netty-buffer-4.0.56.Final.jar
-rw-r--r--  1 user  INTERNAL\Domain Users   150K  3 23 14:02 netty-codec-4.0.56.Final.jar
-rw-r--r--  1 user  INTERNAL\Domain Users   423K  3 23 14:02 netty-common-4.0.56.Final.jar
-rw-r--r--  1 user  INTERNAL\Domain Users   351K  3 23 14:02 netty-handler-4.0.56.Final.jar
-rw-r--r--  1 user  INTERNAL\Domain Users   413K  3 23 14:02 netty-transport-4.0.56.Final.jar

cassandra pom.xml:

        <dependency>
            <groupId>com.datastax.cassandra</groupId>
            <artifactId>cassandra-driver-core</artifactId>
            <version>3.6.0</version>
        </dependency>
        <dependency>
            <!-- netty-all contain netty-transport-native-epoll, https://github.com/netty/netty/issues/8714 -->
            <groupId>io.netty</groupId>
            <artifactId>netty-all</artifactId>
            <version>4.1.42.Final</version>
        </dependency>

@zhoney zhoney merged commit f57ad0d into master Mar 26, 2021
@zhoney zhoney deleted the netty-version-4.1.42 branch March 26, 2021 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants