Skip to content

Commit 47e7794

Browse files
author
lukyan
committed
YT-21484: Consider partition sizes in coordination by sample keys
commit_hash:56feb3c28bc4d963431d8f70140fd2ae2406da78
1 parent 6d72638 commit 47e7794

File tree

3 files changed

+12
-0
lines changed

3 files changed

+12
-0
lines changed

yt/yt/client/api/client_common.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -144,6 +144,8 @@ struct TSelectRowsOptionsBase
144144
ui64 RangeExpansionLimit = 200000;
145145
//! Limits maximum parallel subqueries.
146146
int MaxSubqueries = std::numeric_limits<int>::max();
147+
//! Limits parallel subqueries by row count.
148+
ui64 MinRowCountPerSubquery = 100'000;
147149
//! Path in Cypress with UDFs.
148150
std::optional<TString> UdfRegistryPath;
149151
//! If |true| then logging is more verbose.

yt/yt/client/driver/command-inl.h

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -392,6 +392,14 @@ void TSelectRowsCommandBase<
392392
.GreaterThan(0)
393393
.Optional(/*init*/ false);
394394

395+
registrar.template ParameterWithUniversalAccessor<ui64>(
396+
"min_row_count_per_subquery",
397+
[] (TThis* command) -> auto& {
398+
return command->Options.MinRowCountPerSubquery;
399+
})
400+
.GreaterThan(0)
401+
.Optional(/*init*/ false);
402+
395403
registrar.template ParameterWithUniversalAccessor<std::optional<TString>>(
396404
"udf_registry_path",
397405
[] (TThis* command) -> auto& {

yt/yt_proto/yt/client/api/rpc_proxy/proto/api_service.proto

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -638,6 +638,7 @@ message TReqSelectRows
638638
optional int32 execution_backend = 23; // EExecutionBackend
639639
optional NYT.NTableClient.NProto.TVersionedReadOptions versioned_read_options = 25;
640640
optional bool use_lookup_cache = 26;
641+
optional int64 min_row_count_per_subquery = 27;
641642

642643
optional TSuppressableAccessTrackingOptions suppressable_access_tracking_options = 104;
643644

@@ -836,6 +837,7 @@ message TReqExplainQuery
836837
optional string execution_pool = 11;
837838
optional bool new_range_inference = 12;
838839
optional int32 syntax_version = 13 [default = 1];
840+
optional int64 min_row_count_per_subquery = 14;
839841
}
840842

841843
message TRspExplainQuery

0 commit comments

Comments
 (0)