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
Added a script to import git repository to ClickHouse #14471
Conversation
59f909f
to
abe836a
Compare
Now the tool can easily process ClickHouse, LLVM, Linux and Chromium repositories. ClickHouse repository preprocessing takes 31 seconds. (I picked them as an examples of somewhat large repositories. What is the largest repository on GitHub?). The time of processing is in fact comparable to what time takes to clone the repository. |
|
Infrastructure failure. |
programs/git-import/git-import.cpp
Outdated
*/ | ||
void setLineInfo(std::string full_line) | ||
{ | ||
indent = 0; |
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.
It is to easy to overflow.
Probably, we may calculate using uint32_t
and cut up to 255 if overflowed.
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.
Ok.
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Added a script to import git repository to ClickHouse.
Detailed description / Documentation draft:
Better implementation of #12577
Allows to answer questions like: