Merge pull request #26 from ilikepi63/chore/refactor-file-naming

chore: Refactor File Naming

Author: ilikepi63

src/batch_coordinator.rs src/batch_coordinator/mod.rssrc/batch_coordinator.rs renamed to src/batch_coordinator/mod.rs

@@ -3,6 +3,8 @@
 
 #![allow(dead_code)]
 
+pub mod simple;
+
 use std::collections::HashSet;
 use std::vec::Vec;
 

src/simple_batch_coordinator/index.rs src/batch_coordinator/simple/index.rssrc/simple_batch_coordinator/index.rs renamed to src/batch_coordinator/simple/index.rs

@@ -1,3 +1,6 @@
+mod index;
+
+use index::Index;
 use std::{
     collections::HashSet,
     fs::{File, OpenOptions},
@@ -10,7 +13,7 @@ use uuid::Uuid;
 
 use crate::{
     batch_coordinator::BatchInfo, error::RisklessResult,
-    messages::commit_batch_request::CommitBatchRequest, simple_batch_coordinator::index::Index,
+    messages::commit_batch_request::CommitBatchRequest,
 };
 
 use crate::batch_coordinator::{
@@ -264,7 +267,7 @@ impl BatchCoordinator for SimpleBatchCoordinator {
 
     /// No-op as this operation is not supported in the SimpleBatchCoordinator.
     async fn delete_files(&self, _request: DeleteFilesRequest) {}
-    
+
     /// Always returns false.
     async fn is_safe_to_delete_file(&self, _object_key: String) -> bool {
         false

src/simple_batch_coordinator/simple.rs src/batch_coordinator/simple/mod.rssrc/simple_batch_coordinator/simple.rs renamed to src/batch_coordinator/simple/mod.rs

@@ -14,7 +14,7 @@ use crate::{
         produce_request::{ProduceRequest, ProduceRequestCollection},
         produce_response::ProduceResponse,
     },
-    segment::SharedLogSegment,
+    shared_log_segment::SharedLogSegment,
     utils::request_response::Request,
 };
 

src/broker.rs

@@ -6,8 +6,7 @@ pub mod batch_coordinator;
 mod utils;
 mod broker;
 pub mod messages;
-mod segment;
-pub mod simple_batch_coordinator;
+mod shared_log_segment;
 
 pub use broker::{Broker, BrokerConfiguration};
 pub mod error;