Use std::optional if available.

Currently, SDK requires C++11 minimum.
So, boost::optional type is used for optional values.
For C++17 and above more convenient is to use std::optional instead.
This PR makes the optional type used configurable.

Relates-To: NLAM-23
Signed-off-by: sopov <ext-alexander.sopov@here.com>

Author: asopov-here

CONTRIBUTING.md

@@ -257,12 +257,13 @@ inline PublishDataRequest& WithLayerId(std::string&& layer_id) {
 
 #### API optional parameters
 
-The SDK uses `boost::optional` for optional members, parameters, or return types. Also, function comments should indicate whether the parameter is optional or required.
+The SDK uses `porting::optional` mapped to either `boost::optional` or `std::optional` for optional members,
+parameters, or return types. Also, function comments should indicate whether the parameter is optional or required.
 
 Example:
 
 ```cpp
-inline const boost::optional<std::string>& GetBillingTag() const {
+inline const porting::optional<std::string>& GetBillingTag() const {
   return billing_tag_;
 }
 

docs/dataservice-cache-example.md

@@ -89,7 +89,7 @@ You can get data from a [versioned layer](https://www.here.com/docs/bundle/data-
    olp::dataservice::read::VersionedLayerClient layer_client (
                        client::HRN catalog,
                        std::string layer_id,
-                       boost::optional<int64_t> catalog_version,
+                       porting::optional<int64_t> catalog_version,
                        client::OlpClientSettings settings);
    ```
 
@@ -102,7 +102,7 @@ You can get data from a [versioned layer](https://www.here.com/docs/bundle/data-
    ```cpp
    auto request = olp::dataservice::read::DataRequest()
                     .WithPartitionId(first_partition_id)
-                    .WithBillingTag(boost::none)
+                    .WithBillingTag(olp::porting::none)
                     .WithFetchOption(olp::dataservice::read::FetchOptions::OnlineIfNotFound);
    ```
 

docs/dataservice-read-catalog-example.md

@@ -359,7 +359,7 @@ You can request any data version from a [versioned layer](https://www.here.com/d
    olp::dataservice::read::VersionedLayerClient layer_client (
                        client::HRN catalog,
                        std::string layer_id,
-                       boost::optional<int64_t> catalog_version,
+                       porting::optional<int64_t> catalog_version,
                        client::OlpClientSettings settings);
    ```
 

examples/ProtectedCacheExample.cpp

@@ -62,13 +62,16 @@ bool HandleDataResponse(
 }  // namespace
 
 int RunExampleReadWithCache(const AccessKey& access_key,
-                            const olp::cache::CacheSettings& cache_settings, const std::string& catalog) {
+                            const olp::cache::CacheSettings& cache_settings,
+                            const std::string& catalog) {
   OLP_SDK_LOG_INFO_F(
       kLogTag, "Mutable cache path is \"%s\"",
-      cache_settings.disk_path_mutable.get_value_or("none").c_str());
+      olp::porting::get_value_or(cache_settings.disk_path_mutable, "none")
+          .c_str());
   OLP_SDK_LOG_INFO_F(
       kLogTag, "Protected cache path is \"%s\"",
-      cache_settings.disk_path_protected.get_value_or("none").c_str());
+      olp::porting::get_value_or(cache_settings.disk_path_protected, "none")
+          .c_str());
   // Create a task scheduler instance
   std::shared_ptr<olp::thread::TaskScheduler> task_scheduler =
       olp::client::OlpClientSettingsFactory::CreateDefaultTaskScheduler(1u);
@@ -84,8 +87,8 @@ int RunExampleReadWithCache(const AccessKey& access_key,
       olp::authentication::AuthenticationCredentials::ReadFromFile();
 
   // Initialize authentication settings.
-  olp::authentication::Settings settings{
-      read_credentials_result.get_value_or({access_key.id, access_key.secret})};
+  olp::authentication::Settings settings{olp::porting::get_value_or(
+      read_credentials_result, {access_key.id, access_key.secret})};
   settings.task_scheduler = task_scheduler;
   settings.network_request_handler = http_client;
 
@@ -108,23 +111,22 @@ int RunExampleReadWithCache(const AccessKey& access_key,
 
   // Create appropriate layer client with HRN, layer name and settings.
   olp::dataservice::read::VersionedLayerClient layer_client(
-      olp::client::HRN(catalog), first_layer_id, boost::none, client_settings);
+      olp::client::HRN(catalog), first_layer_id, olp::porting::none,
+      client_settings);
 
   // Retrieve the partition data
   // Create a DataRequest with appropriate LayerId and PartitionId
-  auto request = olp::dataservice::read::DataRequest()
-                   .WithPartitionId(first_partition_id)
-                   .WithBillingTag(boost::none);
-  if (cache_settings.disk_path_protected.is_initialized()) {
+  auto request =
+      olp::dataservice::read::DataRequest().WithPartitionId(first_partition_id);
+  if (cache_settings.disk_path_protected.has_value()) {
     request.WithFetchOption(olp::dataservice::read::FetchOptions::CacheOnly);
   }
 
   // Run the DataRequest
   auto future = layer_client.GetData(request);
 
   // Wait for DataResponse
-  olp::dataservice::read::DataResponse data_response =
-    future.GetFuture().get();
+  olp::dataservice::read::DataResponse data_response = future.GetFuture().get();
 
   // Compact mutable cache, so it can be used as protected cache
   cache->Compact();
@@ -133,8 +135,8 @@ int RunExampleReadWithCache(const AccessKey& access_key,
   return (HandleDataResponse(data_response) ? 0 : -1);
 }
 
-int RunExampleProtectedCache(const AccessKey& access_key, const std::string& catalog)
-{
+int RunExampleProtectedCache(const AccessKey& access_key,
+                             const std::string& catalog) {
   // Read data with mutable cache.
   olp::cache::CacheSettings cache_settings;
   cache_settings.disk_path_mutable =
@@ -145,6 +147,6 @@ int RunExampleProtectedCache(const AccessKey& access_key, const std::string& cat
   }
   // Read data with protected cache. Set mutable cache to none.
   cache_settings.disk_path_protected = cache_settings.disk_path_mutable;
-  cache_settings.disk_path_mutable = boost::none;
+  cache_settings.disk_path_mutable = olp::porting::none;
   return RunExampleReadWithCache(access_key, cache_settings, catalog);
 }

examples/ReadExample.cpp

@@ -119,7 +119,7 @@ bool HandleDataResponse(
 }  // namespace
 
 int RunExampleRead(const AccessKey& access_key, const std::string& catalog,
-                   const boost::optional<int64_t>& catalog_version) {
+                   const olp::porting::optional<int64_t>& catalog_version) {
   // Create a task scheduler instance
   std::shared_ptr<olp::thread::TaskScheduler> task_scheduler =
       olp::client::OlpClientSettingsFactory::CreateDefaultTaskScheduler(1u);
@@ -135,8 +135,8 @@ int RunExampleRead(const AccessKey& access_key, const std::string& catalog,
       olp::authentication::AuthenticationCredentials::ReadFromFile();
 
   // Initialize authentication settings.
-  olp::authentication::Settings settings{
-      read_credentials_result.get_value_or({access_key.id, access_key.secret})};
+  olp::authentication::Settings settings{olp::porting::get_value_or(
+      read_credentials_result, {access_key.id, access_key.secret})};
   settings.task_scheduler = task_scheduler;
   settings.network_request_handler = http_client;
 
@@ -161,8 +161,8 @@ int RunExampleRead(const AccessKey& access_key, const std::string& catalog,
         olp::client::HRN(catalog), client_settings);
 
     // Create CatalogRequest
-    auto request =
-        olp::dataservice::read::CatalogRequest().WithBillingTag(boost::none);
+    auto request = olp::dataservice::read::CatalogRequest().WithBillingTag(
+        olp::porting::none);
 
     // Run the CatalogRequest
     auto future = catalog_client.GetCatalog(request);
@@ -184,8 +184,8 @@ int RunExampleRead(const AccessKey& access_key, const std::string& catalog,
   if (!first_layer_id.empty()) {
     // Retrieve the partitions metadata
     // Create a PartitionsRequest with appropriate LayerId
-    auto request =
-        olp::dataservice::read::PartitionsRequest().WithBillingTag(boost::none);
+    auto request = olp::dataservice::read::PartitionsRequest().WithBillingTag(
+        olp::porting::none);
 
     // Run the PartitionsRequest
     auto future = layer_client.GetPartitions(request);
@@ -204,9 +204,8 @@ int RunExampleRead(const AccessKey& access_key, const std::string& catalog,
   if (!first_partition_id.empty()) {
     // Retrieve the partition data
     // Create a DataRequest with appropriate LayerId and PartitionId
-    auto request = olp::dataservice::read::DataRequest()
-                       .WithPartitionId(first_partition_id)
-                       .WithBillingTag(boost::none);
+    auto request = olp::dataservice::read::DataRequest().WithPartitionId(
+        first_partition_id);
 
     // Run the DataRequest
     auto future = layer_client.GetData(request);

examples/ReadExample.h

@@ -21,7 +21,7 @@
 
 #include "Examples.h"
 
-#include <boost/optional.hpp>
+#include <olp/core/porting/optional.hpp>
 
 /**
  * @brief Dataservice read example. Authenticate client using access key id and
@@ -35,4 +35,5 @@
  */
 int RunExampleRead(
     const AccessKey& access_key, const std::string& catalog,
-    const boost::optional<int64_t>& catalog_version = boost::none);
+                   const olp::porting::optional<int64_t>& catalog_version =
+                       olp::porting::none);

examples/StreamLayerReadExample.cpp

@@ -63,8 +63,7 @@ int GetDataFromMessages(dataread::StreamLayerClient& client,
     auto handle = message.GetMetaData().GetDataHandle();
     if (handle) {
       OLP_SDK_LOG_INFO_F(kLogTag, "Message data: handle - %s, size - %lu",
-                         handle.get().c_str(),
-                         message.GetMetaData().GetDataSize().get());
+                         handle->c_str(), *message.GetMetaData().GetDataSize());
       // use GetData(const model::Message& message) with message instance to
       // request actual data with data handle.
       auto message_future = client.GetData(message);
@@ -73,14 +72,14 @@ int GetDataFromMessages(dataread::StreamLayerClient& client,
         OLP_SDK_LOG_WARNING_F(kLogTag,
                               "Failed to get data for data handle %s - HTTP "
                               "Status: %d Message: %s",
-                              handle.get().c_str(),
+                              handle->c_str(),
                               message_result.GetError().GetHttpStatusCode(),
                               message_result.GetError().GetMessage().c_str());
         continue;
       }
       auto message_data = message_result.MoveResult();
       OLP_SDK_LOG_INFO_F(kLogTag, "GetData for %s successful: size - %lu",
-                         handle.get().c_str(), message_data->size());
+                         handle->c_str(), message_data->size());
     } else {
       // If data is less than 1 MB, the data  content published directly in the
       // metadata and encoded in Base64.

examples/main.cpp

@@ -68,7 +68,7 @@ int RequiredArgumentError(const tools::Option& arg) {
 }
 int ParseArguments(const int argc, char** argv, AccessKey& access_key,
                    std::string& catalog,
-                   boost::optional<int64_t>& catalog_version,
+                   olp::porting::optional<int64_t>& catalog_version,
                    std::string& layer_id,
                    olp::dataservice::read::SubscribeRequest::SubscriptionMode&
                        subscription_mode) {
@@ -128,7 +128,7 @@ int ParseArguments(const int argc, char** argv, AccessKey& access_key,
       if (ss.fail() || !ss.eof()) {
         std::cout << "invalid catalog version value -- '" << *it
                   << "', but int64 is expected." << std::endl;
-        catalog_version = boost::none;
+        catalog_version = olp::porting::none;
       }
     } else if (IsMatch(*it, tools::kLayerIdOption)) {
       if (++it == arguments.end()) {
@@ -168,7 +168,7 @@ int ParseArguments(const int argc, char** argv, AccessKey& access_key,
 
 int RunExamples(const AccessKey& access_key, int examples_to_run,
                 const std::string& catalog,
-                const boost::optional<int64_t>& catalog_version,
+                const olp::porting::optional<int64_t>& catalog_version,
                 const std::string& layer_id,
                 olp::dataservice::read::SubscribeRequest::SubscriptionMode
                     subscription_mode) {
@@ -213,7 +213,7 @@ int main(int argc, char** argv) {
   std::string catalog;     // the HRN of the catalog to which you to publish data
   std::string layer_id;    // the of the layer inside the catalog to which you
                            // want to publish data
-  boost::optional<int64_t> catalog_version;  // version of the catalog.
+  olp::porting::optional<int64_t> catalog_version;  // version of the catalog.
   auto subscription_mode =
       olp::dataservice::read::SubscribeRequest::SubscriptionMode::
           kSerial;  // subscription mode for read stream layer example

external/CMakeLists.txt

@@ -97,7 +97,7 @@ if(OLP_SDK_ENABLE_DEFAULT_CACHE_LMDB)
     endif()
 endif()
 
-find_package(Boost QUIET)
+find_package(Boost 1.82.0 QUIET)
 if(NOT TARGET Boost AND NOT Boost_FOUND)
     add_subdirectory(boost)
     set(BOOST_ROOT ${EXTERNAL_BOOST_ROOT} PARENT_SCOPE)

olp-cpp-sdk-authentication/include/olp/authentication/AuthenticationClient.h

@@ -36,7 +36,7 @@
 #include <olp/authentication/Types.h>
 #include <olp/core/client/ApiResponse.h>
 #include <olp/core/client/CancellationToken.h>
-#include <boost/optional.hpp>
+#include <olp/core/porting/optional.hpp>
 
 /**
  * @brief Rules all the other namespaces.
@@ -65,15 +65,15 @@ class AUTHENTICATION_API AuthenticationClient {
     /**
      * @brief (Optional) The scope assigned to the access token.
      */
-    boost::optional<std::string> scope{boost::none};
+    porting::optional<std::string> scope{porting::none};
 
     /**
      * @brief (Optional) The device ID assigned to the access token.
      *
      * @note This field is only necessary if you want to apply a oauth rate
      * limit on a particular device.
      */
-    boost::optional<std::string> device_id{boost::none};
+    porting::optional<std::string> device_id{porting::none};
 
     /**
      * @brief (Optional) The number of seconds left before the access
@@ -89,7 +89,7 @@ class AUTHENTICATION_API AuthenticationClient {
      * requires it. Fully overrides default body and resets the request content
      * type.
      */
-    boost::optional<std::string> custom_body{boost::none};
+    porting::optional<std::string> custom_body{olp::porting::none};
   };
 
   /**