Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(interactive): support multiple properties on edge #3962

Merged
merged 15 commits into from
Jun 26, 2024

Conversation

liulx20
Copy link
Collaborator

@liulx20 liulx20 commented Jun 19, 2024

  • To support multiple properties on edge, we introduce new type Record and RecordView here. Record is associated with a property array through a pointer. RecordView consists of an offset and a pointer to a Table.

  • Then we specialized a new csr type MutableCsr<RecordView> as multi-properties edge storage. MutableCsr<RecordView> contains a Table through which we organize multi-properties. When get_data() is called, a RecordView Object will be returned, we can get properties with the RecordView, and when it is necessary to insert edges into the graph, we need to encapsulate the multi-attributes into a Record.

@liulx20 liulx20 marked this pull request as draft June 19, 2024 11:53
@liulx20 liulx20 changed the title feature(interactive): support multiple property on edge feature(interactive): support multiple properties on edge Jun 20, 2024
@liulx20 liulx20 changed the title feature(interactive): support multiple properties on edge feat(interactive): support multiple properties on edge Jun 20, 2024
@liulx20 liulx20 force-pushed the multip branch 3 times, most recently from e2f2dd7 to ec03f24 Compare June 20, 2024 08:22
@liulx20 liulx20 force-pushed the multip branch 3 times, most recently from ac1729b to 9484a81 Compare June 23, 2024 02:13
@liulx20 liulx20 force-pushed the multip branch 4 times, most recently from f63f61f to 11a6731 Compare June 26, 2024 03:19
@liulx20 liulx20 marked this pull request as ready for review June 26, 2024 03:20
@liulx20 liulx20 requested review from luoxiaojian and zhanglei1949 and removed request for luoxiaojian June 26, 2024 03:20
@liulx20 liulx20 requested review from luoxiaojian and zhanglei1949 and removed request for zhanglei1949 June 26, 2024 03:20
@liulx20 liulx20 merged commit def4096 into alibaba:main Jun 26, 2024
38 of 40 checks passed
@liulx20 liulx20 self-assigned this Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants