Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(cassandra): adapt hugegraph-cassandra from 3.11.12 to 4.0.10 #2260

Closed
wants to merge 11 commits into from
Closed

feat(cassandra): adapt hugegraph-cassandra from 3.11.12 to 4.0.10 #2260

wants to merge 11 commits into from

Conversation

Zony7
Copy link
Contributor

@Zony7 Zony7 commented Jul 28, 2023

we should check the compatibility in:

  • Cassandra Server V3.11
  • Cassandra Server V4.0

Expect to be able to connect the two versions properly?

@imbajin imbajin added dependencies Incompatible dependencies of package cassandra Cassandra backend labels Jul 28, 2023
Copy link
Member

@imbajin imbajin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

refer here to fix the license check problem (or exclude some duplicate dependencies)

@codecov
Copy link

codecov bot commented Jul 28, 2023

Codecov Report

Merging #2260 (ac3ebf5) into master (77c7612) will decrease coverage by 0.65%.
The diff coverage is 100.00%.

❗ Current head ac3ebf5 differs from pull request most recent head 2856049. Consider uploading reports for the commit 2856049 to get more accurate results

@@             Coverage Diff              @@
##             master    #2260      +/-   ##
============================================
- Coverage     65.08%   64.43%   -0.65%     
+ Complexity      979      680     -299     
============================================
  Files           498      498              
  Lines         41240    41240              
  Branches       5738     5738              
============================================
- Hits          26841    26575     -266     
- Misses        11744    12011     +267     
+ Partials       2655     2654       -1     
Files Changed Coverage Δ
...egraph/backend/store/cassandra/CassandraShard.java 54.12% <ø> (ø)
...raph/backend/store/cassandra/CassandraMetrics.java 85.03% <100.00%> (ø)

... and 12 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Zony7
Copy link
Contributor Author

Zony7 commented Jul 28, 2023

Anyone who interested in Cassandra 4 can take a look at this discussion #2261

@Zony7 Zony7 deleted the branch apache:master August 27, 2023 15:00
@Zony7 Zony7 closed this Aug 27, 2023
@Zony7 Zony7 deleted the master branch August 27, 2023 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cassandra Cassandra backend dependencies Incompatible dependencies of package
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants