echauchot commented on code in PR #19586:
URL: https://github.com/apache/flink/pull/19586#discussion_r896832924
##########
flink-connectors/flink-connector-cassandra/pom.xml:
##########
@@ -36,9 +36,11 @@ under the License.
<!-- Allow users to pass custom connector versions -->
<properties>
- <cassandra.version>2.2.5</cassandra.version>
- <driver.version>3.0.0</driver.version>
- <guava.version>18.0</guava.version>
+ <!-- Cassandra version 4.x allow to address clusters of version
3.x as well -->
+ <cassandra.version>4.0.3</cassandra.version>
+ <!--driver 3.x works with 3.x and 4.x versions of Cassandra but
driver 4.x is a complete refactoring with different API-->
+ <driver.version>3.11.2</driver.version>
Review Comment:
@zentol a question related to netty: I went through
[FLINK-8295](https://issues.apache.org/jira/browse/FLINK-8295) and in this
ticket we relocate netty to `com.datastax.shaded.netty` so that cassandra
driver could find it. But it would have found it also with default `io.netty`
namespace so why can't we just keep netty transitive dep as it is ? Is it
conflicting with netty elsewhere in flink ? (I checked the version used in
cassandra is the same as in flink parent)
--
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]
