slfan1989 commented on PR #7016: URL: https://github.com/apache/ozone/pull/7016#issuecomment-2315689691
> Seems some related ci failures, such as `org.apache.hadoop.ipc.RemoteException(com.google.protobuf.UninitializedMessageException): Message missing required fields: scmHaEnabled`. > > @slfan1989 From my point, maybe we don't need to make so many changes. How about simplifying the codes similar to the `—json` ? cc @adoroszlai @ChenSammi @whbing Thank you for your question. I believe using a clear method to indicate whether SCM is using Ratis would be better. While a JSON output approach is also feasible, determining whether Ratis is used based on the size of the length might not be very reliable. -- 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: issues-unsubscr...@ozone.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@ozone.apache.org For additional commands, e-mail: issues-h...@ozone.apache.org
