ardada2468 commented on code in PR #16761:
URL: https://github.com/apache/kafka/pull/16761#discussion_r1701082806


##########
tools/src/main/java/org/apache/kafka/tools/StreamsResetter.java:
##########
@@ -93,7 +93,7 @@ public class StreamsResetter {
     private static final String USAGE = "This tool helps to quickly reset an 
application in order to reprocess "
             + "its data from scratch.\n"
             + "* This tool resets offsets of input topics to the earliest 
available offset (by default), or to a specific defined position"
-            + " and it skips to the end of intermediate topics (topics that 
are input and output topics, e.g., used by deprecated through() method).\n"
+            + " and it skips to the end of intermediate topics (topics that 
are input and output topics.\n"

Review Comment:
   Good Catch, I have just patched that in the latest commit.



##########
streams/src/test/java/org/apache/kafka/streams/kstream/internals/KStreamImplTest.java:
##########
@@ -1285,62 +1257,6 @@ public void 
shouldNotAllowNullValueJoinerWithKeyOnLeftJoinWithGlobalTableWithNam
         assertThat(exception.getMessage(), equalTo("joiner can't be null"));
     }
 
-    @SuppressWarnings({"unchecked", "deprecation"}) // specifically testing 
the deprecated variant
-    @Test
-    public void testNumProcesses() {

Review Comment:
   Okay, I have also patched this in the latest commit



-- 
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: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to