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.64k stars 1.71k forks source link

[3.2] DeltaCatalog.createTable should respect PROP_IS_MANAGED_LOCATION #3663

Closed tdas closed 2 months ago

tdas commented 2 months ago

Which Delta project/connector is this regarding?

Description

Even if a table has the location field, it should still be a managed table if PROP_IS_MANAGED_LOCATION is present in the table properties.

Note: this case won't happen with Spark integration solely. It's only an issue for third-party catalogs that delegate requests to DeltaCatalog, such as Unity Catalog.

How was this patch tested?

new test

Does this PR introduce any user-facing changes?

no