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

Be more verbose when failing to fetch metadata #1021

Closed
fee-mendes opened this issue Jun 24, 2024 · 3 comments
Closed

Be more verbose when failing to fetch metadata #1021

fee-mendes opened this issue Jun 24, 2024 · 3 comments
Assignees
Milestone

Comments

@fee-mendes
Copy link
Member

fee-mendes commented Jun 24, 2024

As seen in scylladb/scylladb#19107, sometimes the driver may fail to fetch metadata from its peers and print messages in the logs such as:

2024-06-24T19:05:16.729306Z  WARN scylla::transport::topology: Failed to establish control connection and fetch metadata on all known peers. Falling back to initial contact points.
2024-06-24T19:05:16.729347Z  WARN scylla::transport::topology: Failed to fetch metadata using current control connection control_connection_address="172.31.29.124:9042" error=Protocol Error: system.peers or system.local has invalid column type
2024-06-24T19:05:16.733118Z ERROR scylla::transport::topology: Could not fetch metadata error=Protocol Error: system.peers or system.local has invalid column type

It is wise to know exactly which invalid data was found, such as from which host_id, which column was found to be invalid and which value was returned.

Similar work has been done for the Python driver in scylladb/python-driver#303

@fee-mendes fee-mendes changed the title Be more verbose on when failing to fetch metadata Be more verbose when failing to fetch metadata Jun 24, 2024
@wprzytula wprzytula added this to the 0.14.0 milestone Jun 25, 2024
@mykaul
Copy link
Contributor

mykaul commented Jul 29, 2024

The root cause might be due to #1023 btw.

@dkropachev
Copy link
Collaborator

@fee-mendes, original problem is solved in #1023, and verbose logging is going to be implemented at #462

@wprzytula
Copy link
Collaborator

This has been included in release 0.13.2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants