Skip to content

Commit

Permalink
Change the default checksum to match hadoop fs
Browse files Browse the repository at this point in the history
Although CRC32 is the default in the proto definitions, the java client sets it to CRC32C.
  • Loading branch information
colinmarc committed Jul 11, 2023
1 parent 77b16e0 commit 325f0cf
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion internal/transfer/block_writer.go
Original file line number Diff line number Diff line change
Expand Up @@ -218,7 +218,7 @@ func (bw *BlockWriter) writeBlockWriteRequest(w io.Writer) error {
MaxBytesRcvd: proto.Uint64(uint64(bw.Offset)),
LatestGenerationStamp: proto.Uint64(uint64(bw.generationTimestamp())),
RequestedChecksum: &hdfs.ChecksumProto{
Type: hdfs.ChecksumTypeProto_CHECKSUM_CRC32.Enum(),
Type: hdfs.ChecksumTypeProto_CHECKSUM_CRC32C.Enum(),
BytesPerChecksum: proto.Uint32(outboundChunkSize),
},
}
Expand Down

0 comments on commit 325f0cf

Please sign in to comment.