fix: preserve compact locality by default

Signed-off-by: Ethan Urbanski <ethan@urbanskitech.com>

Author: ethan-tyler

crates/core/src/kernel/snapshot/mod.rs

@@ -365,7 +365,7 @@ impl Snapshot {
     ///
     /// ## Returns
     ///
-    /// A stream of [`LogicalFileView`] objects.
+    /// A stream of [`LogicalFileView`] objects, newest first.
     pub fn file_views(
         &self,
         log_store: &dyn LogStore,
@@ -779,7 +779,7 @@ impl EagerSnapshot {
     ///
     /// ## Returns
     ///
-    /// A stream of [`LogicalFileView`] objects.
+    /// A stream of [`LogicalFileView`] objects, newest first.
     pub fn file_views(
         &self,
         log_store: &dyn LogStore,
@@ -1095,6 +1095,31 @@ mod tests {
         Ok(())
     }
 
+    #[tokio::test]
+    async fn test_eager_file_views_return_newest_files_first() -> TestResult {
+        let (_dir, mut table) = checkpoint_rebase_table().await?;
+
+        append_test_add(&mut table, "part-00002.snappy.parquet").await?;
+        let version = table.version().unwrap();
+        checkpoints::create_checkpoint(&table, None).await?;
+
+        let snapshot =
+            EagerSnapshot::try_new(table.log_store().as_ref(), Default::default(), None).await?;
+        assert_eq!(snapshot.version(), version);
+
+        let paths = eager_file_paths(&snapshot, table.log_store().as_ref()).await?;
+        assert_eq!(
+            paths,
+            vec![
+                "part-00002.snappy.parquet",
+                "part-00001.snappy.parquet",
+                "part-00000.snappy.parquet",
+            ]
+        );
+
+        Ok(())
+    }
+
     #[tokio::test]
     async fn test_snapshot_update_explicit_same_version_adopts_late_checkpoint() -> TestResult {
         let (_dir, table) = checkpoint_rebase_table().await?;