search

apache / incubator-graphar

An open source, standard data file format for graph data storage and retrieval.
https://graphar.apache.org/
Apache License 2.0
192 stars 40 forks source link

[java-info] Add vertex, edge, graph info #484

Closed Thespica closed 1 month ago

Thespica commented 1 month ago

Reason for this PR

The third step in issue #478 .

What changes are included in this PR?

Are these changes tested?

no

Are there any user-facing changes?

no

SemyonSinchenko commented 1 month ago

Hi, John, you may need to add some API document in the code , that would help user/developer to know what the method use for

Is it the right time for that? Because we are in the middle of a decision about using proto: with proto info, classes should be refactored again. I would like to recommend postponing writing docs.

acezen commented 1 month ago

Hi, John, you may need to add some API document in the code , that would help user/developer to know what the method use for

Is it the right time for that? Because we are in the middle of a decision about using proto: with proto info, classes should be refactored again. I would like to recommend postponing writing docs.

Yes, you are right, considering the proto decision, postponing the docs avoids refactor again, thanks Sem