Skip to content

Commit

Permalink
fix(analytical): trim the physical memory usage when unloading graphs (
Browse files Browse the repository at this point in the history
…#3548)

Signed-off-by: Tao He <[email protected]>
  • Loading branch information
sighingnow authored Feb 24, 2024
1 parent e6d942f commit 76ac755
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions analytical_engine/core/grape_instance.cc
Original file line number Diff line number Diff line change
Expand Up @@ -197,6 +197,10 @@ bl::result<void> GrapeInstance::unloadGraph(const rpc::GSParams& params) {
// ensure all fragments get deleted
MPI_Barrier(comm_spec_.comm());
VINEYARD_SUPPRESS(client_->DelData(frag_id, false, true));
// trim vineyardd's shared memory
bool trimmed = false;
MPI_Barrier(comm_spec_.comm());
VINEYARD_SUPPRESS(client_->MemoryTrim(trimmed));
}
}
VLOG(1) << "Unloading Graph " << graph_name;
Expand Down

0 comments on commit 76ac755

Please sign in to comment.