peterxcli commented on code in PR #11086:
URL: https://github.com/apache/ozone/pull/11086#discussion_r4477549936
##########
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OzoneManager.java:
##########
@@ -4768,20 +4770,17 @@ TermIndex installCheckpoint(String leaderId, Path
checkpointLocation,
// Pause the State Machine. It flushes the transactions already applied
and rejects new ones until unpause.
omRatisServer.getOmStateMachine().pause();
} catch (Exception e) {
- LOG.error("Failed to stop/ pause the services. Cannot proceed with " +
- "installing the new checkpoint.");
+ LOG.error("Failed to stop/pause the services. Cannot proceed with
installing the new checkpoint.", e);
Review Comment:
I think we can remove this actually, as
https://github.com/apache/ozone/pull/11086/changes#diff-54bbfc5b5dd89a4d035bee206c53d3d11bbb579a2957abca589e313283d98af9L4705
also do `LOG.error(`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
