delta-io / delta

An open-source storage framework that enables building a Lakehouse architecture with compute engines including Spark, PrestoDB, Flink, Trino, and Hive and APIs
https://delta.io
Apache License 2.0
7.62k stars 1.71k forks source link

[Protocol] Amend Row Tracking Protocol to explicitly require domainMetadata #3740

Closed c27kwan closed 1 month ago

c27kwan commented 1 month ago

Which Delta project/connector is this regarding?

Description

Right now, the Delta protocol for Row tracking references the DomainMetadata table feature, but does not explicitly state domainMetadata as one of its required table feature like e.g. Clusteredtable. The code in TableFeature.scala accurately lists domainMetadata as one of the required table feature.

This PR amends the Protocol to accurately reflect the state of the system and the remainder of the Row tracking proposition. Row Tracking cannot require writers to write DomainMetadata if it is not listing domainMetadata as one of its required feature. This is also to be consistent with how other table features handle listing their required table features.

How was this patch tested?

N/A

Does this PR introduce any user-facing changes?

'No'

c27kwan commented 1 month ago

@johanl-db PTAL