Skip to content

Commit

Permalink
[core] Change log error to log info when node disconnect and got dete…
Browse files Browse the repository at this point in the history
…cted by ray syncer. (#35415)

Logging an error will be pushed to the driver and it's annoying since it's actually not an error and will be handled by ray internally.
This PR changes it to log info for usability.
  • Loading branch information
fishbone authored May 16, 2023
1 parent ea6b926 commit 7e61d0b
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/ray/common/ray_syncer/ray_syncer-inl.h
Original file line number Diff line number Diff line change
Expand Up @@ -289,8 +289,8 @@ class RaySyncerBidiReactorBase : public RaySyncerBidiReactor, public T {
if (ok) {
SendNext();
} else {
RAY_LOG_EVERY_MS(ERROR, 1000) << "Failed to send the message to: "
<< NodeID::FromBinary(GetRemoteNodeID());
RAY_LOG_EVERY_MS(INFO, 1000) << "Failed to send the message to: "
<< NodeID::FromBinary(GetRemoteNodeID());
Disconnect();
}
},
Expand All @@ -305,8 +305,8 @@ class RaySyncerBidiReactorBase : public RaySyncerBidiReactor, public T {
ReceiveUpdate(std::move(msg));
StartPull();
} else {
RAY_LOG_EVERY_MS(ERROR, 1000) << "Failed to read the message from: "
<< NodeID::FromBinary(GetRemoteNodeID());
RAY_LOG_EVERY_MS(INFO, 1000) << "Failed to read the message from: "
<< NodeID::FromBinary(GetRemoteNodeID());
Disconnect();
}
},
Expand Down

0 comments on commit 7e61d0b

Please sign in to comment.