Skip to content

codeintel: removing bloom filters

Warren Gifford requested to merge cesar/34862/remove-bloom-filter into main

Created by: Numbers88s

This PR closes #34862 (closed) with the following:

  • Removed the uses of DEBUG_PRECISE_CODE_INTEL_BLOOM_FILTER_BAIL_OUT
  • Simplified the resulting code and eliminated any paths that were now unreachable.
  • Removed the lib/codeintel/bloomfilter package
  • Removed the filter column on the lsif_references table

Test plan

#32908 disabled the bloom filters from the query path. We haven't seen a performance degradation in two months so we can assume it's safe to delete entirely.

Merge request reports