From a2cdee5a97d9943cc3c74d8ebb3429a8aee658c6 Mon Sep 17 00:00:00 2001 From: Songqing Zhang Date: Tue, 30 May 2023 13:49:17 +0800 Subject: [PATCH] Improve GetInnerVertex() by querying local fragment only (#1404) The current implementation of GetInnerVertex() is to query all the fragment, it's not needed as we only need the inner vertex Signed-off-by: songqing --- modules/graph/fragment/arrow_fragment.vineyard-mod | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/modules/graph/fragment/arrow_fragment.vineyard-mod b/modules/graph/fragment/arrow_fragment.vineyard-mod index e5ff8bacb5..79aff02882 100644 --- a/modules/graph/fragment/arrow_fragment.vineyard-mod +++ b/modules/graph/fragment/arrow_fragment.vineyard-mod @@ -355,11 +355,9 @@ class [[vineyard]] ArrowFragment bool GetInnerVertex(label_id_t label, const oid_t& oid, vertex_t& v) const { vid_t gid; - if (vm_ptr_->GetGid(label, internal_oid_t(oid), gid)) { - if (vid_parser_.GetFid(gid) == fid_) { - v.SetValue(vid_parser_.GetLid(gid)); - return true; - } + if (vm_ptr_->GetGid(fid_, label, internal_oid_t(oid), gid)) { + v.SetValue(vid_parser_.GetLid(gid)); + return true; } return false; }