A few places used binary_upgrade_* variables without including the header,
which worked without warnings because the variables are defined in those
places. However that can cause linker complaints with MSVC - except that we
don't see them right now, due to the use of a symbol export file.
Discussion: https://postgr.es/m/
20220512164513.vaheofqp2q24l65r@alap3.anarazel.de
#include "access/relation.h"
#include "access/table.h"
#include "access/tableam.h"
+#include "catalog/binary_upgrade.h"
#include "catalog/catalog.h"
#include "catalog/heap.h"
#include "catalog/index.h"
#include "access/xact.h"
#include "access/xloginsert.h"
#include "access/xlogutils.h"
+#include "catalog/binary_upgrade.h"
#include "catalog/catalog.h"
#include "catalog/dependency.h"
#include "catalog/indexing.h"