citusdata / citusdata/citus

Remove unused headers

Open
#4,558 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C
Stars
12.8k
Forks
794
Avg merge
2d 14h
Merged PRs (30d)
31

Description

We might have around 2000 unused headers in our codebase, this might increase compile time unnecessarily. We should consider removing this.

One idea is generally commenting out an include line to see if the code still compiles but this should be done for pg11, pg12 and pg13.

For example I quickly tried something like this for pg13 and it worked but should consider pg11 and pg12 as well. https://github.com/citusdata/citus/tree/remove/unusedHeaders

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.