From e7fbcd6c9d5b0fd1d6c52ac4b7fa3dd896c770cf Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Radwa=C5=84ski?= Date: Tue, 20 Dec 2022 17:03:18 +0100 Subject: [PATCH] mutation_partition_view: treat query::partition_slice::option::reversed in to_data_query_result as consume_in_reverse::yes The consume_in_reverse::legacy_half_reverse format is soon to be phased out. This commit starts treating frozen_mutations from replicas for reversed queries so that they are consumed with consume_in_reverse::yes. --- mutation_partition.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mutation_partition.cc b/mutation_partition.cc index 684ffb468a..36967baf58 100644 --- a/mutation_partition.cc +++ b/mutation_partition.cc @@ -2209,7 +2209,7 @@ to_data_query_result(const reconcilable_result& r, schema_ptr s, const query::pa auto consumer = compact_for_query_v2(*s, gc_clock::time_point::min(), slice, max_rows, max_partitions, query_result_builder(*s, builder)); auto compaction_state = consumer.get_state(); - const auto reverse = slice.options.contains(query::partition_slice::option::reversed) ? consume_in_reverse::legacy_half_reverse : consume_in_reverse::no; + const auto reverse = slice.options.contains(query::partition_slice::option::reversed) ? consume_in_reverse::yes : consume_in_reverse::no; // FIXME: frozen_mutation::consume supports only forward consumers if (reverse == consume_in_reverse::no) {