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

Optimize Structure And Format #50

Merged
merged 3 commits into from
Apr 23, 2019
Merged

Optimize Structure And Format #50

merged 3 commits into from
Apr 23, 2019

Conversation

solotzg
Copy link
Contributor

@solotzg solotzg commented Apr 22, 2019

By now, the way we add head files is very casual. We should try to avoid including a head file in another head file unless we need to define the memory structure. This PR fix most of unreasonable behaviors.

@solotzg solotzg requested a review from innerr April 22, 2019 10:41
@solotzg
Copy link
Contributor Author

solotzg commented Apr 22, 2019

/rebuild

@solotzg
Copy link
Contributor Author

solotzg commented Apr 22, 2019

/run-integration-tests

Copy link
Contributor

@innerr innerr left a comment

Choose a reason for hiding this comment

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

LGTM


// Remove region's data from partition
// Usage:
// ./storage-client.sh "DBGInvoke rm_region_data(region_id)"
void dbgFuncRegionRmData(Context & context, const ASTs & args, DBGInvoker::Printer output);

void dbgFuncDumpAllRegion(Context& context, const ASTs& args, DBGInvoker::Printer output);
void dbgFuncDumpAllRegion(Context & context, const ASTs & args, DBGInvoker::Printer output);
Copy link
Contributor

Choose a reason for hiding this comment

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

We should use the CH-style Context & context

@solotzg solotzg merged commit 7f84c5f into pingcap:master Apr 23, 2019
@solotzg solotzg deleted the optimize-format branch April 23, 2019 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants