ankitsultana commented on issue #12676: URL: https://github.com/apache/pinot/issues/12676#issuecomment-2013508255
This looks good to me. It would be great if we can put these guidelines in the Apache Pinot Wiki as well. @gviedma : are you also planning to restructure/fix all of the existing dependencies? I am wondering if there's a way to ensure that after the restructuring there won't be any breaking changes. (e.g. if we switch to a bom dependency for grpc, it could potentially change some existing dependency which can cause a regression) -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org