-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Dump tool. #1479
Dump tool. #1479
Conversation
Jenkins go! |
Unit testing passed. |
Unit testing passed. |
} | ||
} | ||
|
||
std::cout << "===========================STATISTICS============================\n"; |
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 don't use LOG(INFO)
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 we need log in such a tool?
Unit testing failed. |
Jenkins go! |
Unit testing passed. |
uint32_t bitmap = | ||
(parts_.empty() ? 0 : 1 << 3) | ||
| (vids_.empty() ? 0 : 1 << 2) | ||
| (tags_.empty() ? 0 : 1 << 1) | ||
| (edges_.empty() ? 0 : 1); | ||
switch (bitmap) { | ||
case 0b0000: { | ||
// nothing specified,seek to first and print them all |
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.
Like this !
auto name = schemaMng_->toTagName(spaceId_, tagId); | ||
if (!name.ok()) { | ||
return folly::stringPrintf("%d", tagId); | ||
} else { | ||
return name.value(); | ||
} |
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.
Aha, A good idea ! Same with a film《Inception》😄
Overall, I think this PR is very good ! |
Unit testing passed. |
Unit testing passed. |
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.
Well done. great job !
close #194 |
close #194 |
Unit testing failed. |
d9fddd2
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.
Well done
Unit testing passed. |
Unit testing passed. |
* Add dumper. * Add dump. * Add statistics. * Update. * Add to_tag_name. * Rebase and update. * Address @bright-starry-sky's comment. * Address @laura-ding's comment. * Delete FLAGS_load_data_interval_secs. Co-authored-by: bright-starry-sky <56461666+bright-starry-sky@users.noreply.github.com> Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
This tool is about to help to dump the records or statistics for a space from rocksdb directly. Please take a look about the usage, any suggestion is appreciated.
Example: