Remove allow unused

Author: OussamaSaoudi

kernel/src/actions/visitors.rs

@@ -166,7 +166,6 @@ impl RowVisitor for ProtocolVisitor {
     }
 }
 
-#[allow(unused)]
 #[derive(Default)]
 #[cfg_attr(feature = "developer-visibility", visibility::make(pub))]
 pub(crate) struct AddVisitor {
@@ -240,7 +239,6 @@ impl RowVisitor for AddVisitor {
     }
 }
 
-#[allow(unused)]
 #[derive(Default)]
 #[cfg_attr(feature = "developer-visibility", visibility::make(pub))]
 pub(crate) struct RemoveVisitor {
@@ -315,7 +313,6 @@ impl RowVisitor for RemoveVisitor {
     }
 }
 
-#[allow(unused)]
 #[derive(Default)]
 #[cfg_attr(feature = "developer-visibility", visibility::make(pub))]
 #[cfg_attr(not(feature = "developer-visibility"), visibility::make(pub(crate)))]

kernel/src/path.rs

@@ -20,20 +20,12 @@ const UUID_PART_LEN: usize = 36;
 enum LogPathFileType {
     Commit,
     SinglePartCheckpoint,
-    #[allow(unused)]
     UuidCheckpoint(String),
     // NOTE: Delta spec doesn't actually say, but checkpoint part numbers are effectively 31-bit
     // unsigned integers: Negative values are never allowed, but Java integer types are always
     // signed. Approximate that as u32 here.
-    #[allow(unused)]
-    MultiPartCheckpoint {
-        part_num: u32,
-        num_parts: u32,
-    },
-    #[allow(unused)]
-    CompactedCommit {
-        hi: Version,
-    },
+    MultiPartCheckpoint { part_num: u32, num_parts: u32 },
+    CompactedCommit { hi: Version },
     Unknown,
 }
 
@@ -42,9 +34,7 @@ enum LogPathFileType {
 #[cfg_attr(not(feature = "developer-visibility"), visibility::make(pub(crate)))]
 struct ParsedLogPath<Location: AsUrl = FileMeta> {
     pub location: Location,
-    #[allow(unused)]
     pub filename: String,
-    #[allow(unused)]
     pub extension: String,
     pub version: Version,
     pub file_type: LogPathFileType,

kernel/tests/common/mod.rs

@@ -84,8 +84,6 @@ pub(crate) fn test_read(
     Ok(())
 }
 
-// TODO (zach): this is listed as unused for acceptance crate
-#[allow(unused)]
 pub(crate) fn read_scan(scan: &Scan, engine: Arc<dyn Engine>) -> DeltaResult<Vec<RecordBatch>> {
     let scan_results = scan.execute(engine)?;
     scan_results