Fix: adapt Makefile, fix uninitialized var

Author: Vladislav Shchetinin

gpcontrib/gp_relsizes_stats/Makefile

@@ -7,6 +7,14 @@ REGRESS         = grants gp_relsizes_stats
 REGRESS_OPTS    = --inputdir=test/
 PGFILEDESC      = "gp_relsizes_stats - an extension to track table on-disc sizes in greenplum"
 PG_CXXFLAGS     += $(COMMON_CPP_FLAGS)
-PG_CONFIG       = pg_config
-PGXS            := $(shell $(PG_CONFIG) --pgxs)
+ifdef USE_PGXS
+PG_CONFIG = pg_config
+PGXS := $(shell $(PG_CONFIG) --pgxs)
 include $(PGXS)
+else
+subdir = gpcontrib/gp_relsizes_stats
+top_builddir = ../..
+include $(top_builddir)/src/Makefile.global
+include $(top_srcdir)/contrib/contrib-global.mk
+endif
+

gpcontrib/gp_relsizes_stats/README.md

@@ -22,7 +22,7 @@ make && make install
 
 ### Confguration
 gp_relsizes_stats configuration parameters:
-| **Parameter** | **Type**     | **Default**  | **Default**  |
+| **Parameter** | **Type**     | **Default**  | **Description**  |
 | ---------------- | --------------- | ------------ | ------------ |
 | `gp_relsizes_stats.enabled`          | bool    | false    | Using `gp_relsizes_stats.enabled` you can enable/disable background stats collection for database where extension installed (actually enable/disable background worker which collecting stats).|
 | `gp_relsizes_stats.restart_naptime`  | int     | 21600000 | Using `gp_relsizes_stats.restart_naptime` you can set naptime between each startup of collecting process. Value set time in milliseconds. Default is equal to 6 hours.|

gpcontrib/gp_relsizes_stats/src/gp_relsizes_stats.c

@@ -110,7 +110,7 @@ static void worker_sigterm(SIGNAL_ARGS) {
  * Returns BGWH_STOPPED on success, BGWH_POSTMASTER_DIED on error/timeout.
  */
 static BgwHandleStatus WaitForBackgroundWorkerShutdownSafely(BackgroundWorkerHandle *handle) {
-    BgwHandleStatus status;
+    BgwHandleStatus status = BGWH_NOT_YET_STARTED;
     int rc;
     int attempts = 0;
     const int max_attempts = 5 * HOUR_TIME / 100; /* maximum 5 hours wait time */