mjsax commented on a change in pull request #9668:
URL: https://github.com/apache/kafka/pull/9668#discussion_r536256494
##########
File path:
streams/src/test/java/org/apache/kafka/streams/StreamsBuilderTest.java
##########
@@ -463,7 +463,36 @@ public void
shouldReuseSourceTopicAsChangelogsWithOptimization20() {
}
@Test
- public void shouldNotReuseSourceTopicAsChangelogsByDefault() {
+ public void shouldNotReuseRepartitionTopicAsChangelogs() {
+ final String topic = "topic";
+ builder.<Long,
String>stream(topic).repartition().toTable(Materialized.as("store"));
+ final Properties props = StreamsTestUtils.getStreamsConfig("appId");
+ props.put(StreamsConfig.TOPOLOGY_OPTIMIZATION_CONFIG,
StreamsConfig.OPTIMIZE);
+ final Topology topology = builder.build(props);
+
+ final InternalTopologyBuilder internalTopologyBuilder =
TopologyWrapper.getInternalTopologyBuilder(topology);
+ internalTopologyBuilder.rewriteTopology(new StreamsConfig(props));
+
+ assertThat(
+ internalTopologyBuilder.buildTopology().storeToChangelogTopic(),
+ equalTo(Collections.singletonMap("store", "appId-store-changelog"))
+ );
+ assertThat(
+ internalTopologyBuilder.stateStores().keySet(),
+ equalTo(Collections.singleton("store"))
+ );
+ assertThat(
+
internalTopologyBuilder.stateStores().get("store").loggingEnabled(),
+ equalTo(true)
+ );
+ assertThat(
+
internalTopologyBuilder.topicGroups().get(1).stateChangelogTopics.keySet(),
+ equalTo(Collections.singleton("appId-store-changelog"))
+ );
+ }
+
+ @Test
+ public void shouldNotReuseRepartitionTopicAsChangelogsByDefault() {
Review comment:
Ah. Yes, the test name should not change.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
