lvyanquan commented on code in PR #139:
URL:
https://github.com/apache/flink-connector-kafka/pull/139#discussion_r1959158247
##########
flink-connector-kafka/src/test/java/org/apache/flink/streaming/connectors/kafka/internals/KafkaTopicPartitionTest.java:
##########
@@ -26,14 +27,14 @@
import static org.assertj.core.api.Assertions.assertThat;
import static org.assertj.core.api.Assertions.fail;
-/** Tests for the {@link KafkaTopicPartition}. */
+/** Tests for the {@link TopicPartition}. */
public class KafkaTopicPartitionTest {
@Test
public void validateUid() {
Review Comment:
Hi, @AHeise.
At the beginning, for the convenience of review, I split the entire work of
bump 2.0 into two PRs: [deleting deprecated
APIs](https://github.com/apache/flink-connector-kafka/pull/139) and [adapting
2.0 APIs](https://github.com/apache/flink-connector-kafka/pull/140).
However, this seems to have resulted in duplicate review work. I have
already deleted the classes mentioned here in the [adapting 2.0
API](https://github.com/apache/flink-connector-kafka/pull/140).
I plan to close this PR and let's trace it in a unified PR.
--
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]
