Update begin delete

[shelton408]

Description

Changes to separate ConnectorTableHandles for deletes into it's own interface (ConnectorDeleteTableHandle) similar to ConnectorInsertTableHandle

If you have a connector that is broken by these changes, please refer to the fixes to presto-iceberg as an example. If you do not implement beginDelete, the changes will not affect you.

Motivation and Context

Please refer to #24520

Impact

Metadata, ConnectorMetadata, MetadataManager changed
->downstream affects execution: TableWriteInfo, ExecutionWriterTarget, TableWriterNode
Create 2 new types: ConnectorDeleteTableHandle, DeleteTableHandle
Types Changed: TableWriterNode.DeleteHandle

Also affects finishDelete, getDeleteRowIdColumnHandle types (AbstractMockMetadata, DelegatingMetadataManager)

Other classes affected:
ConnectorPageSinkProvider -> provide new pagesink for deletes?
ConnectorHandleResolver -> provide new getDeleteTableHandleClass method -> HandleResolver implements new ConnectorDeleteTableHandle

All connectors affected, those that implement DELETE will require more changes.
Kudu and iceberg connector both need to change types used for begin/finish delete
Iceberg currently uses IcebergTableHandle which inherits from basehivetablehandle->connectorTableHandle, and we would need to create a separate IcebergTableHandle for deletes

This would eventually apply to update as well if were going to make the code symmetric

Test Plan

Pass presto-icebergs tests (includes DELETE tests)
TODO: Integration test's with facebooks internal DELETE

Contributor checklist

• Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
• PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
• Documented new properties (with its default value), SQL syntax, functions, or other functionality.
• If release notes are required, they follow the release notes guidelines.
• Adequate tests were added if applicable.
• CI passed.

Release Notes

Please follow release notes guidelines and fill in the release notes below.

== RELEASE NOTES ==

General Changes
* Fix OSS connectors affected by changes 
* Add serialization for new types
* Add pagesink for DELETES to support future use
* Update beginDelete to return the new types, and finishDelete to accept the new types

SPI Changes
* Add a separate ConnectorDeleteTableHandle interface for `ConnectorMetadata.beginDelete` and `ConnectorMetadata.finishDelete`, replacing the previous usage of ConnectorTableHandle
* Add DeleteTableHandle support these changes in Metadata

Hive Connector Changes
* Replaced return type of beginDelete

Iceberg Connector Changes
* Fix IcebergTableHandle implementation to work with new types used in begin/finishDelete

Kudu Connector Changes
* Replaced return type of beginDelete

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: shelton408 (399bba2)

[shelton408]

Has passed presto-Iceberg delete tests In the current state

[steveburnett]

LGTM! (docs)

Pull branch, local doc build, looks good. Thanks!

[ZacBlanco]

The changes on the Iceberg side look good to me

[ghelmling]

Can you update the PR description with a release note, something like:

== RELEASE NOTES ==

SPI Changes
* Add a separate ConnectorDeleteTableHandle interface for `ConnectorMetadata.beginDelete` and `ConnectorMetadata.finishDelete`, replacing the previous usage of ConnectorTableHandle.

[ghelmling]

Left a couple minor notes on comments to clean up.

For the "Release Notes", I think you only really need the "SPI Changes" section and I'm not sure you need to mention DeleteTableHandle as the usage seems internal instead of part of the Connector SPI. You could instead list a second bullet on the ConnectorHandleResolver change that was also needed.

[rschlussel]

Kudu tests are failing. I expect you need to update the KuduHandleResolver similar to Iceberg.