bbejeck commented on code in PR #23586:
URL: https://github.com/apache/kafka/pull/23586#discussion_r4136982086
##########
streams/integration-tests/src/test/java/org/apache/kafka/streams/integration/SourceTopicChangelogRestoreIntegrationTest.java:
##########
@@ -109,7 +109,8 @@ public void setUp(final TestInfo testInfo) throws Exception
{
for (final String key : preloadedKeys()) {
vehicles.add(KeyValue.pair(key, "initial-state"));
}
- produce(TABLE_TOPIC, vehicles);
+ // batch the preload so the 5000 records load quickly (see produce())
+ produce(TABLE_TOPIC, vehicles, false);
Review Comment:
Good catch, the preload is what the restore consumer reads back, so batching
it lets the restore get through it in a handful of fetches. I reverted to one
record per batch everywhere. The test still fails without the fix.
--
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]
