bvaradar commented on a change in pull request #1278: [HUDI-573] Refactoring getter to avoid double extrametadata in json representation of HoodieCommitMetadata URL: https://github.com/apache/incubator-hudi/pull/1278#discussion_r382159855
########## File path: hudi-common/src/main/java/org/apache/hudi/common/model/HoodieCommitMetadata.java ########## @@ -47,15 +47,15 @@ protected Map<String, List<HoodieWriteStat>> partitionToWriteStats; protected Boolean compacted; - private Map<String, String> extraMetadataMap; + private Map<String, String> extraMetadata; Review comment: I was asking about any issues around storing extra-metadata in commit files in active timeline. For example, As we store them in json format in active timeline, wanted to check if DeltaStreamer can read checkpoints without any problem from the json file. But,,having thought about it, this should be fine. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
