From cca4e97bcff287764d8d9646c6f7773f4a9b0c48 Mon Sep 17 00:00:00 2001 From: jeremy Date: Tue, 15 Nov 2022 13:37:43 +0100 Subject: [PATCH] Remove un-needed argument from OrderByEdges --- src/storage/v3/request_helper.hpp | 2 +- src/storage/v3/shard_rsm.cpp | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/storage/v3/request_helper.hpp b/src/storage/v3/request_helper.hpp index 955059f26..4d3018566 100644 --- a/src/storage/v3/request_helper.hpp +++ b/src/storage/v3/request_helper.hpp @@ -150,7 +150,7 @@ std::vector> OrderByVertices(Shard::Accessor &acc, DbAcc } template -std::vector> OrderByEdges(Shard::Accessor &acc, DbAccessor &dba, TIterable &iterable, +std::vector> OrderByEdges(DbAccessor &dba, TIterable &iterable, std::vector &order_bys, const VertexAccessor &vertex_acc) { static_assert(std::is_same_v>); // Can be extended if needed diff --git a/src/storage/v3/shard_rsm.cpp b/src/storage/v3/shard_rsm.cpp index 55b726c79..2acdf9f3c 100644 --- a/src/storage/v3/shard_rsm.cpp +++ b/src/storage/v3/shard_rsm.cpp @@ -987,8 +987,8 @@ msgs::ReadResponses ShardRsm::HandleRead(msgs::ExpandOneRequest &&req) { } else { auto [in_edge_accessors, out_edge_accessors] = GetEdgesFromVertex(src_vertex_acc, req.direction); - const auto in_ordered_edges = OrderByEdges(acc, dba, in_edge_accessors, req.order_by, src_vertex_acc); - const auto out_ordered_edges = OrderByEdges(acc, dba, out_edge_accessors, req.order_by, src_vertex_acc); + const auto in_ordered_edges = OrderByEdges(dba, in_edge_accessors, req.order_by, src_vertex_acc); + const auto out_ordered_edges = OrderByEdges(dba, out_edge_accessors, req.order_by, src_vertex_acc); std::vector in_edge_ordered_accessors; std::transform(in_ordered_edges.begin(), in_ordered_edges.end(), std::back_inserter(in_edge_ordered_accessors),