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

[Spark] Makes fields in CommitOwner-related classes private and introduces getters #3033

Closed LukasRupprecht closed 1 week ago

LukasRupprecht commented 2 weeks ago

Which Delta project/connector is this regarding?

Description

This is the second PR in a series of PRs to move the current CommitOwner interface to its own module. PR1 is here.

This PR makes the fields of all CommitOwner-related classes private and forces callers to use getters to access the fields. This is needed in preparation for making the CommitOwner module a Java module (the same as the existing LogStore module) and to follow Java best practices to keep fields private and only allow access through getters.

How was this patch tested?

Simple refactor so existing tests are sufficient.

Does this PR introduce any user-facing changes?

No