Skip to content

insights: remove graphql clients from backend

Warren Gifford requested to merge insights/graphql-option into main

Created by: leonore

closes #37968 (closed). wehee!

big PR so points to note:

  • site config option has been marked as deprecated
  • removed some delegate functions in the worker
  • pulled some common logic into the worker Handle functions (erroring early on JIT, persisting recordings - means adding any future paths we don't forget anything)
  • remove aaaallll (read: most) the graphql files in our executors! wahey! | Screenshot 2022-07-07 at 16 53 34 | Screenshot 2022-07-07 at 16 53 14 |

follow-up issues:

https://github.com/sourcegraph/sourcegraph/issues/38471 https://github.com/sourcegraph/sourcegraph/issues/38472

Test plan

Ran existing unit tests. Built sg start enterprise-codeinsights Created queries/monitored things on local dev instance, but everything that's been removed is stuff that shouldn't be hit anymore. Screenshot 2022-07-08 at 13 21 13

Merge request reports

Loading