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

cpp-driver should default to Advanced Shard Awareness #65

Open
fee-mendes opened this issue Jul 6, 2022 · 0 comments
Open

cpp-driver should default to Advanced Shard Awareness #65

fee-mendes opened this issue Jul 6, 2022 · 0 comments

Comments

@fee-mendes
Copy link
Member

TLDR; it makes no sense to have a recommended feature that is not the default

It's been long since Scylla <4.3 went out of support, back from a time when it made sense to have basic shard-awareness by default. The world has evolved since then, and Advanced Shard Awareness was introduced, which is able to fallback to basic shard-awareness mode whenever needed.

One potential argument against not enabling it by default would be the need to set cass_cluster_set_local_port_range(start, end) ephemeral port range. I advocate that this parameter should either default to the value of net.ipv4.ip_local_port_range or - if we want something portable - 49152-65535 seems to be the standard "informal agreement" across most platforms.

@jul-stas WDYT?

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

No branches or pull requests

1 participant