Netflix / astyanax

Cassandra Java Client
Apache License 2.0
1.04k stars 357 forks source link

Dependency update #640

Closed chapj2001 closed 6 years ago

chapj2001 commented 6 years ago

Update dependencies and shade cassandra-all usage to ease migration from Astyanax to DataStax Java Driver for Apache Cassandra. This PR does not signal any change in Astyanax deprecation.

Astyanax is deprecated but still used by many projects. By now Astyanax's very old dependencies often increase the effort required to migrate to the DataStax driver. The transition can be especially difficult for projects that need both Astyanax and DataStax drivers to coexist for some time, e.g. an app that wants to migrate its own Astyanax dependency to DataStax driver but still has other transitive dependencies to Astyanax outside its direct control.

Summary of changes and impact to Astyanax users:

chapj2001 commented 6 years ago

re: the suggestion for more unit tests for all the datatype changes, the vast majority of those changes are very simple (reference shaded package of a class from cassandra-all) and they're pretty deep in astyanax-core so that they already get exercised pretty well by existing unit/integration tests from astyanax-test and astyanax-example. I'll see if I can find some code coverage metric to verify that we covered each datatype at least once.