Skip to content

Commit

Permalink
Placement group bug fix (#17320)
Browse files Browse the repository at this point in the history
  • Loading branch information
clay4megtr authored Jul 27, 2021
1 parent 489febc commit 684e2b2
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/ray/gcs/gcs_server/gcs_placement_group_manager.cc
Original file line number Diff line number Diff line change
Expand Up @@ -249,13 +249,14 @@ void GcsPlacementGroupManager::OnPlacementGroupCreationSuccess(
RAY_LOG(INFO) << "Successfully created placement group " << placement_group->GetName()
<< ", id: " << placement_group->GetPlacementGroupID();
placement_group->UpdateState(rpc::PlacementGroupTableData::CREATED);
// Mark the scheduling done firstly.
MarkSchedulingDone();
auto placement_group_id = placement_group->GetPlacementGroupID();
RAY_CHECK_OK(gcs_table_storage_->PlacementGroupTable().Put(
placement_group_id, placement_group->GetPlacementGroupTableData(),
[this, placement_group_id](Status status) {
RAY_CHECK_OK(status);

MarkSchedulingDone();
SchedulePendingPlacementGroups();

// Invoke all callbacks for all `WaitPlacementGroupUntilReady` requests of this
Expand Down

0 comments on commit 684e2b2

Please sign in to comment.