thswlsqls opened a new pull request, #17244: URL: https://github.com/apache/iceberg/pull/17244
## Summary - `DynFields` was copied from `DynMethods`, and several Javadoc comments kept the original `DynMethods` wording. - The class comment said callers "invoke the wrapped method"; the class wraps a `java.lang.reflect.Field`. - `bind(Object)`, `buildChecked(Object)` and `build(Object)` said they return a `BoundMethod`; they return a `BoundField`. - `buildChecked(Object)` said it throws `NoSuchMethodException`; its signature throws `NoSuchFieldException`. - `@throws` and `@param` text said "method"/"receiver" where the code binds a field to a target, and `asStatic()`/`buildStatic()` said "if the method is not static". - Corrected these 12 lines to match the signatures; the sibling `DynMethods` the comments came from shows the correct terms. Javadoc-only, no signature or bytecode change. ## Testing done - No test added — wording only, no behavior change. - `./gradlew :iceberg-common:spotlessApply :iceberg-common:spotlessCheck` — BUILD SUCCESSFUL (JDK 21). -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
