-
Notifications
You must be signed in to change notification settings - Fork 93
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
Add Gerrit Integration #172
Comments
Sounds good. I will be looking at GitLab integration for the next release so no reason I shouldn't have a look at this as well and see what would be involved. |
In that context you might be also interested in how Zoekt added Gerrit ACL support (pages 22 and 23 in the slides). |
Maybe you want to add e.g. the repo at https://gerrit-review.googlesource.com/admin/repos/gerrit to demo.searchcodeserver.com, too? |
BTW, if you want a performance comparison against Zoekt, you could probably take a look at https://cs.bazel.build/ (which indexes quite a lot of repos besides Bazel). |
Interesting. Looks like its based on the trigram search from Russ that google code search used. I wonder what sort of server that's running on. The demo instance of searchcode runs on a $2.50 a month Vultr instance which is probably a bit mean of me to be honest. |
For reference, proper Gerrit integration also requires #122 to be implemented. |
cs.bazel.build runs on a 16 core machine with 60G RAM and attached SSD. It indexes about 60G source code, but SSD should be enough for ~90G. Cost of the machine is about $400/month |
Very similar to #117, Gerrit "does not support any code search so it would be a good idea to plug the gap".
Maybe this is something @lucamilanesio would also be interested in (and be able to help with)?
The text was updated successfully, but these errors were encountered: