keksmd opened a new pull request, #956: URL: https://github.com/apache/incubator-graphar/pull/956
### Reason for this PR Implements the focused Java metadata compatibility slice tracked by #955. It exposes the indexed lookup and immutable property-group removal operations that match the existing C++ metadata surface, without adding physical IO or changing GraphAr layout resolution. ### What changes are included in this PR? - Adds index-based access to graph vertex/edge metadata and to vertex/edge property groups; invalid indexes return `null`. - Adds immutable property-group removal on vertex and edge metadata; the source object remains unchanged. - Covers successful, out-of-range, absent, and immutable-removal cases. ### Are these changes tested? Yes. On current `upstream/main` with `GAR_TEST_DATA=testing`: - `mvn --no-transfer-progress -P!nexus-download -pl info -am spotless:check` - `mvn --no-transfer-progress -P!nexus-download -pl info -am clean verify -Dspotless.check.skip=true` — 125 tests, 0 failures/errors/skips. - `pre-commit run --files` for the seven changed Java files. ### Are there any user-facing changes? Yes. This adds Java metadata API methods only; no wire-format or storage behaviour changes. ## Checklist - [x] I have performed a self-review of my own code. - [x] I have formatted my own code using `make cpplint` before submitting when changed files are in the `cpp` directory. - [x] I have performed `pre-commit run` before commit the changed files. - [x] I have added tests to prove my changes are effective. -- 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]
