kishoreg edited a comment on pull request #7442: URL: https://github.com/apache/pinot/pull/7442#issuecomment-934909659
@amrishlal While we can continue to discuss further if this rule is good or bad, I don't think it's worth our time or your time. @richardstartin did not introduce this rule. We already have this as part of the coding convention in Pinot. What he is trying to do is codify that in check style so that the build catches it instead of relying on the reviewer to catch it. If you really want to change the convention that is used since the start of the project, I encourage you to create another issue or discussion thread. Let's move on. We have so many other things to fix/improve in Pinot. -- 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]
