prestodb / presto

The official home of the Presto distributed SQL query engine for big data
http://prestodb.io
Apache License 2.0
15.99k stars 5.36k forks source link

UT failures in presto-kafka module when upgrading org.apache.zookeeper from 3.4.14 to 3.7.2 #21950

Open SusanPhilip opened 8 months ago

SusanPhilip commented 8 months ago

Unable to upgrade zookeeper version. Kafka has a dependency with zookeeper so upgrading zookeeper leads to UT failures in kakfa. Upgrading kafka to a compatible version version of zookeeper leads to break in code compilation as many classes are not found.

PR associated with the same: https://github.com/prestodb/presto/pull/21119

Your Environment

Errors faced

com.facebook.presto.kafka.TestMinimalFunctionality.startKafka Time elapsed: 5.491 s <<< FAILURE! java.lang.NoClassDefFoundError: com.codahale.metrics.Reservoir com.facebook.presto.kafka.TestMinimalFunctionality.tearDown Time elapsed: 0.002 s <<< FAILURE! java.lang.NullPointerException

While upgrading kafka the below errors were shown: presto/presto-kafka/src/test/java/com/facebook/presto/kafka/util/EmbeddedKafka.java: cannot find symbol class ZkUtils symbol: class location: package kafka.utils presto-kafka/src/test/java/com/facebook/presto/kafka/util/EmbeddedKafka.java:[127,9] cannot find symbol class symbol: class ZkUtils location: class com.facebook.presto.kafka.util.EmbeddedKafka

Akanksha-kedia commented 8 months ago

https://github.com/prestodb/presto/pull/21957