Skip to content

Commit

Permalink
Lint
Browse files Browse the repository at this point in the history
  • Loading branch information
raararaara committed Jul 2, 2024
1 parent 42ca2ba commit a8849b3
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion server/clients/clients.go
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,8 @@ func Deactivate(
// by the servers in the cluster. So, we need to consider the case where the leader is
// not the same as the server that handles the document.

// When deactivating a client, we need to update three DB properties (ClientInfo.Status, ClientInfo.Documents, SyncedSeq) in DB.
// When deactivating a client, we need to update three DB properties
// (ClientInfo.Status, ClientInfo.Documents, SyncedSeq) in DB.
// Updating the sub-properties of ClientInfo guarantees atomicity as it involves a single MongoDB document.
// However, SyncedSeqs are stored in separate documents, so we can't ensure atomic updates for both.
// Currently, if SyncedSeqs update fails, it mainly impacts GC efficiency without causing major issues.
Expand Down

0 comments on commit a8849b3

Please sign in to comment.