RussellSpitzer commented on code in PR #11421:
URL: https://github.com/apache/iceberg/pull/11421#discussion_r1821458413


##########
core/src/test/java/org/apache/iceberg/TestSnapshotJson.java:
##########
@@ -176,4 +176,41 @@ private String createManifestListWithManifestFiles(long 
snapshotId, Long parentS
 
     return localInput(manifestList).location();
   }
+
+  @Test
+  public void testJsonConversionSummaryWithoutOperation() {

Review Comment:
   May be paranoid but I'd like to leave a mark on the test that the behavior 
here is out of spec, something so that readers understand this isn't a required 
behavior



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to