review fix ydb-platform#2

- remove the new option to control index impl tables partition split boundaries inclusion in the describe result of a table from public API. We will work on `ydb tools dump` later to enable it to save index tables split boundaries.

Author: jepett0

ydb/core/grpc_services/rpc_describe_table.cpp

@@ -205,13 +205,10 @@ class TDescribeTableRPC : public TRpcSchemeRequestActor<TDescribeTableRPC, TEvDe
         SetDatabase(navigateRequest.get(), *Request_);
         NKikimrSchemeOp::TDescribePath* record = navigateRequest->Record.MutableDescribePath();
         record->SetPath(path);
-
         if (req->include_shard_key_bounds()) {
             record->MutableOptions()->SetReturnBoundaries(true);
         }
-        if (req->include_index_table_shard_key_bounds()) {
-            record->MutableOptions()->SetReturnIndexTableBoundaries(true);
-        }
+
         if (req->include_partition_stats() && req->include_table_stats()) {
             record->MutableOptions()->SetReturnPartitionStats(true);
         }

ydb/library/backup/backup.cpp

@@ -373,9 +373,7 @@ NTable::TTableDescription DescribeTable(TDriver driver, const TString& fullTable
     NTable::TTableClient client(driver);
 
     TStatus status = client.RetryOperationSync([fullTablePath, &desc](NTable::TSession session) {
-        auto settings = NTable::TDescribeTableSettings()
-            .WithKeyShardBoundary(true)
-            .WithIndexTableKeyShardBoundary(true);
+        auto settings = NTable::TDescribeTableSettings().WithKeyShardBoundary(true);
         auto result = session.DescribeTable(fullTablePath, settings).GetValueSync();
 
         VerifyStatus(result);

ydb/public/api/protos/ydb_table.proto

@@ -72,7 +72,7 @@ message VectorIndexSettings {
         SIMILARITY_UNSPECIFIED = 0;
         SIMILARITY_COSINE = 1;
         SIMILARITY_INNER_PRODUCT = 2;
-    }
+    }   
 
     enum VectorType {
         VECTOR_TYPE_UNSPECIFIED = 0;
@@ -773,16 +773,12 @@ message DescribeTableRequest {
     // Full path
     string path = 2;
     Ydb.Operations.OperationParams operation_params = 4;
-    // Includes partition boundaries (the key values where the partitions are split)
+    // Includes shard key distribution info
     bool include_shard_key_bounds = 5;
     // Includes table statistics
     bool include_table_stats = 6;
     // Includes partition statistics (required include_table_statistics)
     bool include_partition_stats = 7;
-    // Note: The following flag applies only if the described table has indices.
-    // Includes partition boundaries (the key values where the partitions are split)
-    // for tables that implement indices of the described table.
-    bool include_index_table_shard_key_bounds = 8;
 }
 
 message DescribeTableResponse {

ydb/public/sdk/cpp/client/ydb_table/impl/table_client.cpp

@@ -513,16 +513,14 @@ TAsyncDescribeTableResult TTableClient::TImpl::DescribeTable(const TString& sess
     auto request = MakeOperationRequest<Ydb::Table::DescribeTableRequest>(settings);
     request.set_session_id(sessionId);
     request.set_path(path);
-
     if (settings.WithKeyShardBoundary_) {
         request.set_include_shard_key_bounds(true);
     }
-    if (settings.WithIndexTableKeyShardBoundary_) {
-        request.set_include_index_table_shard_key_bounds(true);
-    }
+
     if (settings.WithTableStatistics_) {
         request.set_include_table_stats(true);
     }
+
     if (settings.WithPartitionStatistics_) {
         request.set_include_partition_stats(true);
     }

ydb/public/sdk/cpp/client/ydb_table/table.h

@@ -1627,7 +1627,6 @@ struct TRenameTablesSettings : public TOperationRequestSettings<TRenameTablesSet
 
 struct TDescribeTableSettings : public TOperationRequestSettings<TDescribeTableSettings> {
     FLUENT_SETTING_DEFAULT(bool, WithKeyShardBoundary, false);
-    FLUENT_SETTING_DEFAULT(bool, WithIndexTableKeyShardBoundary, false);
     FLUENT_SETTING_DEFAULT(bool, WithTableStatistics, false);
     FLUENT_SETTING_DEFAULT(bool, WithPartitionStatistics, false);
 };

ydb/services/ydb/backup_ut/ydb_backup_ut.cpp

@@ -416,28 +416,7 @@ Y_UNIT_TEST_SUITE(BackupRestore) {
         );
     }
 
-    Y_UNIT_TEST(RestoreIndexTableSplitBoundaries) {
-        TKikimrWithGrpcAndRootSchema server;
-        auto driver = TDriver(TDriverConfig().SetEndpoint(Sprintf("localhost:%u", server.GetPort())));
-        TTableClient tableClient(driver);
-        auto session = tableClient.GetSession().ExtractValueSync().GetSession();
-        TTempDir tempDir;
-        const auto& pathToBackup = tempDir.Path();
-
-        constexpr const char* table = "/Root/table";
-        constexpr const char* index = "byValue";
-        constexpr ui64 indexPartitions = 10;
-
-        TestIndexTableSplitBoundariesArePreserved(
-            table,
-            index,
-            indexPartitions,
-            session,
-            CreateBackupLambda(driver, pathToBackup, true),
-            CreateRestoreLambda(driver, pathToBackup)
-        );
-    }
-
+    // TO DO: test index impl table split boundaries restoration from a backup
 }
 
 Y_UNIT_TEST_SUITE(BackupRestoreS3) {