MisterRaindrop commented on code in PR #1842:
URL: https://github.com/apache/cloudberry/pull/1842#discussion_r3556049890
##########
src/backend/catalog/heap.c:
##########
@@ -2322,6 +2323,10 @@ heap_drop_with_catalog(Oid relid)
*/
CheckTableForSerializableConflictIn(rel);
+ /* If this is a lake table, remove its pg_lake_table entry */
+ if (RelationIsIcebergTable(rel))
+ RemoveLakeTableEntry(relid);
Review Comment:
I can migrate this logic to object_access_hook, but the issue is that the AM
layer hasn’t been implemented yet on my end, which blocks the migration.
So do I need to implement a default plugin for the AM layer first? Then
replace this default AM plugin later once the official datalake plugin is fully
rolled out?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
