Hi folks,

The configuration doc[1] marks ALLOW_TABLE_LOCATION_OVERLAP as
"(DEPRECATION)", but the feature itself is not deprecated.

The @SuppressWarnings("deprecation") annotation in
FeatureConfiguration.java exists only because the implementation calls
catalogConfigUnsafe(), which is deprecated to maintain backward
compatibility with the legacy catalog property key
allow.overlapping.table.location (without the polaris.config.prefix). The
configuration option itself remains active and fully supported.

I think the docs should remove the "(DEPRECATION)" tag from
ALLOW_TABLE_LOCATION_OVERLAP. We may also review the other entries in the
same table marked "(DEPRECATION)" (for example,
ALLOW_UNSTRUCTURED_TABLE_LOCATION, ALLOW_EXTERNAL_TABLE_LOCATION) to
confirm whether they are truly deprecated features or similarly labeled
because they use catalogConfigUnsafe().

1.
https://github.com/apache/polaris/blob/323484b37bf1544029e7898fc20eb8590c538177/docs/configuration.md#L96


Yufei

Reply via email to