[Kernel] [CC Refactor] Make Protocol implement AbstractProtocol

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 Protocol implement AbstractProtocol #3838

Open scottsand-db opened 3 weeks ago

scottsand-db commented 3 weeks ago

Which Delta project/connector is this regarding?

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

Description

Make Protocol implement AbstractProtocol.

Add some better validation of constructor params, too.

Add a simple ActionSerializerSuite, that is not fully fleshed out, that makes sure we are writing good protocol json.

How was this patch tested?

Existing + new UTs.

Does this PR introduce any user-facing changes?

No.