yihua commented on code in PR #10872: URL: https://github.com/apache/hudi/pull/10872#discussion_r1597306796
########## hudi-utilities/src/main/java/org/apache/hudi/utilities/sources/JsonKafkaSource.java: ########## @@ -57,6 +57,8 @@ */ public class JsonKafkaSource extends KafkaSource<String> { + private static final ObjectMapper OBJECT_MAPPER = new ObjectMapper(); Review Comment: Good point, I think we should revert this change, because anyway the `ObjectMapper` is serde from Spark driver to executor if using the static object, so there's not much gain. -- 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...@hudi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org