move from and to bound to separate message

Author: niksaveliev

ydb/public/api/protos/ydb_topic.proto

@@ -1019,6 +1019,13 @@ message DescribeTopicResponse {
     Ydb.Operations.Operation operation = 1;
 }
 
+message PartitionKeyRange {
+    // Inclusive left border. Emptiness means -inf.
+    optional bytes from_bound = 1;
+    // Exclusive right border. Emptiness means +inf.
+    optional bytes to_bound = 2;
+}
+
 // Describe topic result message that will be inside DescribeTopicResponse.operation.
 message DescribeTopicResult {
     // Description of scheme object.
@@ -1088,10 +1095,7 @@ message DescribeTopicResult {
         // Partition location, filled only when include_location in request is true.
         PartitionLocation partition_location = 6;
 
-        // Inclusive left border. Emptiness means -inf.
-        optional bytes from_bound = 7;
-        // Exclusive right border. Emptiness means +inf.
-        optional bytes to_bound = 8;
+        PartitionKeyRange key_range = 7;
     }
 
     message TopicStats {

ydb/public/sdk/cpp/client/ydb_topic/impl/topic.cpp

@@ -404,12 +404,12 @@ TPartitionInfo::TPartitionInfo(const Ydb::Topic::DescribeTopicResult::PartitionI
         PartitionLocation_ = TPartitionLocation{partitionInfo.partition_location()};
     }
 
-    if (partitionInfo.has_from_bound()) {
-        FromBound_ = TString(partitionInfo.from_bound());
+    if (partitionInfo.has_key_range() && partitionInfo.key_range().has_from_bound()) {
+        FromBound_ = TString(partitionInfo.key_range().from_bound());
     }
 
-    if (partitionInfo.has_to_bound()) {
-        ToBound_ = TString(partitionInfo.to_bound());
+    if (partitionInfo.has_key_range() && partitionInfo.key_range().has_to_bound()) {
+        ToBound_ = TString(partitionInfo.key_range().to_bound());
     }
 }
 

ydb/services/persqueue_v1/actors/schema_actors.cpp

@@ -1076,10 +1076,10 @@ void TDescribeTopicActor::HandleCacheNavigateResponse(TEvTxProxySchemeCache::TEv
             destPart->set_active(sourcePart.GetStatus() == ::NKikimrPQ::ETopicPartitionStatus::Active);
             if (sourcePart.HasKeyRange()) {
                 if (sourcePart.GetKeyRange().HasFromBound()) {
-                    destPart->set_from_bound(sourcePart.GetKeyRange().GetFromBound());
+                    destPart->mutable_key_range()->set_from_bound(sourcePart.GetKeyRange().GetFromBound());
                 }
                 if (sourcePart.GetKeyRange().HasToBound()) {
-                    destPart->set_to_bound(sourcePart.GetKeyRange().GetToBound());
+                    destPart->mutable_key_range()->set_to_bound(sourcePart.GetKeyRange().GetToBound());
                 }
             }