Something went wrong on our end. Please try again.
Created by: rvantonder
Going to pick textsearch to rename unindexed to (not super attached), but point is to represent shared responsibility for calling out to zoekt/indexed search and searcher/unindexed. After this rename I'm going to factor out searcher-only logic into its own package.
The state will then be:
internal/search/textsearch/textsearch.gointernal/search/zoekt/indexed_search.go,...internal/search/searcher/...internal/search/structural/...Feel free to share thoughts on this structure. One thing that sticks out is that we can have
internal/search/indexed and
internal/search/unindexed
instead of
internal/search/zoekt and
internal/search/searcher
I think I prefer the zoekt,searcher pair which makes more sense if we ever want to add other kinds of indexed or unindexed services/backends. This why I'm leaning towards internal/search/searcher package rather than keep unindexed terminology around.
Semantics preserving, covered by unit tests.