@@ -67,8 +67,8 @@ TabletMetadataPB make_metadata() {
6767
6868// How a damaged buffer looks after the exact parse load() performs.
6969struct Outcome {
70- bool parses = false ; // ParseFromString succeeded == load() returns OK
71- int rowsets = 0 ; // rowset list size
70+ bool parses = false ; // ParseFromString succeeded == load() returns OK
71+ int rowsets = 0 ; // rowset list size
7272 bool has_compact_page = false ; // delvec_meta still carries rowset 2263372's page
7373 int delvecs = 0 ;
7474};
@@ -120,9 +120,9 @@ TEST_F(LakeMetadataCorruptionTest, in_place_byte_corruption_silently_drops_delve
120120 ASSERT_TRUE (is_dangerous (classify (clean)) == false ); // sanity: clean copy is fine
121121 ASSERT_TRUE (classify (clean).has_compact_page );
122122
123- int parse_fail = 0 ; // protobuf itself rejected it (would still NOT be a checksum, but rejected)
124- int parse_ok_same = 0 ; // parsed, delvec page still present
125- int dangerous = 0 ; // parsed, rowsets intact, delvec page lost <-- the incident shape
123+ int parse_fail = 0 ; // protobuf itself rejected it (would still NOT be a checksum, but rejected)
124+ int parse_ok_same = 0 ; // parsed, delvec page still present
125+ int dangerous = 0 ; // parsed, rowsets intact, delvec page lost <-- the incident shape
126126 std::vector<size_t > dangerous_offsets;
127127
128128 // Try a few mutations per byte so we exercise tag/length/value damage.
@@ -146,8 +146,8 @@ TEST_F(LakeMetadataCorruptionTest, in_place_byte_corruption_silently_drops_delve
146146 }
147147
148148 std::cerr << " [corruption sweep] file_size=" << clean.size () << " parse_fail=" << parse_fail
149- << " parse_ok_other=" << parse_ok_same << " dangerous(parse_ok+rowsets_intact+page_lost)="
150- << dangerous << std::endl;
149+ << " parse_ok_other=" << parse_ok_same << " dangerous(parse_ok+rowsets_intact+page_lost)=" << dangerous
150+ << std::endl;
151151
152152 // The point: such silent-loss corruptions DO exist on this format.
153153 ASSERT_GT (dangerous, 0 ) << " expected at least one byte corruption that parses cleanly yet loses "
@@ -182,8 +182,7 @@ TEST_F(LakeMetadataCorruptionTest, in_place_byte_corruption_silently_drops_delve
182182 EXPECT_EQ (2 , loaded.rowsets_size ()) << " rowset list should survive the corruption" ;
183183 EXPECT_FALSE (loaded.delvec_meta ().delvecs ().contains (kRowsetCompact ))
184184 << " rowset 2263372's delvec page should be silently gone" ;
185- std::cerr << " [end-to-end] corrupted byte at offset " << off
186- << " -> load() OK, rowsets=" << loaded.rowsets_size ()
185+ std::cerr << " [end-to-end] corrupted byte at offset " << off << " -> load() OK, rowsets=" << loaded.rowsets_size ()
187186 << " , delvecs=" << loaded.delvec_meta ().delvecs ().size () << std::endl;
188187}
189188
0 commit comments