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

[Spark] Reflect a new parameter in the constructor of LogicalRelation for Spark 4.0 #3847

Closed HeartSaVioR closed 2 weeks ago

HeartSaVioR commented 2 weeks ago

Which Delta project/connector is this regarding?

Description

This PR proposes to reflect a new parameter in the constructor of LogicalRelation for Spark 4.0. This PR deals with it via creating a shim object like we have done with IncrementalExecution for parameters difference.

There are overloaded methods to create a LogicalRelation instance and Spark community tried to provide the new method to avoid this change, but Spark couldn't just add a new overloaded method to workaround as it was ambiguous with existing method with default param.

How was this patch tested?

Existing tests.

Does this PR introduce any user-facing changes?

No.