search

google / zetasql

ZetaSQL - Analyzer Framework for SQL
Apache License 2.0
2.32k stars 219 forks source link

Java 17 support #89

Closed kileys closed 2 years ago

kileys commented 2 years ago

Java 17 removed the sun/nio/ch/SocketChannelImpl constructor used here[1] and now throws java.lang.NoSuchMethodError: <init>. Can support be added to use an available constructor?

[1] https://github.com/google/zetasql/blob/master/zetasql/local_service/local_service_jni.cc#L117-L120

matthewcbrown commented 2 years ago

This should be fixed in 2022.04.1, please give it a try (thanks @apilloud!)

apilloud commented 2 years ago

Confirmed this works in https://github.com/apache/beam/pull/17434