[Kernel] [CC Refactor] Make CommitInfo implement AbstractCommitInfo

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

[Kernel] [CC Refactor] Make CommitInfo implement AbstractCommitInfo #3833

Open scottsand-db opened 3 weeks ago

scottsand-db commented 3 weeks ago

Which Delta project/connector is this regarding?

[ ] Spark
[ ] Standalone
[ ] Flink
[ ] Kernel
[ ] Other (fill in here)

Description

This PR makes CommitInfo depend on AbstractCommitInfo. This lets us remove the CoordinatedCommitsUtils.convertCommitInfoToAbstractCommitInfo method.

Also cleans up code along the way, and adds version (read from the file name) to CommitInfo.

How was this patch tested?

Existing UTs.

Does this PR introduce any user-facing changes?

No.