From a3ee99554bbe06a6038ca92bb602a8e3e949ecc0 Mon Sep 17 00:00:00 2001 From: Duarte Nunes Date: Thu, 10 Aug 2017 23:53:51 +0100 Subject: [PATCH] service/storage_proxy: Remove out of date comment Now that we don't go directly to reconciliation for range queries, the result isn't required to have the row and partition counts calculated (we no longer transform a reconciled_result to a query::result). Furthermore, this line was causing a lot of dtests to fail on account of them not expecting an error line in the logs. Signed-off-by: Duarte Nunes Message-Id: <20170810225351.12610-1-duarte@scylladb.com> --- service/storage_proxy.cc | 1 - 1 file changed, 1 deletion(-) diff --git a/service/storage_proxy.cc b/service/storage_proxy.cc index c32cc2312b..a1049f6413 100644 --- a/service/storage_proxy.cc +++ b/service/storage_proxy.cc @@ -3011,7 +3011,6 @@ storage_proxy::query_partition_key_range_concurrent(storage_proxy::clock_type::t cl, cmd, concurrency_factor, timeout, remaining_row_count, remaining_partition_count, trace_state = std::move(trace_state)] (foreign_ptr>&& result) mutable { if (!result->row_count() || !result->partition_count()) { - slogger.error("no row count in query result, should not happen here"); result->calculate_counts(cmd->slice); } remaining_row_count -= result->row_count().value();