diff options
author | thegeorg <thegeorg@yandex-team.com> | 2022-11-24 19:34:10 +0300 |
---|---|---|
committer | thegeorg <thegeorg@yandex-team.com> | 2022-11-24 19:34:10 +0300 |
commit | 0c96f4daac274910e189aac1c9f1bfa7f9c7add0 (patch) | |
tree | b3d8581d94f1c24968d24ef2453627c00a4ac07f | |
parent | 6bb204fd7976045e3c045f7fc7cfdbb836e86d46 (diff) | |
download | ydb-0c96f4daac274910e189aac1c9f1bfa7f9c7add0.tar.gz |
Fix first layer of -Wunused-but-set-variable warnings
-rw-r--r-- | ydb/core/kqp/compute_actor/kqp_compute_actor.cpp | 4 | ||||
-rw-r--r-- | ydb/library/arrow_clickhouse/Aggregator.cpp | 9 |
2 files changed, 0 insertions, 13 deletions
diff --git a/ydb/core/kqp/compute_actor/kqp_compute_actor.cpp b/ydb/core/kqp/compute_actor/kqp_compute_actor.cpp index f9eab53745d..0ba8e1d6ea0 100644 --- a/ydb/core/kqp/compute_actor/kqp_compute_actor.cpp +++ b/ydb/core/kqp/compute_actor/kqp_compute_actor.cpp @@ -61,14 +61,12 @@ void TShardsScanningPolicy::FillRequestScanFeatures(const NKikimrTxDataShard::TK maxInFlight = 1; NKikimrSSA::TProgram program; - bool hasNoGroupBy = false; bool hasGroupByWithFields = false; bool hasGroupByWithNoFields = false; if (meta.HasOlapProgram()) { Y_VERIFY(program.ParseFromString(meta.GetOlapProgram().GetProgram())); for (auto&& command : program.GetCommand()) { if (!command.HasGroupBy()) { - hasNoGroupBy = true; continue; } if (command.GetGroupBy().GetKeyColumns().size()) { @@ -77,8 +75,6 @@ void TShardsScanningPolicy::FillRequestScanFeatures(const NKikimrTxDataShard::TK hasGroupByWithNoFields = true; } } - } else { - hasNoGroupBy = true; } isAggregationRequest = hasGroupByWithFields || hasGroupByWithNoFields; if (isSorted) { diff --git a/ydb/library/arrow_clickhouse/Aggregator.cpp b/ydb/library/arrow_clickhouse/Aggregator.cpp index b71f4538006..3f62b50a5ba 100644 --- a/ydb/library/arrow_clickhouse/Aggregator.cpp +++ b/ydb/library/arrow_clickhouse/Aggregator.cpp @@ -932,15 +932,6 @@ BlocksList Aggregator::convertToBlocks(AggregatedDataVariants & data_variants, b data_variants.aggregator = nullptr; } - size_t rows = 0; - //size_t bytes = 0; - - for (const auto & block : blocks) - { - rows += block->num_rows(); - //bytes += block.bytes(); - } - return blocks; } |