CXXCBC-482: Range scan orchestrator should use best effort retry strategy by default (#542)

Co-authored-by: Sergey Avseyev <[email protected]>

Author: DemetrisChr

core/range_scan_orchestrator_options.hxx

@@ -18,6 +18,7 @@
 #include "range_scan_options.hxx"
 #include "timeout_defaults.hxx"
 
+#include <couchbase/best_effort_retry_strategy.hxx>
 #include <couchbase/mutation_token.hxx>
 #include <couchbase/retry_strategy.hxx>
 
@@ -51,7 +52,7 @@ struct range_scan_orchestrator_options {
     std::uint32_t batch_byte_limit{ range_scan_continue_options::default_batch_byte_limit };
     std::uint16_t concurrency{ default_concurrency };
 
-    std::shared_ptr<couchbase::retry_strategy> retry_strategy{ nullptr };
+    std::shared_ptr<couchbase::retry_strategy> retry_strategy{ make_best_effort_retry_strategy() };
     std::chrono::milliseconds timeout{ timeout_defaults::key_value_scan_timeout };
     std::shared_ptr<couchbase::tracing::request_span> parent_span{};
 };