Copilot commented on code in PR #20498:
URL: https://github.com/apache/kafka/pull/20498#discussion_r2329360643
##########
streams/integration-tests/src/test/java/org/apache/kafka/streams/integration/RestoreIntegrationTest.java:
##########
@@ -161,8 +159,8 @@ public void createTopics(final TestInfo testInfo) throws
InterruptedException {
CLUSTER.createTopic(inputStream, 2, 1);
}
- private Properties props(final boolean stateUpdaterEnabled) {
- return
props(mkObjectProperties(mkMap(mkEntry(InternalConfig.STATE_UPDATER_ENABLED,
stateUpdaterEnabled))));
+ private Properties props() {
Review Comment:
[nitpick] The `props()` method now creates an empty map which might be
confusing. Consider adding a comment explaining that this method creates
default properties without additional state-updater configuration, or rename it
to `defaultProps()` for clarity.
```suggestion
/**
* Creates default properties for a Streams application without
additional state-updater configuration.
*/
private Properties defaultProps() {
```
--
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]
