Re: Refactoring the checkpointer's fsync request queue

On Tue, Mar 05, 2019 at 11:53:16AM -0500, Tom Lane wrote:
> Thomas Munro <thomas(dot)munro(at)gmail(dot)com> writes:
> > +#include "fmgr.h"
> > +#include "storage/block.h"
> > +#include "storage/relfilenode.h"
> > +#include "storage/smgr.h"
> > +#include "storage/sync.h"
>
> > Why do we need to include fmgr.h in md.h?
>
> More generally, any massive increase in an include file's inclusions
> is probably a sign that you need to refactor. Cross-header inclusions
> are best avoided altogether if you can --- obviously that's not always
> possible, but we should minimize them. We've had some very unfortunate
> problems in the past from indiscriminate #includes in headers.

Agree - I do pay attention to these, but this one slipped through the
cracks (copied smgr.h then edited to remove smgr bits). Thanks for
catching this, will fix in the next patch iteration.

--
Shawn Debnath
Amazon Web Services (AWS)