nastra commented on code in PR #8723: URL: https://github.com/apache/iceberg/pull/8723#discussion_r1347392925
########## core/src/main/java/org/apache/iceberg/view/ViewMetadataParser.java: ########## @@ -66,7 +67,9 @@ static void toJson(ViewMetadata metadata, JsonGenerator gen) throws IOException gen.writeStringField(VIEW_UUID, metadata.uuid()); gen.writeNumberField(FORMAT_VERSION, metadata.formatVersion()); gen.writeStringField(LOCATION, metadata.location()); - JsonUtil.writeStringMap(PROPERTIES, metadata.properties(), gen); + if (!metadata.properties().isEmpty()) { Review Comment: @amogh-jahagirdar in newer parsers we typically try to avoid sending empty collections, hence the suggested change. We could still send an empty collection here for views, but we still need to adjust the read-side to not require properties -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org