pjfanning commented on code in PR #7623:
URL: https://github.com/apache/hadoop/pull/7623#discussion_r2225761556
##########
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/webapp/YarnJacksonJaxbJsonProvider.java:
##########
@@ -51,6 +53,9 @@ public YarnJacksonJaxbJsonProvider() {
public ObjectMapper locateMapper(Class<?> type, MediaType mediaType) {
ObjectMapper mapper = super.locateMapper(type, mediaType);
configObjectMapper(mapper);
+ if (type == TimelineAbout.class) {
Review Comment:
@steveloughran I needed to add this to fix tests. I'm not sure what has
changed in Jackson or other libs that requires this to be set here. Also, I
would be concerned that other classes might be affected but that unit tests
don't deserialize instances for those classes.
--
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: common-issues-unsubscr...@hadoop.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org
