keksmd opened a new pull request, #968: URL: https://github.com/apache/incubator-graphar/pull/968
### Reason for this PR Implements the first, value-only slice of #967 (and relates to #947/#944): common long-safe GraphAr chunk and half-open edge-range arithmetic. ### What changes are included in this PR? - Adds the dependency-free `graphar-core` Maven module. - Adds `ChunkMath`, `ChunkRange`, and `EdgeRange` with explicit non-negative, positive-size, and monotonic-offset invariants. - Covers empty ranges and the end-exclusive cross-chunk boundary: `[1008, 1061)` at chunk size `1024` intersects chunks `[0, 2)`. This intentionally does not resolve `EdgeInfo` URIs, read offset files, or add Parquet/storage dependencies; those belong in the following metadata/reader slice. ### Are these changes tested? Yes. ```shell mvn --no-transfer-progress -f maven-projects/pom.xml -pl core -am clean verify -Dspotless.check.skip=true mvn --no-transfer-progress -f maven-projects/pom.xml -pl core -am spotless:check ``` The focused reactor passes 6 tests. `pre-commit run --files ...` and `git diff --check` also pass. ### Are there any user-facing changes? Yes. Java consumers can now use the new `graphar-core` value types; there is no reader or storage behavior change yet. ## Checklist - [x] I have self-reviewed this PR - [ ] I have run `make cpplint` (if this PR contains C++ changes) - [x] I have run `pre-commit` - [x] I have run the relevant tests -- 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]
