abdullah alamoudi has posted comments on this change. Change subject: [NO ISSUE][ING] Fix race between active recovery and rebalance ......................................................................
Patch Set 1: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/2950/1/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/active/RecoveryTask.java File asterixdb/asterix-app/src/main/java/org/apache/asterix/app/active/RecoveryTask.java: PS1, Line 113: return null; > Should we also log here as we do in the case of temporary failure? Or is th For supportability, we should log every place we cancel to know why recovery didn't complete. will create a new patch -- To view, visit https://asterix-gerrit.ics.uci.edu/2950 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I66edb73950bb82baa1a1dfd892cb4b23bb7046be Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: abdullah alamoudi <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-Reviewer: abdullah alamoudi <[email protected]> Gerrit-HasComments: Yes
