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 <[email protected]>
v6d-io · May 30, 2023 · a2cdee5 · a2cdee5
1 parent 19616c8
commit a2cdee5
Showing 1 changed file with 3 additions and 5 deletions.
diff --git 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;
  }