clean up

Author: CTTY

crates/iceberg/src/catalog/memory/catalog.rs

@@ -128,7 +128,7 @@ impl MemoryCatalog {
         config: MemoryCatalogConfig,
         storage_factory: Option<Arc<dyn StorageFactory>>,
     ) -> Result<Self> {
-        // Use provided factory or default to MemoryStorageFactory
+        // Use provided factory or default to LocalFsStorageFactory
         let factory = storage_factory.unwrap_or_else(|| Arc::new(LocalFsStorageFactory));
 
         Ok(Self {

crates/iceberg/src/io/file_io.rs

@@ -199,19 +199,6 @@ pub struct FileIOBuilder {
 
 impl FileIOBuilder {
     /// Creates a new builder with the given storage factory.
-    ///
-    /// # Arguments
-    ///
-    /// * `factory` - The storage factory to use for creating storage instances
-    ///
-    /// # Example
-    ///
-    /// ```rust,ignore
-    /// use iceberg::io::{FileIOBuilder, LocalFsStorageFactory};
-    /// use std::sync::Arc;
-    ///
-    /// let builder = FileIOBuilder::new(Arc::new(LocalFsStorageFactory));
-    /// ```
     pub fn new(factory: Arc<dyn StorageFactory>) -> Self {
         Self {
             factory,
@@ -242,9 +229,6 @@ impl FileIOBuilder {
     }
 
     /// Builds [`FileIO`].
-    ///
-    /// Note: Storage is lazily initialized on first use, so this method
-    /// does not return a Result.
     pub fn build(self) -> FileIO {
         FileIO {
             config: self.config,

crates/iceberg/tests/file_io_s3_test.rs

@@ -122,20 +122,6 @@ mod tests {
         }
     }
 
-    #[test]
-    fn test_file_io_builder_extension_system() {
-        // Test adding and retrieving extensions
-        // Note: Extension system is not part of the new FileIOBuilder API
-        // This test is removed as the extension system was part of the old API
-    }
-
-    #[test]
-    fn test_file_io_builder_multiple_extensions() {
-        // Test adding multiple different types of extensions
-        // Note: Extension system is not part of the new FileIOBuilder API
-        // This test is removed as the extension system was part of the old API
-    }
-
     #[test]
     fn test_custom_aws_credential_loader_instantiation() {
         // Test creating CustomAwsCredentialLoader with mock loader