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-graphar/blob/main/maven-projects/java/src/main/java/org/apache/graphar/graphinfo/VertexInfo.java) is missing the `vertexInfo::getPropertyGroupByIndex` interface, which is available in the [C++ code](https://github.com/apache/incubator-graphar/blob/main/cpp/src/graphar/graph_info.h) . Also, the current Java project depends on version `v0.10.0` of the C++ code, which is quite outdated. Should we consider updating the Java project first? > > Hi [@yangxk1](https://github.com/yangxk1) , maybe there are some ambiguous expression. The `java-info` here refer to the [info](https://github.com/apache/incubator-graphar/tree/main/maven-projects/info) packages, rather than the [original java](https://github.com/apache/incubator-graphar/tree/main/maven-projects/java) package. You can reach out to us if there is any other problem : ) Thanks for your reply! I've opened a PR to improve the [GraphLoaderTest#testLoad](https://github.com/apache/incubator-graphar/blob/main/maven-projects/info/src/test/java/org/apache/graphar/info/GraphLoaderTest.java) method in java-info. -- 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: commits-unsubscr...@graphar.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@graphar.apache.org For additional commands, e-mail: commits-h...@graphar.apache.org
