Skip to content

Conversation

wdberkeley
Copy link
Contributor

Manual backport to resolve a trivial BUILD file conflict.

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v25.2.x
  • v25.1.x
  • v24.3.x

Release Notes

Improvements

  • A new configuration option iceberg_topic_name_dot_replacement specifies a string that will replace dots in Iceberg topic names when those names are converted into Iceberg table names. This allows working around downstream systems which do not allow table names with dots in them. Be careful to avoid table name collisions by picking a proper replacement string, and do not change the replacement string if there exists an Iceberg topic with dots in the name.

This defines a configuration property `iceberg_topic_name_dot_replacement`
that, when set, will replace .'s in table names when a Kafka topic name
is translated into the corresponding Iceberg table name (or DLQ table name).

The replacement is done using the current value of the setting, so to
preserve the link between topic and table, the setting should not be
changed once there is an Iceberg topic whose name contains dots. It'd be
nice to copy the cluster configuration into topic configuration when an
Iceberg topic is created or when Iceberg mode is set for a topic, so the
link is preserved even if the cluster configuration is changed, but
that's a far more invasive change. This workaround may suffice.

(cherry picked from commit 5cf2be1)
@vbotbuildovich
Copy link
Collaborator

CI test results

test results on build#71169
test_class test_method test_arguments test_kind job_url test_status passed reason
kafka_server_rpfixture kafka_server_rpfixture unit https://buildkite.com/redpanda/redpanda/builds/71169#0198cd61-254a-4dda-972d-d382af8b5517 FLAKY 1/2

@lf-rep lf-rep merged commit 96c300f into redpanda-data:v25.1.x Aug 28, 2025
13 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants