[Kernel] Rename `TableClient` to `Engine`

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

6.96k stars 1.58k forks source link

[Kernel] Rename `TableClient` to `Engine` #3015

Closed allisonport-db closed 2 weeks ago

allisonport-db commented 2 weeks ago

Which Delta project/connector is this regarding?

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

Description

Renames TableClient to Engine in package names and code.

A follow-up PR will be needed to update the USER_GUIDE and README

How was this patch tested?

Existing tests suffice.

Does this PR introduce any user-facing changes?

Yes, renames public interface.

vkorukanti commented 2 weeks ago

Please check the flink failures. It looks like intermittent Unidos errors,

abhishekrb19 commented 2 weeks ago

I'm guessing this would be a breaking change for all Kernel API consumers, but one that could be fixed with a simple find-and-replace?

vkorukanti commented 2 weeks ago

I'm guessing this would be a breaking change for all Kernel API consumers, but one that could be fixed with a simple find-and-replace?

Thats correct. Sorry for the trouble. We have to rename this eventually to better reflect the interface and functionality it provides. It is better to do it now as the Kernel APIs are still evolving.