edith007 opened a new pull request, #3881: URL: https://github.com/apache/iceberg-python/pull/3881
## Problem `RestCatalog.drop_table()` passed a Python `bool` as a query param. `requests` serializes `True`/`False` as `"True"`/`"False"`, which is invalid for OpenAPI `type: boolean` query parameters and causes 400s on strict REST catalogs (e.g. Aliyun OSS Tables). ## Solution Serialize explicitly as lowercase `"true"` / `"false"`. ## Test plan - [x] added unit test asserting query string is `purgeRequested=true|false` - [x] `pytest tests/catalog/test_rest.py -k "drop_table_serializes_purge or delete_table_204"` Fixes #3836 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
