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

[ERROR] Could not retrieve sharding info from connection #73

Closed
evll opened this issue May 30, 2023 · 6 comments
Closed

[ERROR] Could not retrieve sharding info from connection #73

evll opened this issue May 30, 2023 · 6 comments
Assignees
Labels
cassandra4.x Compatibility with Cassandra 4.x enhancement New feature or request
Milestone

Comments

@evll
Copy link

evll commented May 30, 2023

When using this extension for Cassandra >= 4, the following error occurs on connection:
cassandra | [ERROR] Could not retrieve sharding info from connection to xx.xxx.75. Continuing WITHOUT SHARD-AWARENESS. (connector.cpp:315)

The error does not prevent the connection from working, but it's polluting logs and is annoying in general.

Cassandra does not support shard-awareness, so is there a way to disable this functionality for it?

@CodeLieutenant
Copy link
Member

This is scylla aware driver first. I can add cassandra-cpp as build param if it's important to you

@CodeLieutenant
Copy link
Member

Driver is not tested on Cassandra 4.x, only on 3.x as this is what scylladb is based on

@CodeLieutenant CodeLieutenant self-assigned this May 31, 2023
@CodeLieutenant CodeLieutenant added enhancement New feature or request cassandra4.x Compatibility with Cassandra 4.x labels May 31, 2023
@CodeLieutenant CodeLieutenant added this to the v1.4.0 milestone May 31, 2023
@evll
Copy link
Author

evll commented May 31, 2023

This is scylla aware driver first. I can add cassandra-cpp as build param if it's important to you

I would really appreciate that, thanks!

@CodeLieutenant
Copy link
Member

I'll create seperate branch for that so that you can test it if it works

@CodeLieutenant CodeLieutenant pinned this issue Jun 6, 2023
@CodeLieutenant CodeLieutenant moved this from To do to In progress in ScyllaDB Driver Jul 16, 2023
@CodeLieutenant
Copy link
Member

CodeLieutenant commented Jul 29, 2023

Sorry for the wait, I was on vaccation for a long time (a little bit burnt out).
I have implemented dynamic loading of libcassandra instead of libscylladb on #81 this pull request.
If you still need this feature, testing it would be helpful.

Now this library will officially support libcassandra behind a CMake option -DPHP_SCYLLADB_USE_LIBCASSANDRA=ON.
By default now everything is dynamically linked instead of static and compiling, so the building process is a little bit changed for dynamic linking as it requires LibUV and LibScyllaDB/LibCassandra to be present on the running system. There is also a way to compile from source with CMake option if you prefer it that way:

For LibUV compilation: -DPHP_SCYLLADB_LIBUV_FROM_SRC=ON -DPHP_SCYLLADB_LIBUV_STATIC=ON
For LibScyllaDB: -DPHP_SCYLLADB_LIBSCYLLADB_FROM_SRC=ON -DPHP_SCYLLADB_LIBSCYLLADB_STATIC=ON
For CassandraDB -DPHP_SCYLLADB_LIBCASSANDRA_FROM_SRC=ON -PHP_SCYLLADB_LIBCASSANDRA_STATIC=ON

If you need any help please reopen this Issue

@CodeLieutenant CodeLieutenant moved this from In progress to Done in ScyllaDB Driver Jul 29, 2023
@evll
Copy link
Author

evll commented Aug 1, 2023

FYI: we tested the ReleaseLibCassandra preset, and it worked well, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cassandra4.x Compatibility with Cassandra 4.x enhancement New feature or request
Projects
Status: Done
Development

No branches or pull requests

2 participants