[gp_stats_collector] Rename yagp_hooks_collector to gp_stats_collector

Rename extension, shared library, SQL objects, GUC prefix, test files,
and all internal identifiers.  Restore accidentally deleted files.
Clean up stray gmon.out.

Author: NJrslv

.github/workflows/build-cloudberry-rocky8.yml

@@ -320,6 +320,10 @@ jobs:
                                "gpcontrib/gp_sparse_vector:installcheck",
                                "gpcontrib/gp_toolkit:installcheck"]
               },
+              {"test":"gpcontrib-gp-stats-collector",
+               "make_configs":["gpcontrib/gp_stats_collector:installcheck"],
+               "extension":"gp_stats_collector"
+              },
               {"test":"ic-fixme",
                "make_configs":["src/test/regress:installcheck-fixme"],
                "enable_core_check":false
@@ -540,10 +544,11 @@ jobs:
         if: needs.check-skip.outputs.should_skip != 'true'
         env:
           SRC_DIR: ${{ github.workspace }}
+          CONFIGURE_EXTRA_OPTS: --with-gp-stats-collector
         run: |
           set -eo pipefail
           chmod +x "${SRC_DIR}"/devops/build/automation/cloudberry/scripts/configure-cloudberry.sh
-          if ! time su - gpadmin -c "cd ${SRC_DIR} && SRC_DIR=${SRC_DIR} ENABLE_DEBUG=${{ env.ENABLE_DEBUG }} ${SRC_DIR}/devops/build/automation/cloudberry/scripts/configure-cloudberry.sh"; then
+          if ! time su - gpadmin -c "cd ${SRC_DIR} && SRC_DIR=${SRC_DIR} ENABLE_DEBUG=${{ env.ENABLE_DEBUG }} CONFIGURE_EXTRA_OPTS=${{ env.CONFIGURE_EXTRA_OPTS }} ${SRC_DIR}/devops/build/automation/cloudberry/scripts/configure-cloudberry.sh"; then
             echo "::error::Configure script failed"
             exit 1
           fi
@@ -1400,6 +1405,7 @@ jobs:
         if: success() && needs.check-skip.outputs.should_skip != 'true'
         env:
           SRC_DIR: ${{ github.workspace }}
+          BUILD_DESTINATION: /usr/local/cloudberry-db
         shell: bash {0}
         run: |
           set -o pipefail
@@ -1423,6 +1429,30 @@ jobs:
           # 2. Follow the same pattern as optimizer
           # 3. Update matrix entries to include the new setting
 
+          # Create extension if required
+          if [[ "${{ matrix.extension != '' }}" == "true" ]]; then
+            case "${{ matrix.extension }}" in
+              gp_stats_collector)
+                if ! su - gpadmin -c "source ${BUILD_DESTINATION}/cloudberry-env.sh && \
+                  source ${SRC_DIR}/gpAux/gpdemo/gpdemo-env.sh && \
+                  gpconfig -c shared_preload_libraries -v 'gp_stats_collector' && \
+                  gpstop -ra && \
+                  echo 'CREATE EXTENSION IF NOT EXISTS gp_stats_collector; \
+                        SHOW shared_preload_libraries; \
+                        TABLE pg_extension;' | \
+                    psql postgres"
+                then
+                    echo "Error creating gp_stats_collector extension"
+                    exit 1
+                fi
+                ;;
+              *)
+                echo "Unknown extension: ${{ matrix.extension }}"
+                exit 1
+                ;;
+            esac
+          fi
+
           # Set PostgreSQL options if defined
           PG_OPTS=""
           if [[ "${{ matrix.pg_settings.optimizer != '' }}" == "true" ]]; then

.github/workflows/build-cloudberry.yml

@@ -271,9 +271,9 @@ jobs:
                 },
                 "enable_core_check":false
               },
-              {"test":"gpcontrib-yagp-hooks-collector",
-               "make_configs":["gpcontrib/yagp_hooks_collector:installcheck"],
-               "extension":"yagp_hooks_collector"
+              {"test":"gpcontrib-gp-stats-collector",
+               "make_configs":["gpcontrib/gp_stats_collector:installcheck"],
+               "extension":"gp_stats_collector"
               },
               {"test":"ic-expandshrink",
                "make_configs":["src/test/isolation2:installcheck-expandshrink"]
@@ -539,7 +539,7 @@ jobs:
         if: needs.check-skip.outputs.should_skip != 'true'
         env:
           SRC_DIR: ${{ github.workspace }}
-          CONFIGURE_EXTRA_OPTS: --with-yagp-hooks-collector
+          CONFIGURE_EXTRA_OPTS: --with-gp-stats-collector
         run: |
           set -eo pipefail
           chmod +x "${SRC_DIR}"/devops/build/automation/cloudberry/scripts/configure-cloudberry.sh
@@ -1441,17 +1441,17 @@ jobs:
           # Create extension if required
           if [[ "${{ matrix.extension != '' }}" == "true" ]]; then
             case "${{ matrix.extension }}" in
-              yagp_hooks_collector)
+              gp_stats_collector)
                 if ! su - gpadmin -c "source ${BUILD_DESTINATION}/cloudberry-env.sh && \
                   source ${SRC_DIR}/gpAux/gpdemo/gpdemo-env.sh && \
-                  gpconfig -c shared_preload_libraries -v 'yagp_hooks_collector' && \
+                  gpconfig -c shared_preload_libraries -v 'gp_stats_collector' && \
                   gpstop -ra && \
-                  echo 'CREATE EXTENSION IF NOT EXISTS yagp_hooks_collector; \
+                  echo 'CREATE EXTENSION IF NOT EXISTS gp_stats_collector; \
                         SHOW shared_preload_libraries; \
                         TABLE pg_extension;' | \
                     psql postgres"
                 then
-                    echo "Error creating yagp_hooks_collector extension"
+                    echo "Error creating gp_stats_collector extension"
                     exit 1
                 fi
                 ;;

.github/workflows/build-deb-cloudberry.yml

@@ -252,6 +252,10 @@ jobs:
                                "gpcontrib/gp_sparse_vector:installcheck",
                                "gpcontrib/gp_toolkit:installcheck"]
               },
+              {"test":"gpcontrib-gp-stats-collector",
+               "make_configs":["gpcontrib/gp_stats_collector:installcheck"],
+               "extension":"gp_stats_collector"
+              },
               {"test":"ic-cbdb-parallel",
                "make_configs":["src/test/regress:installcheck-cbdb-parallel"]
               }
@@ -448,13 +452,14 @@ jobs:
         shell: bash
         env:
           SRC_DIR: ${{ github.workspace }}
+          CONFIGURE_EXTRA_OPTS: --with-gp-stats-collector
         run: |
           set -eo pipefail
 
           export BUILD_DESTINATION=${SRC_DIR}/debian/build
 
           chmod +x "${SRC_DIR}"/devops/build/automation/cloudberry/scripts/configure-cloudberry.sh
-          if ! time su - gpadmin -c "cd ${SRC_DIR} && SRC_DIR=${SRC_DIR} ENABLE_DEBUG=${{ env.ENABLE_DEBUG }} BUILD_DESTINATION=${BUILD_DESTINATION} ${SRC_DIR}/devops/build/automation/cloudberry/scripts/configure-cloudberry.sh"; then
+          if ! time su - gpadmin -c "cd ${SRC_DIR} && SRC_DIR=${SRC_DIR} ENABLE_DEBUG=${{ env.ENABLE_DEBUG }} CONFIGURE_EXTRA_OPTS=${{ env.CONFIGURE_EXTRA_OPTS }} BUILD_DESTINATION=${BUILD_DESTINATION} ${SRC_DIR}/devops/build/automation/cloudberry/scripts/configure-cloudberry.sh"; then
             echo "::error::Configure script failed"
             exit 1
           fi
@@ -1341,6 +1346,7 @@ jobs:
         if: success() && needs.check-skip.outputs.should_skip != 'true'
         env:
           SRC_DIR: ${{ github.workspace }}
+          BUILD_DESTINATION: ${{ github.workspace }}/debian/build
         shell: bash {0}
         run: |
           set -o pipefail
@@ -1365,6 +1371,30 @@ jobs:
           # 3. Update matrix entries to include the new setting
 
 
+          # Create extension if required
+          if [[ "${{ matrix.extension != '' }}" == "true" ]]; then
+            case "${{ matrix.extension }}" in
+              gp_stats_collector)
+                if ! su - gpadmin -c "source ${BUILD_DESTINATION}/cloudberry-env.sh && \
+                  source ${SRC_DIR}/gpAux/gpdemo/gpdemo-env.sh && \
+                  gpconfig -c shared_preload_libraries -v 'gp_stats_collector' && \
+                  gpstop -ra && \
+                  echo 'CREATE EXTENSION IF NOT EXISTS gp_stats_collector; \
+                        SHOW shared_preload_libraries; \
+                        TABLE pg_extension;' | \
+                    psql postgres"
+                then
+                    echo "Error creating gp_stats_collector extension"
+                    exit 1
+                fi
+                ;;
+              *)
+                echo "Unknown extension: ${{ matrix.extension }}"
+                exit 1
+                ;;
+            esac
+          fi
+
           # Set PostgreSQL options if defined
           PG_OPTS=""
           if [[ "${{ matrix.pg_settings.optimizer != '' }}" == "true" ]]; then

.gitignore

@@ -74,8 +74,3 @@ lib*.pc
 /tmp_install/
 /.cache/
 /install/
-*.o
-*.so
-src/protos/
-.vscode
-compile_commands.json

Makefile

@@ -3,12 +3,14 @@
 # to build Postgres with a different make, we have this make file
 # that, as a service, will look for a GNU make and invoke it, or show
 # an error message if none could be found.
+
 # If the user were using GNU make now, this file would not get used
 # because GNU make uses a make file named "GNUmakefile" in preference
 # to "Makefile" if it exists. PostgreSQL is shipped with a
 # "GNUmakefile". If the user hasn't run the configure script yet, the
 # GNUmakefile won't exist yet, so we catch that case as well.
 
+
 # AIX make defaults to building *every* target of the first rule.  Start with
 # a single-target, empty rule to make the other targets non-default.
 all:

configure

@@ -722,7 +722,7 @@ with_apr_config
 with_libcurl
 with_rt
 with_zstd
-with_yagp_hooks_collector
+with_gp_stats_collector
 with_libbz2
 LZ4_LIBS
 LZ4_CFLAGS
@@ -943,7 +943,7 @@ with_zlib
 with_lz4
 with_libbz2
 with_zstd
-with_yagp_hooks_collector
+with_gp_stats_collector
 with_rt
 with_libcurl
 with_apr_config
@@ -11153,14 +11153,14 @@ fi
 fi
 
 #
-# yagp_hooks_collector
+# gp_stats_collector
 #
 
 
 
-# Check whether --with-yagp-hooks-collector was given.
-if test "${with_yagp_hooks_collector+set}" = set; then :
-  withval=$with_yagp_hooks_collector;
+# Check whether --with-gp-stats-collector was given.
+if test "${with_gp_stats_collector+set}" = set; then :
+  withval=$with_gp_stats_collector;
   case $withval in
     yes)
       :
@@ -11169,12 +11169,12 @@ if test "${with_yagp_hooks_collector+set}" = set; then :
       :
       ;;
     *)
-      as_fn_error $? "no argument expected for --with-yagp-hooks-collector option" "$LINENO" 5
+      as_fn_error $? "no argument expected for --with-gp-stats-collector option" "$LINENO" 5
       ;;
   esac
 
 else
-  with_yagp_hooks_collector=no
+  with_gp_stats_collector=no
 
 fi
 

configure.ac

@@ -1366,11 +1366,22 @@ AC_MSG_RESULT([$with_zstd])
 AC_SUBST(with_zstd)
 
 #
-# yagp_hooks_collector
+# gp_stats_collector
 #
-PGAC_ARG_BOOL(with, yagp_hooks_collector, no,
-              [build with YAGP hooks collector extension])
-AC_SUBST(with_yagp_hooks_collector)
+PGAC_ARG_BOOL(with, gp_stats_collector, no,
+              [build with stats collector extension])
+AC_SUBST(with_gp_stats_collector)
+
+if test "$with_gp_stats_collector" = yes; then
+  PKG_CHECK_MODULES([PROTOBUF], [protobuf >= 3.0.0],
+    [],
+    [AC_MSG_ERROR([protobuf >= 3.0.0 is required for gp_stats_collector])]
+  )
+  AC_PATH_PROG([PROTOC], [protoc], [no])
+  if test "$PROTOC" = no; then
+    AC_MSG_ERROR([protoc is required for gp_stats_collector but was not found in PATH])
+  fi
+fi
 
 if test "$with_zstd" = yes; then
   dnl zstd_errors.h was renamed from error_public.h in v1.4.0

gpcontrib/Makefile

@@ -35,8 +35,8 @@ else
                diskquota
 endif
 
-ifeq "$(with_yagp_hooks_collector)" "yes"
-	recurse_targets += yagp_hooks_collector
+ifeq "$(with_gp_stats_collector)" "yes"
+	recurse_targets += gp_stats_collector
 endif
 ifeq "$(with_zstd)" "yes"
 	recurse_targets += zstd

…ntrib/yagp_hooks_collector/.clang-format …contrib/gp_stats_collector/.clang-formatgpcontrib/yagp_hooks_collector/.clang-format renamed to gpcontrib/gp_stats_collector/.clang-format gpcontrib/yagp_hooks_collector/.clang-format renamed to gpcontrib/gp_stats_collector/.clang-format

@@ -0,0 +1,5 @@
+*.o
+*.so
+src/protos/
+.vscode
+compile_commands.json