exec_profile: Use cluster lbp for exec profiles by default

muzarski commented 1 month ago

The documentation of cass_execution_profile_set_load_balance_... functions states:

 <b>Note:</b> Profile-based load balancing policy is disabled by default.
 cluster load balancing policy is used when profile does not contain a policy.

It seems to be true, see: https://github.com/scylladb/cpp-driver/blob/master/src/request_processor.cpp#L196-L209.

We should adjust cpp-rust-driver so it behaves the same way.

This PR consist mostly of some refactor commits that prepare for the logic change in the last commit.

Pre-review checklist

[x] I have split my patch into logically separate commits.
[x] All commit messages clearly explain what they change and why.
[x] PR description sums up the changes and reasons why they should be introduced.
~[ ] I have implemented Rust unit tests for the features/changes introduced.~
~[ ] I have enabled appropriate tests in .github/workflows/build.yml in gtest_filter.~
~[ ] I have enabled appropriate tests in .github/workflows/cassandra.yml in gtest_filter.~

muzarski commented 1 month ago

v2: Reverted the changes introduced by lbp_config: move LoadBalancingKind to upper layer (removed this commit). Now, the load_balancing_kind field is optional in LoadBalancingConfig.

muzarski commented 3 weeks ago

v3: Rebased on https://github.com/scylladb/cpp-rust-driver/pull/200. Addressed all of @Lorak-mmk comments

muzarski commented 3 weeks ago

Rebased on master

scylladb / cpp-rust-driver

exec_profile: Use cluster lbp for exec profiles by default #197

Pre-review checklist