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

searcher: make indexing of repos concurrent #272

Merged
merged 6 commits into from
Jan 18, 2018
Merged

searcher: make indexing of repos concurrent #272

merged 6 commits into from
Jan 18, 2018

Commits on Dec 30, 2017

  1. searcher: make indexing of repos concurrent 

    This patch attempts to improve the startup time of hound vastly in cases
when we have huge number of repositories and hound would generally take
long time to start because of its sequential nature of indexing.

Now, we have the startup indexing in a concurrent way while respecting
the config.MaxConcurrentIndexers parameter set by users in config.json.

Fixes hound-search#250
    @sahildua2305
    sahildua2305 committed Dec 30, 2017
    Configuration menu
    Copy the full SHA
    74815e9 View commit details
    Browse the repository at this point in the history

  2. searcher: modify concurrent routine 

    Change the concurrent searchers init routine to simplify the
implementation and use only one common channel to return successfully
created searchers as well as errors. This way we also get rid of the go
routine collecting the results on the channel by making the main routine
collect the results by blocking the execution and listening on the
channel.
    @sahildua2305
    sahildua2305 committed Dec 30, 2017
    Configuration menu
    Copy the full SHA
    8d51b7d View commit details
    Browse the repository at this point in the history

  3. searcher: make results channel unbuffered

    @sahildua2305
    sahildua2305 committed Dec 30, 2017
    Configuration menu
    Copy the full SHA
    afb8e5d View commit details
    Browse the repository at this point in the history

  4. searcher: remove explicit assignment of nil fields

    @sahildua2305
    sahildua2305 committed Dec 30, 2017
    Configuration menu
    Copy the full SHA
    a6aaac0 View commit details
    Browse the repository at this point in the history

  5. searcher: fix warning in golang 1.3 

    `for range cfg.Repos` throws syntax error in golang 1.3. Hence using
`len(cfg.Repos)` instead to iterate.
    @sahildua2305
    sahildua2305 committed Dec 30, 2017
    Configuration menu
    Copy the full SHA
    e74b892 View commit details
    Browse the repository at this point in the history

  6. searcher: make result channel buffered

    @sahildua2305
    sahildua2305 committed Dec 30, 2017
    Configuration menu
    Copy the full SHA
    302e597 View commit details
    Browse the repository at this point in the history