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

Support Ingest SST Files #654

Merged
merged 22 commits into from
Aug 7, 2019
Merged

Support Ingest SST Files #654

merged 22 commits into from
Aug 7, 2019

Conversation

darionyaphet
Copy link
Contributor

@darionyaphet darionyaphet commented Jul 18, 2019

In this PR, you can ingest the SST files after download from HDFS.

INGEST command can be used in console.

wadeliuyi
wadeliuyi previously approved these changes Jul 19, 2019
Copy link
Contributor

@wadeliuyi wadeliuyi left a comment

Choose a reason for hiding this comment

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

LGTM.

wadeliuyi
wadeliuyi previously approved these changes Jul 23, 2019
Copy link
Contributor

@wadeliuyi wadeliuyi left a comment

Choose a reason for hiding this comment

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

LGTM

critical27
critical27 previously approved these changes Jul 23, 2019
Copy link
Contributor

@critical27 critical27 left a comment

Choose a reason for hiding this comment

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

LGTM

@darionyaphet darionyaphet dismissed stale reviews from critical27 and wadeliuyi via 623e80b July 25, 2019 15:26
Copy link
Contributor

@dangleptr dangleptr left a comment

Choose a reason for hiding this comment

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

Totally, the pr looks good to me now.

@nebula-community-bot
Copy link
Member

Unit testing passed.

@nebula-community-bot
Copy link
Member

Unit testing passed.

1 similar comment
@nebula-community-bot
Copy link
Member

Unit testing passed.

@nebula-community-bot
Copy link
Member

Unit testing passed.

dangleptr
dangleptr previously approved these changes Aug 6, 2019
Copy link
Contributor

@dangleptr dangleptr left a comment

Choose a reason for hiding this comment

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

Well done. Thanks for taking care of it.

@nebula-community-bot
Copy link
Member

Unit testing passed.

critical27
critical27 previously approved these changes Aug 7, 2019
Copy link
Contributor

@critical27 critical27 left a comment

Choose a reason for hiding this comment

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

LGTM

@darionyaphet darionyaphet dismissed stale reviews from critical27 and dangleptr via 3e8c18a August 7, 2019 03:40
@nebula-community-bot
Copy link
Member

Unit testing passed.

Copy link
Contributor

@dangleptr dangleptr left a comment

Choose a reason for hiding this comment

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

Well done

@jude-zhu jude-zhu requested a review from wadeliuyi August 7, 2019 09:34
@dangleptr dangleptr merged commit a9e6bf6 into vesoft-inc:master Aug 7, 2019
@darionyaphet darionyaphet deleted the ingest branch August 7, 2019 09:45
yixinglu pushed a commit to yixinglu/nebula that referenced this pull request Feb 16, 2020
It is used to ingest sst files directly after download files from hdfs.
tong-hao pushed a commit to tong-hao/nebula that referenced this pull request Jun 1, 2021
It is used to ingest sst files directly after download files from hdfs.
yixinglu added a commit to yixinglu/nebula that referenced this pull request Mar 21, 2022
* Fix iterator with snapshot(parameter position) (vesoft-inc#3785)

Co-authored-by: Harris.Chu <[email protected]>
Co-authored-by: Sophie <[email protected]>

* add getNeighborIter size

* refactor subgraph

* fix error

* address comment

* remove project from subgraph plan

* add comment

* remove project

* fix error

Co-authored-by: Alex Xing <[email protected]>
Co-authored-by: Harris.Chu <[email protected]>
Co-authored-by: Sophie <[email protected]>
Co-authored-by: Yee <[email protected]>

Co-authored-by: jimingquan <[email protected]>
Co-authored-by: Alex Xing <[email protected]>
Co-authored-by: Harris.Chu <[email protected]>
Co-authored-by: Sophie <[email protected]>
Co-authored-by: Yee <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants