Marton Greber has posted comments on this change. ( http://gerrit.cloudera.org:8080/24518 )
Change subject: [java] Replace deprecated APIs to silence build warnings ...................................................................... Patch Set 6: (1 comment) just one more question else LGTM. http://gerrit.cloudera.org:8080/#/c/24518/6/java/kudu-client/src/test/java/org/apache/kudu/client/TestPartitionPruner.java File java/kudu-client/src/test/java/org/apache/kudu/client/TestPartitionPruner.java: http://gerrit.cloudera.org:8080/#/c/24518/6/java/kudu-client/src/test/java/org/apache/kudu/client/TestPartitionPruner.java@67 PS6, Line 67: @Deprecated nit q: Adding `@Deprecated` here fixes the dep-ann lint warning, but the callers at lines 892 and 983 in this same file don't have `@SuppressWarnings("deprecation")`. If `-Xlint:deprecation` is enabled for test compilation, this would introduce new deprecation warnings at those call sites rather than silencing them. Is the build still clean, or do the callers need suppression too? -- To view, visit http://gerrit.cloudera.org:8080/24518 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia10fdbca3733d9b16caf8d4c0ad00e2234092d39 Gerrit-Change-Number: 24518 Gerrit-PatchSet: 6 Gerrit-Owner: Zoltan Chovan <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Wed, 01 Jul 2026 10:11:55 +0000 Gerrit-HasComments: Yes
