[#714] feat(Java): Chunk Info Reader
This PR introduces a set of utility methods designed to efficiently read information pertaining to graph vertex chunks. These utilities will streamline the process of accessing and interpreting how vertices are grouped and stored, which is crucial for optimizing graph processing and analysis tasks.
Specifically, this PR provides:
Methods to query and retrieve metadata about vertex chunks.
Functions to read the boundaries and contents of individual vertex chunks.
Utilities to facilitate the navigation and processing of vertex data in a chunked manner.
This enhancement will benefit features requiring granular access to graph vertex data, such as distributed graph algorithms, incremental graph updates, and optimized data loading.
If
mlines of data is split intonfiles:
- First
n-1files havechunkSize(get fromVertexInfo/EdgeInfo getChunkSize()) entries each- Last file has the remainder:
m - (n-1)*chunkSizeSo we don’t need to read file record count.
How to know about "m"? the count of the data file associated with the vertex?
If
mlines of data is split intonfiles:
- First
n-1files havechunkSize(get fromVertexInfo/EdgeInfo getChunkSize()) entries each- Last file has the remainder:
m - (n-1)*chunkSizeSo we don’t need to read file record count.
How to know about "m"? the count of the data file associated with the vertex?
It can be read from vertex_count.
@yangxk1 pls provide review and merge
@yangxk1 pls allow CI to re-trigger automatically when a new code is commited by me so that I don't have to wait for you to do it manually. Thanks.
@yangxk1 pls allow CI to re-trigger automatically when a new code is commited by me so that I don't have to wait for you to do it manually. Thanks.
To ensure security, workflow execution for first-time contributors requires approval from a project committer.
You can change the branches setting in java-info.yml to trigger the workflow in your own fork, or running the script commands in java-info.yml directly in your local environment.
Just remember to revert any changes to the .yml file before submitting your final PR.
@yangxk1 thanks for merging the PR on the version parsing, pls also do the same for this PR to let us merge this. PS: Allow edits by maintainers is enabled.
@yangxk1 thanks for updating the PR on the version parsing, pls also do the same for this PR to let us merge this. PS: Allow edits by maintainers is enabled.
This PR does not use protobuf, so it is not that urgent. I will still help you as soon as possible.
@yangxk1 pls approve and merge this or let me know the changes required to merge before the next release so it can be included in the next release.
Hi @unical1988 ,I think you should open an issue to discuss the necessity of this pr.
@yangxk1 i already describe here whats the pr intended for, couldnt be clearer
You need to think about these:
If
mlines of data is split intonfiles:
- First
n-1files havechunkSize(get fromVertexInfo/EdgeInfo getChunkSize()) entries each- Last file has the remainder:
m - (n-1)*chunkSizeSo we don’t need to read file record count.
Open an Issue focus on discussion rather than description. This is also the specification of the CONTRIBUTING document.
@yangxk1 i can and will add tests anything else has been discussed here, pls note that this is implemented in C++
@yangxk1 i can and will add tests anything else has been discussed here, pls note that this is implemented in C++
@yangxk1 do you agree that this function doesn't require opening an issue ? why do you think it is not correct?
- We have not discussed whether this function is necessary.
- We don’t know whether it is implemented in
java-infoorjava-ioor other some modules.