mchades opened a new pull request, #10988:
URL: https://github.com/apache/gravitino/pull/10988
### What changes were proposed in this pull request?
This PR adds view CRUD support in `client-java` `RelationalCatalog` and
includes a follow-up hardening patch:
- implement `ViewCatalog` operations: `listViews`, `loadView`, `createView`,
`alterView`, and `dropView`
- convert view representations/columns between API objects and DTOs
- add explicit full-namespace validation for table/view request path
formatting
- rename `getTableFullNamespace` to `getEntityFullNamespace` for clearer
shared semantics
- add/extend unit tests for view happy paths and edge cases (null
representations, invalid namespace)
### Why are the changes needed?
Java client needs first-class view management APIs for relational catalogs
as part of the generic view support epic. The hardening changes also make
invalid namespace inputs fail fast with clear errors.
Fix: #10987
### Does this PR introduce _any_ user-facing change?
Yes. `client-java` relational catalog now supports view CRUD operations
through `ViewCatalog` APIs.
### How was this patch tested?
- Added and updated unit tests in `TestRelationalCatalogView`
- Ran:
- `./gradlew :clients:client-java:test --tests
org.apache.gravitino.client.TestRelationalCatalogView --no-daemon`
- `./gradlew :clients:client-java:test --tests
org.apache.gravitino.client.TestRelationalCatalog --tests
org.apache.gravitino.client.TestRelationalCatalogView --no-daemon`
--
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]
