Kafka Streams dev console topology view can't handle "." in topic name

olekp commented 11 months ago

Describe the bug

When Kafka Streams use topic with "." (e.g. "my.special.topic"), graph tab fails (no graph is displayed), in browser console there is message:

graphviz.ts:153 Uncaught Error: syntax error in line 4 near '.'

    at S.layout (graphviz.ts:153:19)
    at S.dot (graphviz.ts:211:21)
    at QwcKafkaStreamsTopology._selectGraphTab (qwc-kafka-streams-topology.js:84:32)
    at QwcKafkaStreamsTopology._tabSelectedChanged (qwc-kafka-streams-topology.js:78:24)
    at QwcKafkaStreamsTopology.<anonymous> (qwc-kafka-streams-topology.js:49:71)
    at z.handleEvent (lit-html.ts:2018:29)
    at dispatchNotifyEvent (property-effects.js:328:43)
    at Object.runNotifyEffect [as fn] (property-effects.js:353:3)
    at runEffectsForProperty (property-effects.js:176:12)
    at runNotifyEffects (property-effects.js:263:18)

Expected behavior

Topology visualization can handle topics with "." in name.

Actual behavior

No response

How to Reproduce?

Use https://github.com/quarkusio/quarkus-quickstarts/tree/3.2/kafka-streams-quickstart Change org.acme.kafka.streams.aggregator.streams.TopologyProducer#WEATHER_STATIONS_TOPIC to

static final String WEATHER_STATIONS_TOPIC = "weather.stations";

Output of `uname -a` or `ver`

No response

Output of `java -version`

No response

Quarkus version or git rev

3.6.1

Build tool (ie. output of `mvnw --version` or `gradlew --version`)

No response

Additional information

No response

quarkus-bot[bot] commented 11 months ago

/cc @alesj (kafka,kafka-streams), @cescoffier (kafka), @gunnarmorling (kafka-streams), @ozangunalp (kafka,kafka-streams), @rquinio (kafka-streams)

dcotfr commented 10 months ago

I think that this issue could be solved by pull request https://github.com/quarkusio/quarkus/pull/37124.

quarkusio / quarkus