cleanup

aws-s3-transfer-manager/src/operation/download.rs

@@ -174,7 +174,7 @@ fn handle_discovery_chunk(
                 .map(|aggregated| ChunkResponse {
                     seq,
                     data: aggregated,
-                    metadata: metadata.expect("metadata is available"),
+                    metadata: metadata.expect("chunk metadata is available"),
                 })
                 .map_err(error::discovery_failed);
 

aws-s3-transfer-manager/src/operation/download/body.rs

@@ -15,14 +15,13 @@ use crate::error::ErrorKind;
 
 use super::chunk_meta::ChunkMetadata;
 
-#[derive(Debug, Clone)]
-
 /// Non-contiguous Binary Data Storage
 ///
 /// When data is read from the network, it is read in a sequence of chunks that are not in
 /// contiguous memory. [`AggregatedBytes`](crate::byte_stream::AggregatedBytes) provides a view of
 /// this data via [`impl Buf`](bytes::Buf) or it can be copied into contiguous storage with
 /// [`.into_bytes()`](crate::byte_stream::AggregatedBytes::into_bytes).
+#[derive(Debug, Clone)]
 pub struct AggregatedBytes(SegmentedBuf<Bytes>);
 
 impl AggregatedBytes {
@@ -56,7 +55,7 @@ impl AggregatedBytes {
         while let Some(buf) = value.next().await {
             match buf {
                 Ok(buf) => output.push(buf),
-                Err(err) => return Err(crate::error::from_kind(ErrorKind::IOError)(err)),
+                Err(err) => return Err(crate::error::from_kind(ErrorKind::ChunkFailed)(err)),
             };
         }
         Ok(AggregatedBytes(output))

aws-s3-transfer-manager/src/operation/download/object_meta.rs

@@ -116,7 +116,6 @@ impl From<HeadObjectOutput> for ObjectMetadata {
             extended_request_id: value.extended_request_id().map(|s| s.to_string()),
             request_charged: value.request_charged,
             delete_marker: value.delete_marker,
-            // accept_ranges: value.accept_ranges,
             expiration: value.expiration,
             restore: value.restore,
             last_modified: value.last_modified,