qijiale76 opened a new pull request, #2445: URL: https://github.com/apache/uniffle/pull/2445
### What changes were proposed in this pull request? This PR corrects the reflection logic in `RssDAGAppMaster.DagInitialCallback` when setting the `className` for `InputDescriptor`. It now correctly obtains the `Field` object directly from `InputDescriptor`'s class hierarchy, instead of incorrectly reusing the `Field` object derived from `OutputDescriptor`. ### Why are the changes needed? The previous code mistakenly used the `Field` for `className` obtained from `OutputDescriptor` to modify the `InputDescriptor`. This **happened to work** likely because both `InputDescriptor` and `OutputDescriptor` inherit the `className` field from a common superclass. Java's reflection allows applying a `Field` object to any instance that possesses that field (often through inheritance), even if the `Field` was retrieved via a different related class. This fix ensures the reflection is used correctly and explicitly for `InputDescriptor`, improving code clarity, robustness, and maintainability. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Existing tests. -- 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]
