Skip to content

Conversation

@vbotbuildovich
Copy link
Collaborator

Backport of PR #27188

Glue doesn't support the purge option when dropping tables, returning a
400 error immediately, even after the underlying table has been dropped:

```
WARN  2025-08-08 19:00:21,698 [shard 0:main] iceberg - catalog_client.cc:367 - [/iceberg/v1/catalogs/12345/namespaces/redpanda/tables/table?purgeRequested=true] error: http_call_error: Bad Request, message: '{"error":{"code":400,"message":"PurgeRequested cannot be true for Glue iceberg tables.","type":"InvalidInputException"}}'
```

When in this state, translation for this topic and all other topics of
the same name is stuck indefinitely, because when
redpanda.iceberg.delete is true, we will try to drop the table of older
versions of the topic before allowing creation of the new table.

This commit adds a stop-gap to at least be able to get unstuck by
manually deleting the table. When dropping, we'll first check if the
dropped table has been deleted already.

(cherry picked from commit 8dd12da)
Redshift complains when using an null 'partitions' field in the manifest
list:

```
ERROR: Wrong type in Avro file. Detail: ----------------------------------------------- error: Wrong type in Avro file. code: 15003 context: Field: partitions. Expected: 12. Got: 7 query: -1[child_sequence:1] location: avro_utils.hpp:55 process: padbmaster [pid=1073913955] ----------------------------------------------- [ErrorId: 1-68963282-20644b700a198f806702f6aa]
```

This fix has Redpanda serialize empty 'partitions' as an empty array.
Tested manually against Glue and Redshift.

(cherry picked from commit 61c9992)
@vbotbuildovich vbotbuildovich added this to the v25.1.x-next milestone Aug 11, 2025
@vbotbuildovich vbotbuildovich added the kind/backport PRs targeting a stable branch label Aug 11, 2025
@vbotbuildovich vbotbuildovich requested a review from andrwng August 11, 2025 18:26
@vbotbuildovich
Copy link
Collaborator Author

CI test results

test results on build#70575
test_class test_method test_arguments test_kind job_url test_status passed reason
gtest_cloud_storage_rpfixture gtest_cloud_storage_rpfixture unit https://buildkite.com/redpanda/redpanda/builds/70575#01989a64-39fb-4d61-851b-10fea1899ef4 FLAKY 1/2

@andrwng andrwng merged commit cea7738 into redpanda-data:v25.1.x Aug 12, 2025
17 checks passed
@lf-rep lf-rep modified the milestones: v25.1.x-next, 25.1.10-rc2 Aug 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/redpanda kind/backport PRs targeting a stable branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants