matvj250 opened a new pull request, #15954: URL: https://github.com/apache/pinot/pull/15954
This is a continuation of this PR: https://github.com/apache/pinot/pull/15868. We want to migrate certain API files to pinot-spi for compatibility gating. broker.api.AccessControl, server.access.AccessControl, and controller.api.access.AccessControl can all be migrated to pinot-spi by migrating their necessary files - ex. common.request.BrokerRequest - and updating the pinot-spi pom with the io.netty, javax.ws.rs, javax.annotation and thrift dependencies. In addition, I disabled the checkstyle checks for the thrift-generated files that were migrated. These files were moved to pinot-spi/auth, with specific subfolders for broker and server, controller, core, and request (originally from common.request) files. To summarize, the changes made in this PR are the migration of the files, making additions to the pinot-spi pom, updating suppressions.xml for the moved thrift-generated files, and changing the import paths in other files accordingly (using mvn spotless:apply to fix any ordering discrepancies). This is a full list of packages that were migrated: - broker.api.RequesterIdentity - common.request.BrokerRequest - common.request.DataSource - common.request.Expression - common.request.ExpressionType - common.request.Function - common.request.Identifier - common.request.JoinType - common.request.Join - common.request.Literal - common.request.PinotQuery - common.request.QuerySource - controller.api.access.AccessControl - controller.api.access.AccessType - core.auth.FineGrainedAccessControl - core.auth.TargetType - server.access.AccessControl -- 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
