Skip to content

feat: support use sql to construct graph for loader#263

Merged
imbajin merged 5 commits intoapache:masterfrom
simon824:master
May 5, 2022
Merged

feat: support use sql to construct graph for loader#263
imbajin merged 5 commits intoapache:masterfrom
simon824:master

Conversation

@simon824
Copy link
Member

closed #262

@imbajin imbajin changed the title [Feature] Support custom sql for hugegraph-loader module feat: support use sql to construct graph for loader Apr 20, 2022
@imbajin
Copy link
Member

imbajin commented Apr 20, 2022

could u add a user case for it? (in the comment area)

@simon824
Copy link
Member Author

could u add a user case for it? (in the comment area)

  1. Vertices/edges and properties are not in the same table, need to join them.
  2. Need to apply sql function on the specified column.

@imbajin
Copy link
Member

imbajin commented Apr 21, 2022

could u add a user case for it? (in the comment area)

  1. Vertices/edges and properties are not in the same table, need to join them.
  2. Need to apply sql function on the specified column.

Thanks. Do we have a minimal sql case for it? And it's useful for user-doc later

@simon824
Copy link
Member Author

Thanks. Do we have a minimal sql case for it? And it's useful for user-doc later

select t1.name, t1.age, t2.city from t1 left join t2 on t1.id = t2.id where t2.date > '2021-01-01'

I can help to update doc later.

@imbajin imbajin requested review from javeme April 29, 2022 06:50
javeme
javeme previously approved these changes Apr 29, 2022
Copy link
Member

@imbajin imbajin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, u could add a doc later

@imbajin imbajin merged commit e0b34a7 into apache:master May 5, 2022
@javeme javeme added the loader hugegraph-loader label Jul 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

loader hugegraph-loader

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Support custom sql for hugegraph-loader module

3 participants