Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/12213#discussion_r58791357
--- Diff:
core/src/main/scala/org/apache/spark/status/api/v1/JacksonMessageWriter.scala
---
@@ -47,7 +47,6 @@ private[v1] class JacksonMessageWriter extends
MessageBodyWriter[Object]{
super.writeValueAsString(t)
}
}
-
mapper.registerModule(com.fasterxml.jackson.module.scala.DefaultScalaModule)
--- End diff --
This was added by @squito in #5940 in order to allow SparkStatusAPI POJOs
to be serialized using Jackson. These POJOs are defined in
https://github.com/apache/spark/blob/a4ead6d3881f071a2ae53ff1c961c6ac388cac1d/core/src/main/scala/org/apache/spark/status/api/v1/api.scala
I wonder if the default values in the `ApplicationAttemptInfo` are going to
be handled differently without the `jackson-module-scala` stuff. We might have
to explicitly add annotations in order to pin those defaults.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
