jojochuang opened a new pull request, #8753: URL: https://github.com/apache/hadoop/pull/8753
## Summary - Mark `objectMapper` and `requestHandler` in `McpHttpServlet` as `transient` so SpotBugs no longer reports `SE_BAD_FIELD` on trunk `hadoop-common`. - Unblocks Yetus **spotbugs-strict-precheck** for PRs that touch modules depending on `hadoop-common` (e.g. HDFS-17981 / #8744). ## Context Follow-up to YARN-11977 (MCP HTTP server in hadoop-common). Trunk currently reports one extant SpotBugs warning: `McpHttpServlet` defines non-transient non-serializable instance field `requestHandler`. HttpServlet implements `Serializable`, but these servlet instances are not serialized in normal deployment; `transient` matches common servlet practice. ## Test plan - [x] `./mvnw -pl hadoop-common-project/hadoop-common -DskipTests test-compile spotbugs:spotbugs` - [ ] Yetus trunk spotbugs precheck on PR https://issues.apache.org/jira/browse/HADOOP-19993 Made with [Cursor](https://cursor.com) -- 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]
