Fix compile errors detected by gcc 12 (#503)

* Check the string value, not the array itself
* Use global namespace to eliminate ambiguity
* Ignore assert condition.

Reviewed-by: Zhang Mingli [email protected]

Author: gfphoenix78

src/backend/gpopt/gpdbwrappers.cpp

@@ -2168,7 +2168,7 @@ gpdb::ParseExternalTableUri(const char *uri)
 {
 	GP_WRAP_START;
 	{
-		return ParseExternalTableUri(uri);
+		return ::ParseExternalTableUri(uri);
 	}
 	GP_WRAP_END;
 	return nullptr;

src/backend/gpopt/translate/CTranslatorQueryToDXL.cpp

@@ -4211,10 +4211,16 @@ CTranslatorQueryToDXL::CreateDXLProjectNullsForGroupingSets(
 
 		INT sort_group_ref = INT(target_entry->ressortgroupref);
 
+#if 0
+		// FIXME: The following assert is wrong for its semantics that may
+		// call a member function on a null pointer.
+		// The assert expression is highly relative to the if condition below.
+		// We should figure out what the assert state really is and add it back.
 		GPOS_ASSERT_IMP(
 			nullptr == sort_grouping_col_mapping,
 			nullptr != sort_grouping_col_mapping->Find(&sort_group_ref) &&
 				"Grouping column with no mapping");
+#endif
 
 		if (0 < sort_group_ref && 0 < colid &&
 			nullptr == sort_grouping_col_mapping->Find(&sort_group_ref))

src/backend/postmaster/loginmonitor.c

@@ -572,7 +572,7 @@ LoginMonitorWorkerMain(int argc, char *argv[]) {
 		SetConfigOption("synchronous_commit", "local",
 				PGC_SUSET, PGC_S_OVERRIDE);
 
-	if (LoginMonitorShmem->curr_user_name) {
+	if (LoginMonitorShmem->curr_user_name[0] != '\0') {
 		InitPostgres(DB_FOR_COMMON_ACCESS, InvalidOid, NULL, InvalidOid, NULL, false);
 		SetProcessingMode(NormalProcessing);
 		set_ps_display(LoginMonitorShmem->curr_user_name);