rdblue commented on code in PR #6799:
URL: https://github.com/apache/iceberg/pull/6799#discussion_r1111313870
##########
core/src/test/java/org/apache/iceberg/TableTestBase.java:
##########
@@ -162,6 +167,19 @@ public class TableTestBase {
static final FileIO FILE_IO = new TestTables.LocalFileIO();
+ // Mapping of Avro codec name used by Iceberg to name used by Avro (and
appearing in Avro metadata
+ // under the key, avro.codec).
+ // In tests, we use the Iceberg name to specify the codec, and we verify the
codec used by reading
+ // the Avro metadata and checking for the Avro name in avro.codec.
+ static final Map<String, String> AVRO_CODEC_NAME_MAPPING =
+ ImmutableMap.<String, String>builder()
+ .put(Avro.Codec.UNCOMPRESSED.name(), DataFileConstants.NULL_CODEC)
+ .put(Avro.Codec.ZSTD.name(), DataFileConstants.ZSTANDARD_CODEC)
+ .put(Avro.Codec.GZIP.name(), DataFileConstants.DEFLATE_CODEC)
+ .build();
+
+ static final long SNAPSHOT_ID = 987134631982734L;
Review Comment:
Actually, I see some tests with `DUMMY_SNAPSHOT_ID`. Can you reuse that
instead of making a new one? You may need to move it here.
--
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]
