sadanand48 commented on PR #5541: URL: https://github.com/apache/ozone/pull/5541#issuecomment-1814639382
@nandakumar131 , This patch doesn't modify the pipelineInfo/pipelineID that gets printed as part of container info command, but rather adds a new field that stores the RatisPipelineID that the container was initially created on during OPEN/CLOSING states. This can be helpful in debugging issues using the ratisPipelineID say if one has to look into the logs to understand the lifecycle of that pipeline. Using only ephemeral pipeline here is confusing as it is newly generated everytime for a read and one will assume that the pipeline never got created if logs are searched using it. Can we still move forward with this patch as a debugging improvement? -- 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]
