Fix pqv1 max partitions (#8485)

niksaveliev · web-flow · commit b37864739589 · 2024-08-30T12:14:28.000+05:00
diff --git a/ydb/public/sdk/cpp/client/ydb_persqueue_public/impl/persqueue_impl.h b/ydb/public/sdk/cpp/client/ydb_persqueue_public/impl/persqueue_impl.h
@@ -56,7 +56,7 @@ class TPersQueueClient::TImpl : public TClientImplCommon<TPersQueueClient::TImpl
 
         bool autoPartitioningSettingsDefined = false;
         if (settings.MaxPartitionsCount_.Defined()) {
-            props.mutable_auto_partitioning_settings()->set_max_active_partitions(settings.PartitionsCount_);
+            props.mutable_auto_partitioning_settings()->set_max_active_partitions(*settings.MaxPartitionsCount_);
             autoPartitioningSettingsDefined = true;
         }
         if (settings.AutoPartitioningStrategy_.Defined()) {
@@ -76,9 +76,7 @@ class TPersQueueClient::TImpl : public TClientImplCommon<TPersQueueClient::TImpl
             autoPartitioningSettingsDefined = true;
         }
 
-        if (!autoPartitioningSettingsDefined) {
-            props.set_partitions_count(settings.PartitionsCount_);
-        } else {
+        if (autoPartitioningSettingsDefined) {
             props.mutable_auto_partitioning_settings()->set_min_active_partitions(settings.PartitionsCount_);
         }
 

Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@ class TPersQueueClient::TImpl : public TClientImplCommon<TPersQueueClient::TImpl`
`56`	`56`
`57`	`57`	`bool autoPartitioningSettingsDefined = false;`
`58`	`58`	`if (settings.MaxPartitionsCount_.Defined()) {`
`59`		`- props.mutable_auto_partitioning_settings()->set_max_active_partitions(settings.PartitionsCount_);`
	`59`	`+ props.mutable_auto_partitioning_settings()->set_max_active_partitions(*settings.MaxPartitionsCount_);`
`60`	`60`	`autoPartitioningSettingsDefined = true;`
`61`	`61`	`}`
`62`	`62`	`if (settings.AutoPartitioningStrategy_.Defined()) {`
`@@ -76,9 +76,7 @@ class TPersQueueClient::TImpl : public TClientImplCommon<TPersQueueClient::TImpl`
`76`	`76`	`autoPartitioningSettingsDefined = true;`
`77`	`77`	`}`
`78`	`78`
`79`		`- if (!autoPartitioningSettingsDefined) {`
`80`		`- props.set_partitions_count(settings.PartitionsCount_);`
`81`		`- } else {`
	`79`	`+ if (autoPartitioningSettingsDefined) {`
`82`	`80`	`props.mutable_auto_partitioning_settings()->set_min_active_partitions(settings.PartitionsCount_);`
`83`	`81`	`}`
`84`	`82`