quantranhong1999 commented on PR #2780:
URL: https://github.com/apache/james-project/pull/2780#issuecomment-3135598791

   >Issue has been resolved and merged and should be available in the next 
release of the driver 4.19.1: 
https://github.com/apache/cassandra-java-driver/pull/2029
   Until then we are stuck I'm afraid in terms of compatibility.
   
   I just read the PR. From what I understand, with our current code and the 
potential 4.19.1 driver upgrade, it would work with Cassandra 5.0 and above. 
But I am afraid that it would not work with the existing deployments with 
Cassandra < 5.0.
   
   > Culprit on our side with compression usage: 
https://github.com/apache/james-project/blob/master/server/blob/blob-cassandra/src/main/java/org/apache/james/blob/cassandra/cache/CassandraBlobCacheDataDefinition.java#L38
   
   The `withCompression` method that we are using seems to change the 
`chunk_length_kb` option (work with Cassandra 4.x) to the `chunk_length_in_kb` 
option (work only with Cassandra 5.0).
   
   And the driver introduced a few deprecated methods to use the deprecated 
`chunk_length_kb` option with prior Cassandra 5.0.
   
   cf 
https://github.com/apache/cassandra-java-driver/pull/2029/files#r2170601291
   
   So it seems there is no single bullet for both prior Cassandra 5.0 and upper 
Cassandra 5.0.
   
   IMO we may
   - Option 1: avoid setting the chunk option to preserve inter-compatibility 
regardless of Cassandra version
   - Option 2: have a JVM property regarding Cassandra version used (not sure 
the chunk option would be worth the complication)
   
   


-- 
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: notifications-unsubscr...@james.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org
For additional commands, e-mail: notifications-h...@james.apache.org

Reply via email to