-
Notifications
You must be signed in to change notification settings - Fork 121
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 incremental loading data for 'vineyard-graph-loader' #1563
Conversation
✅ Deploy Preview for v6d ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
e103c02
to
bc6471d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your efforts for addressed this tough task!
Overall looks good, and some implementation details need to be discussed.
And remember to use clang-format to format your codes before pushing, you could find the way to use it in the CI yamls. |
f47c218
to
0acbcb4
Compare
@@ -13,6 +13,7 @@ See the License for the specific language governing permissions and | |||
limitations under the License. | |||
*/ | |||
|
|||
#include <_types/_uint32_t.h> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#include <_types/_uint32_t.h> |
@@ -13,6 +13,7 @@ See the License for the specific language governing permissions and | |||
limitations under the License. | |||
*/ | |||
|
|||
#include <_types/_uint64_t.h> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not delete these headers?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. @SighingSnow could you please revert the etcd-cpp-apiv3 submodule changes?
Signed-off-by: SighingSnow <1263750383@qq.com>
05821c5
to
7559a79
Compare
What do these changes do?
Add support for incremental vertices and edges for vineyard-graph-loader.
For step_by_step, the expected behaviors are as below
For whole or the default way, the expected behaviors are as below,
Related issue number
FIXES #1295
@siyuan0322 would you mind helping me review of the code