jerryshao opened a new pull request, #13088:
URL: https://github.com/apache/gravitino/pull/13088

   **Cherry-pick Information:**
   - Original PR: #13086 (still open; this backport is raised ahead of its 
merge, and must be kept in sync with any review changes there)
   - Original issue: #13085
   - Target branch: `branch-1.3`
   - Status: ✅ Conflicts resolved manually
   
   **Resolution:**
   - Applied the final #13086 change as one commit instead of replaying its 
individual review commits.
   - `TestGravitinoInterceptionService`: `branch-1.3` does not have the table 
existence-probe tests or their helpers, so only the new catalog connection 
tests, their helpers, and their imports are added.
   - `TestCatalogConnectionTestAuthorizationExecutor`: 
`AuthorizeExecutorFactory.create` on `branch-1.3` has no 
`allowCheckExistenceExpression` parameter, so the factory test passes nine 
arguments.
   - Production code, docs, the integration test, and the Python test are the 
same as #13086.
   
   **Validation:**
   - Server unit tests on `branch-1.3`: all 
`org.apache.gravitino.server.web.filter.*` tests (including the new interceptor 
and executor tests), `TestCatalogAuthorizationExpression`, and 
`TestCatalogOperations`.
   - Compiled the Java client test sources, including `CatalogAuthorizationIT`. 
The Docker integration test and the Python unit test were not run locally.
   - `./gradlew spotlessApply`, `:server-common:javadoc`, `:docs:build`
   - Conflict-marker scan and `git diff --check`
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   


-- 
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]

Reply via email to