sekikn commented on PR #19281:
URL: https://github.com/apache/pulsar/pull/19281#issuecomment-1397840229
Thank you for the review @Jason918 @tisonkun! As @tisonkun pointed out, I
found this PR (upgrading client to 2.8.1) doesn't work with the old version of
Alluxio server (2.7.3), probably due to the API change.
```
Starting function instance...
Exception in thread "alluxio-client-netty-event-loop-RPC-11" Exception in
thread "alluxio-client-netty-event-loop-RPC-25" Exception in thread
"alluxio-client-netty-event-loop-RPC-31" Exception in thread
"alluxio-client-netty-event-loop-RPC-1" Exception in thread
"alluxio-client-netty-event-loop-RPC-7" Exception in thread
"alluxio-client-netty-event-loop-RPC-2" java.lang.NoClassDefFoundError:
io/netty/util/concurrent/DefaultPromise$1
...
Exception in thread "Thread-4" java.lang.NoClassDefFoundError:
alluxio/util/WaitForOptions
at
alluxio.grpc.GrpcChannelPool.waitForConnectionReady(GrpcChannelPool.java:232)
at
alluxio.grpc.GrpcChannelPool.lambda$acquireChannel$1(GrpcChannelPool.java:99)
at
java.base/java.util.concurrent.ConcurrentHashMap.compute(ConcurrentHashMap.java:1940)
at alluxio.grpc.GrpcChannelPool.acquireChannel(GrpcChannelPool.java:94)
at alluxio.grpc.GrpcChannelBuilder.build(GrpcChannelBuilder.java:92)
at alluxio.conf.Configuration.loadConfiguration(Configuration.java:439)
at alluxio.ClientContext.loadConf(ClientContext.java:120)
at
alluxio.client.metrics.ClientMasterSync.loadConf(ClientMasterSync.java:122)
at
alluxio.client.metrics.ClientMasterSync.heartbeat(ClientMasterSync.java:74)
at
alluxio.client.metrics.MetricsHeartbeatContext.heartbeat(MetricsHeartbeatContext.java:95)
at
alluxio.client.metrics.MetricsHeartbeatContext.access$300(MetricsHeartbeatContext.java:52)
at
alluxio.client.metrics.MetricsHeartbeatContext$MetricsMasterSyncShutDownHook.lambda$null$0(MetricsHeartbeatContext.java:228)
at
java.base/java.util.concurrent.ConcurrentHashMap.forEach(ConcurrentHashMap.java:1603)
at
alluxio.client.metrics.MetricsHeartbeatContext$MetricsMasterSyncShutDownHook.lambda$new$1(MetricsHeartbeatContext.java:228)
at java.base/java.lang.Thread.run(Thread.java:833)
Caused by: java.lang.ClassNotFoundException: alluxio.util.WaitForOptions
at java.base/java.net.URLClassLoader.findClass(URLClassLoader.java:445)
at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:587)
at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:520)
... 15 more
```
On the other hand, the current client version still works with the latest
version of Alluxio server, though it warns version mismatch:
```
2023-01-20T11:01:04,141+0900 [metrics-master-heartbeat-0] WARN
alluxio.util.ConfigurationUtils - Alluxio CLIENT version (2.7.3) does not match
Alluxio cluster version (2.9.0)
```
So it seems better not to merge this PR and stay in the current client
version at this time. Maybe I should add some integration tests against
different versions of Alluxio first so that we can ensure backward
compatibility with them. I'll try it later, and I'll withdraw this PR for now.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
