Tables cleanup work

sdk/tables/assets.json

@@ -2,5 +2,5 @@
   "AssetsRepo": "Azure/azure-sdk-assets",
   "AssetsRepoPrefixPath": "cpp",
   "TagPrefix": "cpp/tables",
-  "Tag": "cpp/tables_cb42be8953"
+  "Tag": "cpp/tables_0b3363cab7"
 }

sdk/tables/azure-data-tables/inc/azure/data/tables/account_sas_builder.hpp

@@ -107,14 +107,14 @@ namespace Azure { namespace Data { namespace Tables { namespace Sas {
     Delete = 4,
 
     /**
-     * @brief Indicates that Add is permitted.
+     * @brief Indicates that List is permitted.
      */
-    Add = 8,
+    List = 8,
 
     /**
-     * @brief Indicates that List is permitted.
+     * @brief Indicates that Add is permitted.
      */
-    List = 16,
+    Add = 16,
 
     /**
      * @brief Indicates that Update is permitted.

sdk/tables/azure-data-tables/inc/azure/data/tables/table_client.hpp

@@ -88,21 +88,6 @@ namespace Azure { namespace Data { namespace Tables {
         std::shared_ptr<Core::Credentials::TokenCredential> credential,
         const TableClientOptions& options = {});
 
-    /**
-     * @brief Initializes a new instance of tableClient.
-     *
-     * @param tableName The name of the table.
-     * @param credential The named key credential used to sign requests.
-     * @param url A url referencing the table that includes the name of the account and the name of
-     * @param options Optional client options that define the transport pipeline policies for
-     * authentication, retries, etc., that are applied to every request.
-     */
-    explicit TableClient(
-        const std::string& tableName,
-        std::shared_ptr<Azure::Data::Tables::Credentials::NamedKeyCredential> credential,
-        std::string url,
-        const TableClientOptions& options = {});
-
     /**
      * @brief Initializes a new instance of tableClient.
      *
@@ -118,41 +103,6 @@ namespace Azure { namespace Data { namespace Tables {
         const std::string& tableName,
         const TableClientOptions& options = {});
 
-    /**
-     * @brief Initializes a new instance of tableClient.
-     *
-     * @param connectionString the connection string used to initialize.
-     * @param tableName The name of the table.
-     * @param options Optional client options that define the transport pipeline policies for
-     * authentication, retries, etc., that are applied to every request.
-     * @return TableClient.
-     */
-    static TableClient CreateFromConnectionString(
-        const std::string& connectionString,
-        const std::string& tableName,
-        const TableClientOptions& options = {});
-
-    /**
-     * @brief Retrieves details about any stored access policies specified on the table that may be
-     * used with Shared Access Signatures.
-     *
-     * @param context for canceling long running operations.
-     * @return Get access policy result.
-     */
-    Response<Models::TableAccessPolicy> GetAccessPolicy(Core::Context const& context = {});
-
-    /**
-     * @brief Sets stored access policies for the table that may be used with Shared Access
-     * Signatures.
-     *
-     * @param tableAccessPolicy The TableAccessPolicy to set.
-     * @param context for canceling long running operations.
-     * @return Set access policy result.
-     */
-    Response<Models::SetTableAccessPolicyResult> SetAccessPolicy(
-        Models::TableAccessPolicy const& tableAccessPolicy,
-        Core::Context const& context = {});
-
     /**
      * @brief Add entity in a table.
      *
@@ -236,14 +186,14 @@ namespace Azure { namespace Data { namespace Tables {
      * @return Table entity.
      */
     Response<Models::TableEntity> GetEntity(
-        const std::string& partitionKey,
-        const std::string& rowKey,
+        std::string const& partitionKey,
+        std::string const& rowKey,
         Core::Context const& context = {});
 
     /**
      * @brief Submits a transaction.
      *
-     * @param steps the transaction steps to execute.
+     * @param steps The transaction steps to execute.
      * @param context for canceling long running operations.
      * @return Submit transaction result.
      */

sdk/tables/azure-data-tables/inc/azure/data/tables/table_service_client.hpp

@@ -50,20 +50,6 @@ namespace Azure { namespace Data { namespace Tables {
         std::shared_ptr<Core::Credentials::TokenCredential> credential,
         const TableClientOptions& options = {});
 
-    /**
-     * @brief Initializes a new instance of tableClient.
-     *
-     * @param serviceUrl A url referencing the table that includes the name of the account and the
-     * name of the table.
-     * @param credential The named key credential used to sign requests.
-     * @param options Optional client options that define the transport pipeline policies for
-     * authentication, retries, etc., that are applied to every request.
-     */
-    explicit TableServiceClient(
-        const std::string& serviceUrl,
-        std::shared_ptr<Azure::Data::Tables::Credentials::NamedKeyCredential> credential,
-        const TableClientOptions& options = {});
-
     /**
      * @brief Initializes a new instance of tableClient.
      *
@@ -78,18 +64,6 @@ namespace Azure { namespace Data { namespace Tables {
         std::shared_ptr<Azure::Data::Tables::Credentials::AzureSasCredential> credential,
         const TableClientOptions& options = {});
 
-    /**
-     * @brief Initializes a new instance of tableClient.
-     *
-     * @param connectionString the connection string used to initialize.
-     * @param options Optional client options that define the transport pipeline policies for
-     * authentication, retries, etc., that are applied to every request.
-     * @return TableServiceClient.
-     */
-    static TableServiceClient CreateFromConnectionString(
-        const std::string& connectionString,
-        const TableClientOptions& options = {});
-
     /**
      * @brief Creates a new table under the given account.
      *
@@ -176,7 +150,6 @@ namespace Azure { namespace Data { namespace Tables {
   private:
     std::shared_ptr<Core::Http::_internal::HttpPipeline> m_pipeline;
     std::shared_ptr<Core::Credentials::TokenCredential> m_tokenCredential;
-    std::shared_ptr<Azure::Data::Tables::Credentials::NamedKeyCredential> m_namedKeyCredential;
     Core::Url m_url;
   };
 }}} // namespace Azure::Data::Tables

sdk/tables/azure-data-tables/samples/CMakeLists.txt

@@ -8,26 +8,26 @@ if(MSVC)
 endif()
 
 add_executable(tables-getting-started tables_getting_started.cpp )
-target_link_libraries(tables-getting-started PRIVATE azure-data-tables get-env-helper)
+target_link_libraries(tables-getting-started PRIVATE azure-data-tables azure-identity get-env-helper)
 create_per_service_target_build_for_sample(tables tables-getting-started)
 target_compile_definitions(tables-getting-started PRIVATE _azure_BUILDING_SAMPLES)
 
 add_executable(tables_entity_operations tables_entity_operations.cpp)
-target_link_libraries(tables_entity_operations PRIVATE azure-data-tables get-env-helper)
+target_link_libraries(tables_entity_operations PRIVATE azure-data-tables azure-identity get-env-helper)
 create_per_service_target_build_for_sample(tables tables_entity_operations)
 target_compile_definitions(tables_entity_operations PRIVATE _azure_BUILDING_SAMPLES)
 
 add_executable(tables_service_operations tables_service_operations.cpp)
-target_link_libraries(tables_service_operations PRIVATE azure-data-tables get-env-helper)
+target_link_libraries(tables_service_operations PRIVATE azure-data-tables azure-identity get-env-helper)
 create_per_service_target_build_for_sample(tables tables_service_operations)
 target_compile_definitions(tables_service_operations PRIVATE _azure_BUILDING_SAMPLES)
 
 add_executable(tables_transactions_ok tables_transactions_ok.cpp)
-target_link_libraries(tables_transactions_ok PRIVATE azure-data-tables get-env-helper)
+target_link_libraries(tables_transactions_ok PRIVATE azure-data-tables azure-identity get-env-helper)
 create_per_service_target_build_for_sample(tables tables_transactions_ok)
 target_compile_definitions(tables_transactions_ok PRIVATE _azure_BUILDING_SAMPLES)
 
 add_executable(tables_transactions_fail tables_transactions_fail.cpp)
-target_link_libraries(tables_transactions_fail PRIVATE azure-data-tables get-env-helper)
+target_link_libraries(tables_transactions_fail PRIVATE azure-data-tables azure-identity get-env-helper)
 create_per_service_target_build_for_sample(tables tables_transactions_fail)
 target_compile_definitions(tables_transactions_fail PRIVATE _azure_BUILDING_SAMPLES)

sdk/tables/azure-data-tables/samples/tables_entity_operations.cpp

@@ -2,6 +2,7 @@
 // Licensed under the MIT License.
 
 #include <azure/data/tables.hpp>
+#include <azure/identity.hpp>
 
 #include <cstdio>
 #include <iostream>
@@ -12,26 +13,14 @@ using namespace Azure::Data::Tables;
 using namespace Azure::Data::Tables::Models;
 const std::string TableName = "table";
 
-std::string GetConnectionString()
-{
-  const static std::string ConnectionString = "";
-
-  if (!ConnectionString.empty())
-  {
-    return ConnectionString;
-  }
-  const static std::string envConnectionString = std::getenv("STANDARD_STORAGE_CONNECTION_STRING");
-  if (!envConnectionString.empty())
-  {
-    return envConnectionString;
-  }
-  throw std::runtime_error("Cannot find connection string.");
-}
-
 int main()
 {
-  auto tableServiceClient = TableServiceClient::CreateFromConnectionString(GetConnectionString());
-  auto tableClient = TableClient::CreateFromConnectionString(GetConnectionString(), TableName);
+  const std::string accountName{std::getenv("ACCOUNT_NAME")};
+  auto credential = std::make_shared<Azure::Identity::DefaultAzureCredential>();
+  auto tableServiceClient = Azure::Data::Tables::TableServiceClient(
+      "https://" + accountName + ".table.core.windows.net/", credential);
+  auto tableClient = Azure::Data::Tables::TableClient(
+      "https://" + accountName + ".table.core.windows.net/", TableName, credential);
 
   // create new table
   tableServiceClient.CreateTable(TableName);

sdk/tables/azure-data-tables/samples/tables_getting_started.cpp

@@ -2,34 +2,22 @@
 // Licensed under the MIT License.
 
 #include <azure/data/tables.hpp>
+#include <azure/identity.hpp>
 
 #include <cstdio>
 #include <iostream>
 #include <stdexcept>
 #include <thread>
 
-std::string GetConnectionString()
-{
-  const static std::string ConnectionString = "";
-
-  if (!ConnectionString.empty())
-  {
-    return ConnectionString;
-  }
-  const static std::string envConnectionString = std::getenv("STANDARD_STORAGE_CONNECTION_STRING");
-  if (!envConnectionString.empty())
-  {
-    return envConnectionString;
-  }
-  throw std::runtime_error("Cannot find connection string.");
-}
-
 using namespace Azure::Data::Tables;
 const std::string TableName = "sample1";
 
 int main()
 {
-  auto tableServiceClient = TableServiceClient::CreateFromConnectionString(GetConnectionString());
+  const std::string accountName{std::getenv("ACCOUNT_NAME")};
+  auto credential = std::make_shared<Azure::Identity::DefaultAzureCredential>();
+  auto tableServiceClient = Azure::Data::Tables::TableServiceClient(
+      "https://" + accountName + ".table.core.windows.net/", credential);
 
   // create new table
   tableServiceClient.CreateTable(TableName);

sdk/tables/azure-data-tables/samples/tables_service_operations.cpp

@@ -2,35 +2,22 @@
 // Licensed under the MIT License.
 
 #include <azure/data/tables.hpp>
+#include <azure/identity.hpp>
 
 #include <cstdio>
 #include <iostream>
 #include <stdexcept>
 #include <thread>
 
-std::string GetConnectionString()
-{
-  const static std::string ConnectionString = "";
-
-  if (!ConnectionString.empty())
-  {
-    return ConnectionString;
-  }
-  const static std::string envConnectionString = std::getenv("STANDARD_STORAGE_CONNECTION_STRING");
-  if (!envConnectionString.empty())
-  {
-    return envConnectionString;
-  }
-  throw std::runtime_error("Cannot find connection string.");
-}
-
 using namespace Azure::Data::Tables;
 const std::string TableName = "sample1";
 
 int main()
 {
-  auto tableServiceClient = TableServiceClient::CreateFromConnectionString(GetConnectionString());
-
+  const std::string accountName{std::getenv("ACCOUNT_NAME")};
+  auto credential = std::make_shared<Azure::Identity::DefaultAzureCredential>();
+  auto tableServiceClient = Azure::Data::Tables::TableServiceClient(
+      "https://" + accountName + ".table.core.windows.net/", credential);
   // query tables
   auto tables = tableServiceClient.QueryTables();
 

sdk/tables/azure-data-tables/samples/tables_transactions_fail.cpp

@@ -2,6 +2,7 @@
 // Licensed under the MIT License.
 
 #include <azure/data/tables.hpp>
+#include <azure/identity.hpp>
 
 #include <cstdio>
 #include <iostream>
@@ -13,26 +14,13 @@ using namespace Azure::Data::Tables::Models;
 
 const std::string TableName = "transactions2";
 
-std::string GetConnectionString()
-{
-  const static std::string ConnectionString = "";
-
-  if (!ConnectionString.empty())
-  {
-    return ConnectionString;
-  }
-  const static std::string envConnectionString = std::getenv("STANDARD_STORAGE_CONNECTION_STRING");
-  if (!envConnectionString.empty())
-  {
-    return envConnectionString;
-  }
-  throw std::runtime_error("Cannot find connection string.");
-}
-
 int main()
 {
-  // Get the service client
-  auto tableServiceClient = TableServiceClient::CreateFromConnectionString(GetConnectionString());
+  const std::string accountName{std::getenv("ACCOUNT_NAME")};
+  auto credential = std::make_shared<Azure::Identity::DefaultAzureCredential>();
+  auto tableServiceClient = Azure::Data::Tables::TableServiceClient(
+      "https://" + accountName + ".table.core.windows.net/", credential);
+
   // create table
   tableServiceClient.CreateTable(TableName);
   // get table client from table service client

sdk/tables/azure-data-tables/samples/tables_transactions_ok.cpp

@@ -2,6 +2,7 @@
 // Licensed under the MIT License.
 
 #include <azure/data/tables.hpp>
+#include <azure/identity.hpp>
 
 #include <cstdio>
 #include <iostream>
@@ -13,27 +14,13 @@ using namespace Azure::Data::Tables::Models;
 
 const std::string TableName = "transactions";
 
-std::string GetConnectionString()
-{
-  const static std::string ConnectionString = "";
-
-  if (!ConnectionString.empty())
-  {
-    return ConnectionString;
-  }
-  const static std::string envConnectionString = std::getenv("STANDARD_STORAGE_CONNECTION_STRING");
-  if (!envConnectionString.empty())
-  {
-    return envConnectionString;
-  }
-  throw std::runtime_error("Cannot find connection string.");
-}
-
 int main()
 {
-  // Get the service client
-  auto tableServiceClient = TableServiceClient::CreateFromConnectionString(GetConnectionString());
-  // create table
+  const std::string accountName{std::getenv("ACCOUNT_NAME")};
+  auto credential = std::make_shared<Azure::Identity::DefaultAzureCredential>();
+  auto tableServiceClient = Azure::Data::Tables::TableServiceClient(
+      "https://" + accountName + ".table.core.windows.net/", credential);
+
   tableServiceClient.CreateTable(TableName);
   // get table client from table service client
   auto tableClient = tableServiceClient.GetTableClient(TableName);