aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorthegeorg <thegeorg@yandex-team.com>2022-11-24 19:34:10 +0300
committerthegeorg <thegeorg@yandex-team.com>2022-11-24 19:34:10 +0300
commit0c96f4daac274910e189aac1c9f1bfa7f9c7add0 (patch)
treeb3d8581d94f1c24968d24ef2453627c00a4ac07f
parent6bb204fd7976045e3c045f7fc7cfdbb836e86d46 (diff)
downloadydb-0c96f4daac274910e189aac1c9f1bfa7f9c7add0.tar.gz
Fix first layer of -Wunused-but-set-variable warnings
-rw-r--r--ydb/core/kqp/compute_actor/kqp_compute_actor.cpp4
-rw-r--r--ydb/library/arrow_clickhouse/Aggregator.cpp9
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;
}