Also record direct dependency table -> server

Author: Y--

include/pgduckdb/pg/relations.hpp

@@ -35,4 +35,6 @@ const char *QuoteIdentifier(const char *ident);
 
 const char *GetRelationName(Relation rel);
 
+Oid GetOid(Form_pg_class* rel);
+
 } // namespace pgduckdb

src/pg/relations.cpp

@@ -172,4 +172,8 @@ GetRelationName(Relation rel) {
 	return RelationGetRelationName(rel);
 }
 
+Oid GetOid(Form_pg_class* rel) {
+	return rel->oid;
+}
+
 } // namespace pgduckdb

src/pgduckdb_background_worker.cpp

@@ -22,34 +22,35 @@
 
 extern "C" {
 #include "postgres.h"
-#include "fmgr.h"
 #include "access/xact.h"
-#include "miscadmin.h"
-#include "pgstat.h"
-#include "executor/spi.h"
+#include "catalog/dependency.h"
+#include "catalog/namespace.h"
+#include "catalog/objectaddress.h"
+#include "catalog/pg_authid.h"
+#include "catalog/pg_class.h"
+#include "catalog/pg_extension.h"
+#include "catalog/pg_foreign_server.h"
+#include "catalog/pg_namespace.h"
 #include "commands/dbcommands.h"
 #include "common/file_utils.h"
+#include "executor/spi.h"
+#include "fmgr.h"
+#include "miscadmin.h"
+#include "pgstat.h"
 #include "postmaster/bgworker.h"
 #include "postmaster/interrupt.h"
 #include "storage/ipc.h"
 #include "storage/latch.h"
-#include "tcop/tcopprot.h"
 #include "storage/proc.h"
 #include "storage/shmem.h"
-#include "utils/builtins.h"
-#include "catalog/dependency.h"
-#include "catalog/pg_authid.h"
-#include "catalog/namespace.h"
-#include "catalog/pg_namespace.h"
-#include "catalog/pg_extension.h"
-#include "catalog/pg_foreign_server.h"
+#include "tcop/tcopprot.h"
 #include "utils/acl.h"
+#include "utils/builtins.h"
 #include "utils/guc.h"
 #include "utils/memutils.h"
 #include "utils/palloc.h"
 #include "utils/snapmgr.h"
 #include "utils/syscache.h"
-#include "catalog/objectaddress.h"
 }
 
 #include "pgduckdb/pgduckdb.h"
@@ -231,6 +232,16 @@ BgwMainLoop() {
 	elog(LOG, "pg_duckdb background worker for database '%s' (%u) has now terminated.", db_name, MyDatabaseId);
 }
 
+void
+recordDependencyOnMDServer(ObjectAddress *object_address) {
+	ObjectAddress server_address = {
+	    .classId = ForeignServerRelationId,
+	    .objectId = GetMotherduckForeignServerOid(),
+	    .objectSubId = 0,
+	};
+	recordDependencyOn(object_address, &server_address, DEPENDENCY_NORMAL);
+}
+
 } // namespace pgduckdb
 
 extern "C" {
@@ -774,6 +785,26 @@ CreateTable(const char *postgres_schema_name, const char *table_name, const char
 		return false;
 	}
 
+	// Record the dependency on the MotherDuck server
+	{
+		HeapTuple new_table_tuple =
+		    SearchSysCache2(RELNAMENSP, CStringGetDatum(table_name), ObjectIdGetDatum(schema_oid));
+		if (HeapTupleIsValid(new_table_tuple)) {
+			Form_pg_class postgres_relation = (Form_pg_class)GETSTRUCT(new_table_tuple);
+			ReleaseSysCache(new_table_tuple);
+			ObjectAddress object_address = {
+			    .classId = RelationRelationId,
+			    .objectId = GetOid(postgres_relation),
+			    .objectSubId = 0,
+			};
+			recordDependencyOnMDServer(&object_address);
+		} else {
+			// Something went really wrong (since `create_table_succeeded`)
+			elog(WARNING, "Failed to find table '%s.%s' so won't record dependency to 'motherduck' SERVER", table_name,
+			     postgres_schema_name);
+		}
+	}
+
 	if (did_delete_table) {
 		/*
 		 * Commit the subtransaction that contains both the drop and the create that
@@ -933,12 +964,7 @@ CreateSchemaIfNotExists(const char *postgres_schema_name, bool is_default_db) {
 		    .objectId = schema_oid,
 		    .objectSubId = 0,
 		};
-		ObjectAddress server_address = {
-		    .classId = ForeignServerRelationId,
-		    .objectId = GetMotherduckForeignServerOid(), // pgduckdb::ExtensionOid(),
-		    .objectSubId = 0,
-		};
-		recordDependencyOn(&schema_address, &server_address, DEPENDENCY_NORMAL);
+		recordDependencyOnMDServer(&schema_address);
 	}
 
 	/* Success, so we commit the subtransaction */