Enhancement: change link methd to reduce PLT overhead

During TPC-DS testing, we observed that compiling postgres with
libpostgres.so introduces PTL function call overhead for some functions.
By linking object files (*.o) directly instead, we achieved a 5-8%
performance improvement in the TPC-DS 1TB benchmark.

This commit modifies the postgres linking process to directly link with
the object files rather than the shared library.

Additionally, this update fixes a minor bug: the pax extension has a
dependency on libpostgres.so. Now, when enabling the pax entension, we
check that enable_shared_postgres_backend is set to 'yes' to ensure
proper functionality.

Author: gongxun0928

configure

@@ -759,8 +759,8 @@ HAVE_CXX14
 enable_gpcloud
 enable_shared_postgres_backend
 enable_mapreduce
-enable_catalog_ext
 enable_serverless
+enable_catalog_ext
 enable_orca
 autodepend
 PKG_CONFIG_LIBDIR
@@ -1608,7 +1608,7 @@ Optional Features:
   --enable-cassert        enable assertion checks (for debugging)
   --disable-orca          disable ORCA optimizer
   --enable-catalog-ext    enable Cloudberry catalog extension
-  --enable-serverless     enable Cloudberry serverless mode
+  --enable-serverless     use serverless mode of Cloudberry
   --enable-mapreduce      enable Cloudberry Mapreduce support
   --disable-shared-postgres-backend
                           enable Cloudberry shared postgres backend support
@@ -1618,7 +1618,7 @@ Optional Features:
                           library)
   --disable-preload-ic-module
                           disable preload interconnect module
-  --enable-pax            enable pax support
+  --disable-pax           disable pax support
   --disable-thread-safety disable thread-safety in client libraries
   --enable-openssl-redirect
                           enable redirect openssl interface to internal
@@ -8380,7 +8380,7 @@ $as_echo "checking whether to build with catalog extension... $enable_catalog_ex
 
 
 #
-# Enable serverless mode
+# --enable-serverless uses serverless mode of Cloudberry
 #
 
 
@@ -8407,8 +8407,8 @@ else
 fi
 
 
-{ $as_echo "$as_me:${as_lineno-$LINENO}: result: checking whether to use serverless architecture of Cloudberry ... $enable_serverless" >&5
-$as_echo "checking whether to use serverless architecture of Cloudberry ... $enable_serverless" >&6; }
+{ $as_echo "$as_me:${as_lineno-$LINENO}: result: checking whether to use serverless mode of Cloudberry... $enable_serverless" >&5
+$as_echo "checking whether to use serverless mode of Cloudberry... $enable_serverless" >&6; }
 
 
 #
@@ -9101,6 +9101,7 @@ fi
 { $as_echo "$as_me:${as_lineno-$LINENO}: result: checking whether to build with preload ic module ... $enable_preload_ic_module" >&5
 $as_echo "checking whether to build with preload ic module ... $enable_preload_ic_module" >&6; }
 
+
 #
 # pax support
 #
@@ -9125,13 +9126,16 @@ $as_echo "#define USE_PAX_STORAGE 1" >>confdefs.h
 
 else
   enable_pax=yes
-  
-  $as_echo "#define USE_PAX_STORAGE 1" >>confdefs.h
+
+$as_echo "#define USE_PAX_STORAGE 1" >>confdefs.h
+
 fi
 
+
 { $as_echo "$as_me:${as_lineno-$LINENO}: result: checking whether to build with pax support ... $enable_pax" >&5
 $as_echo "checking whether to build with pax support ... $enable_pax" >&6; }
 
+
 #
 # Include directories
 #
@@ -15699,6 +15703,13 @@ fi
 
 
 
+# for contrib/pax
+if test "$enable_pax" = yes; then
+  if test "$enable_shared_postgres_backend" = no; then
+    as_fn_error $? "pax support requires --enable-shared-postgres-backend" "$LINENO" 5
+  fi
+fi
+
 # for contrib/sepgsql
 if test "$with_selinux" = yes; then
   { $as_echo "$as_me:${as_lineno-$LINENO}: checking for security_compute_create_name in -lselinux" >&5

configure.ac

@@ -929,10 +929,10 @@ AC_SUBST(enable_preload_ic_module)
 # pax support
 #
 PGAC_ARG_BOOL(enable, pax, yes,
-              [disable PAX support],
+              [disable pax support],
               [AC_DEFINE(USE_PAX_STORAGE, 1,
                          [Define to 1 to support PAX])])
-AC_MSG_RESULT([checking whether to build with PAX support ... $enable_pax])
+AC_MSG_RESULT([checking whether to build with pax support ... $enable_pax])
 AC_SUBST(enable_pax)
 
 #
@@ -1749,6 +1749,13 @@ fi
 AC_SUBST(LDAP_LIBS_FE)
 AC_SUBST(LDAP_LIBS_BE)
 
+# for contrib/pax
+if test "$enable_pax" = yes; then
+  if test "$enable_shared_postgres_backend" = no; then
+    AC_MSG_ERROR([pax support requires --enable-shared-postgres-backend])
+  fi
+fi
+
 # for contrib/sepgsql
 if test "$with_selinux" = yes; then
   AC_CHECK_LIB(selinux, security_compute_create_name, [],

src/backend/Makefile

@@ -73,8 +73,12 @@ LIBS += -lsystemd
 endif
 
 ##########################################################################
+TARGETS = submake-libpgport submake-catalog-headers submake-utils-headers postgres $(POSTGRES_IMP)
+ifeq ($(enable_shared_postgres_backend),yes)
+TARGETS += libpostgres.so
+endif
 
-all: submake-libpgport submake-catalog-headers submake-utils-headers postgres $(POSTGRES_IMP)
+all: $(TARGETS)
 
 ifneq ($(PORTNAME), cygwin)
 ifneq ($(PORTNAME), win32)
@@ -85,10 +89,8 @@ libpostgres.so: $(OBJS)
 	$(CXX) -shared $(CXXFLAGS) $(LDFLAGS) $(LDFLAGS_SL) $(export_dynamic) \
 	  $(filter-out main/main.o, $(call expand_subsys,$^)) $(LIBS) -o $@
 
-postgres: main/main.o libpostgres.so $(top_builddir)/src/port/libpgport_srv.a $(top_builddir)/src/common/libpgcommon_srv.a
-	$(CXX) $(CXXFLAGS) $(LDFLAGS) $(LDFLAGS_EX) $(export_dynamic) \
-	  main/main.o libpostgres.so $(top_builddir)/src/port/libpgport_srv.a \
-	  $(top_builddir)/src/common/libpgcommon_srv.a $(LIBS) -o $@
+postgres: $(OBJS)
+	$(CXX) $(CXXFLAGS) $(call expand_subsys,$^) $(LDFLAGS) $(LDFLAGS_EX) $(export_dynamic) $(LIBS) -o $@
 
 else
 postgres: $(OBJS)