Minor tidy ups and test fix

Author: adamreeve

parquet/src/arrow/async_reader/mod.rs

@@ -216,8 +216,7 @@ impl<T: AsyncRead + AsyncSeek + Unpin + Send> AsyncFileReader for T {
             let mut buf = Vec::with_capacity(metadata_len);
             self.take(metadata_len as _).read_to_end(&mut buf).await?;
 
-            let parquet_metadata_reader = ParquetMetaDataReader::decode_metadata(&buf)?;
-            Ok(Arc::new(parquet_metadata_reader))
+            Ok(Arc::new(ParquetMetaDataReader::decode_metadata(&buf)?))
         }
         .boxed()
     }

parquet/src/file/metadata/reader.rs

@@ -679,8 +679,8 @@ impl ParquetMetaDataReader {
     /// file. The format of `buf` is the Thrift compact binary protocol, as specified
     /// by the [Parquet Spec].
     ///
-    /// This method handles using either decode_footer of
-    /// decode_metadata_with_encryption depending on whether the encryption
+    /// This method handles using either `decode_metadata` or
+    /// `decode_metadata_with_encryption` depending on whether the encryption
     /// feature is enabled.
     ///
     /// [Parquet Spec]: https://github.com/apache/parquet-format#metadata
@@ -698,7 +698,9 @@ impl ParquetMetaDataReader {
         #[cfg(not(feature = "encryption"))]
         let result = {
             if footer_tail.is_encrypted_footer() {
-                Err(general_err!("Parquet error: Parquet file has an encrypted footer but the encryption feature is disabled"))
+                Err(general_err!(
+                    "Parquet file has an encrypted footer but the encryption feature is disabled"
+                ))
             } else {
                 Self::decode_metadata(buf)
             }