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.

Sign up for GitHub

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

Jump to bottom

build docker image of db_dump and other tools #2454

Merged
merged 2 commits into from
Mar 16, 2021
Merged

build docker image of db_dump and other tools #2454

merged 2 commits into from
Mar 16, 2021

Conversation

jievince
Copy link
Contributor

@jievince jievince commented Mar 5, 2021 •
edited
Loading

nebula-tools image includes:

  • storage_perf
  • storage_integrity
  • simple_kv_verify
  • db_dump

@jievince jievince added the ready-for-testing PR: ready for the CI test label Mar 5, 2021
@jievince jievince requested review from yixinglu and a team March 5, 2021 05:14
@jievince jievince requested a review from dutor March 5, 2021 05:39
@Shylock-Hg
Copy link
Contributor

nebula-tools image includes:

  • storage_perf
  • storage_integrity
  • simple_kv_verify
  • db_dump

This tools should put into storage images, or how can user use it?

@jievince
Copy link
Contributor Author

jievince commented Mar 9, 2021 •
edited
Loading

nebula-tools image includes:

  • storage_perf
  • storage_integrity
  • simple_kv_verify
  • db_dump

This tools should put into storage images, or how can user use it?

Can have two images share the same namespace or volumn directory

@yixinglu yixinglu merged commit e89a428 into vesoft-inc:master Mar 16, 2021
@jievince jievince deleted the tools branch March 17, 2021 03:35
tong-hao pushed a commit to tong-hao/nebula that referenced this pull request Jun 1, 2021
@jievince
build nebula-tools docker image (vesoft-inc#2454)
283fb2c
yixinglu pushed a commit to yixinglu/nebula that referenced this pull request Sep 14, 2023
@nebula-bots @czpmango
Support match from id Set and fix some related issues (vesoft-inc#2454)
44f0f40 
* Fix match by ids

small delete

* Delete some code comments

* fix ut

Co-authored-by: kyle.cao <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@laura-ding laura-ding laura-ding approved these changes

@yixinglu yixinglu yixinglu approved these changes

@dutor dutor Awaiting requested review from dutor

Assignees
No one assigned
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jievince @Shylock-Hg @yixinglu @laura-ding