Re: [I] feat(java, info): Unit test for loading graph [incubator-graphar]

2025-05-20 Thread via GitHub
yangxk1 commented on issue #676: URL: https://github.com/apache/incubator-graphar/issues/676#issuecomment-2893360061 > > I noticed that some interfaces in the Java code are not fully aligned with those in the C++ code. For example, the [Java code](https://github.com/apache/incubator-grapha

Re: [I] feat(java, info): Unit test for loading graph [incubator-graphar]

2025-05-20 Thread via GitHub
yangxk1 commented on issue #676: URL: https://github.com/apache/incubator-graphar/issues/676#issuecomment-2893375821 While writing unit tests, I found three issues: 1. Extra `/` in path concatenation(e.g. `"vertex/person//id/"`) (minor). 2. Missing `getVersion()` in GraphInfo, Vert

[PR] feat(java-info): add java unit test for loading graph (#676) [incubator-graphar]

2025-05-20 Thread via GitHub
yangxk1 opened a new pull request, #683: URL: https://github.com/apache/incubator-graphar/pull/683 ### Reason for this PR As described in issue #676 , there is currently no unit test to verify loading graph in java-info module. This PR adds unit tests for the Java info module to ensu