Zoltan Chovan 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)

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 ca
Build stays clean, verified :kudu-client:compileTestJava compiles with no 
warnings at the call sites.
Two reasons: Java compilation here doesn't enable -Xlint:deprecation (only the 
Scala compiler sets -Xlint)
also, per JLS 9.6.4.6[0] uses of a deprecated member within the same top-level 
class (the callers at 892/983 are in TestPartitionPruner, same as the method) 
don't generate deprecation warnings. So no @SuppressWarnings needed on the 
callers.


[0]https://docs.oracle.com/javase/specs/jls/se8/html/jls-9.html#jls-9.6.4.6



--
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 12:18:55 +0000
Gerrit-HasComments: Yes

Reply via email to