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

Reply via email to