REST: Explicitly send purgeRequested=false when dropping table without purge

[hackintoshrao]

Summary

Modify RESTSessionCatalog.dropTable() to explicitly send purgeRequested=false instead of omitting the parameter.

Problem

Currently, dropTable(context, identifier) sends a DELETE request without the purgeRequested parameter, relying on the server to default to false.

However, some catalogs default purgeRequested=true to work around Spark's bug where it doesn't forward purgeRequested=true even with DROP TABLE ... PURGE (#11023, #11317).

With such catalogs, calling catalog.dropTable(identifier, false) unexpectedly purges table data because the parameter is omitted and the server defaults to true.

Solution

Explicitly send purgeRequested=false:

client.delete(
    paths.table(identifier),
    ImmutableMap.of("purgeRequested", "false"),
    ...);

This ensures the client's intent is honored regardless of server defaults.

Related Issues

• deletion & purge improvements for undelete feature in REST catalog #11023 - deletion & purge improvements for undelete feature in REST catalog
• Spark: add property to disable client-side purging in spark #11317 - Spark: add property to disable client-side purging in spark