akg268 commented on issue #12055: URL: https://github.com/apache/pinot/issues/12055#issuecomment-5198772471
Hi Pinot maintainers, I'd like to work on this issue. My tentative approach is to add a typed reason-code representation for the server-to-controller segment completion requests, wire it through SegmentCompletionProtocol and the server/controller handling paths, and keep backward compatibility by tolerating unknown or missing reason codes from older servers while preserving the existing reason string behavior where needed. Could this be assigned to me? Also, does this approach seem aligned with what you had in mind for this cleanup? -- 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]
