@@ -17,9 +17,13 @@
 
 #include "parquet/metadata.h"
 
+#include <gmock/gmock.h>
 #include <gtest/gtest.h>
 
+#include <cstring>
+
 #include "arrow/util/key_value_metadata.h"
+#include "parquet/exception.h"
 #include "parquet/file_reader.h"
 #include "parquet/file_writer.h"
 #include "parquet/schema.h"
@@ -344,6 +348,23 @@ void AssertColumnChunkMinMax(const FileMetaData& metadata,
 
 }  // namespace
 
+TEST(Metadata, UnknownCompressionCodecThrows) {
+  format::FileMetaData thrift_metadata = SingleInt32MetadataWithStats();
+  auto metadata = ParseMetadata(SerializeMetadata(thrift_metadata));
+
+  auto& column_chunk = thrift_metadata.row_groups.at(0).columns.at(0);
+  const int32_t raw_codec = 8;
+  static_assert(sizeof(column_chunk.meta_data.codec) == sizeof(raw_codec));
+  std::memcpy(&column_chunk.meta_data.codec, &raw_codec, sizeof(raw_codec));
+
+  auto column = ColumnChunkMetaData::Make(&column_chunk, metadata->schema()->Column(0));
+
+  EXPECT_THROW_THAT([&]() { column->compression(); }, ParquetException,
+                    ::testing::Property(
+                        &ParquetException::what,
+                        ::testing::HasSubstr("Unsupported compression codec value: 8")));
+}
+
 TEST(Metadata, UnknownColumnOrderIgnoresMinMax) {
   format::FileMetaData thrift_metadata = SingleInt32MetadataWithStats();
   // Simulate an unsupported ColumnOrder value: unknown union fields are skipped by
 
@@ -227,7 +227,7 @@ inline typename Compression::type LoadEnumSafe(const format::CompressionCodec::t
   const auto max_value =
       static_cast<decltype(raw_value)>(format::CompressionCodec::LZ4_RAW);
   if (raw_value < min_value || raw_value > max_value) {
-    return Compression::UNCOMPRESSED;
+    throw ParquetException("Unsupported compression codec value: ", raw_value);
   }
   return FromThriftUnsafe(*in);
 }
Original file line number	Diff line number	Diff line change
`@@ -227,7 +227,7 @@ inline typename Compression::type LoadEnumSafe(const format::CompressionCodec::t`
`227`	`227`	`const auto max_value =`
`228`	`228`	`static_cast<decltype(raw_value)>(format::CompressionCodec::LZ4_RAW);`
`229`	`229`	`if (raw_value < min_value \|\| raw_value > max_value) {`
`230`		`- return Compression::UNCOMPRESSED;`
	`230`	`+ throw ParquetException("Unsupported compression codec value: ", raw_value);`
`231`	`231`	`}`
`232`	`232`	`return FromThriftUnsafe(*in);`
`233`	`233`	`}`