kfaraz opened a new pull request, #16217: URL: https://github.com/apache/druid/pull/16217
The serialization of a `Map<String, TaskReport>` was originally fixed in https://github.com/apache/druid/pull/12938. Essentially, while serializing a `Map` or even a `List` containing `TaskReport` objects, the `type` information is lost. Thus, the object cannot be serialized back. This is a known issue with Jackson. The way this has been tackled in the Druid code till now is: - Use special logic in `SingleFileTaskReportFileWriter.writeReportToStream()` - For live reports, build a raw map instead of a concrete `TaskReport` object This PR attempts to simplify that solution by adding a new `TaskReport.ReportMap` class. ### Changes - Add class `TaskReport.ReportMap` - Have `TaskReport.buildTaskReports()` build the new class - Add serde tests for writing to string, file for the known task report types -- 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: commits-unsubscr...@druid.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
