Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9717 )
Change subject: fs: clarify failures on non-disk-failures ...................................................................... fs: clarify failures on non-disk-failures Currently non-handled failures (e.g. disk space errors) of MRS/DMS flush failures crash with errors similar to the following: F0316 14:36:07.240557 131158 tablet_replica_mm_ops.cc:145] Check failed: tablet->HasBeenStopped() FlushMRS failure is only allowed if the tablet is stopped first This message is not clear to users, who have no concept of "stopping" a tablet. Nor is it clear for developers, as it may read as pointing to a bug. These messages have been updated to be more user-friendly: Unrecoverable flush failure caused by error: <error> Change-Id: I062bbb69847f7da5d381281a6c24b806bafc2620 Reviewed-on: http://gerrit.cloudera.org:8080/9717 Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon <[email protected]> --- M src/kudu/tablet/tablet_replica_mm_ops.cc 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Kudu Jenkins: Verified Todd Lipcon: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/9717 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I062bbb69847f7da5d381281a6c24b806bafc2620 Gerrit-Change-Number: 9717 Gerrit-PatchSet: 4 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
