sourcegraph / sourcegraph-public-snapshot

Code AI platform with Code Search & Cody
https://sourcegraph.com
Other
10.11k stars 1.29k forks source link

insights: convert compute group by insight to use new insights aggregation logic #39943

Open chwarwick opened 2 years ago

chwarwick commented 2 years ago

Since there is a planned path from search aggregations to adding it as an insight on a dashboard the compute group by insight will need to use the new aggregation logic defined in https://github.com/sourcegraph/sourcegraph/issues/40207. This is because the current compute command used does not support all search query types.

depends on https://github.com/sourcegraph/sourcegraph/issues/40207

/cc @joelkw @felixfbecker @vovakulikov @unclejustin

coury-clark commented 2 years ago

Since chunk matches are not yet available in the public API we will need to migrate to using the internal Search Client for this feature