about summary refs log tree commit diff
diff options
context:
space:
mode:
authorFlorian Klink <flokli@flokli.de>2023-05-17T11·21+0300
committerclbot <clbot@tvl.fyi>2023-05-17T13·06+0000
commit3a4e29c26141a24caa71a0dbaf40a6f8d1c2adef (patch)
tree32e2b8d73e38ea838914cdb27c7d7bca412d8581
parenta6c7869393930c667e837b99cd36e0d8e9eb2498 (diff)
refactor(tvix/store): rename import::{import_path -> ingest_path} r/6149
This distinguishes it better from the EvalIO::import_path method.
Also update the docstring to explain what it does (and what it doesn't).

Change-Id: I32a8b2869fa67a894df28532b22bf170961a2abf
Reviewed-on: https://cl.tvl.fyi/c/depot/+/8578
Reviewed-by: tazjin <tazjin@tvl.su>
Autosubmit: flokli <flokli@flokli.de>
Tested-by: BuildkiteCI
-rw-r--r--tvix/store/src/bin/tvix-store.rs4
-rw-r--r--tvix/store/src/import.rs13
-rw-r--r--tvix/store/src/tests/import.rs8
3 files changed, 13 insertions, 12 deletions
diff --git a/tvix/store/src/bin/tvix-store.rs b/tvix/store/src/bin/tvix-store.rs
index 29f8a92cb1be..807cc07643d1 100644
--- a/tvix/store/src/bin/tvix-store.rs
+++ b/tvix/store/src/bin/tvix-store.rs
@@ -9,7 +9,7 @@ use std::path::PathBuf;
 use tracing_subscriber::prelude::*;
 use tvix_store::blobservice::SledBlobService;
 use tvix_store::directoryservice::SledDirectoryService;
-use tvix_store::import::import_path;
+use tvix_store::import::ingest_path;
 use tvix_store::nar::NARCalculationService;
 use tvix_store::nar::NonCachingNARCalculationService;
 use tvix_store::pathinfoservice::SledPathInfoService;
@@ -135,7 +135,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
             );
 
             for path in paths {
-                let root_node = import_path(&mut blob_service, &mut directory_service, &path)?;
+                let root_node = ingest_path(&mut blob_service, &mut directory_service, &path)?;
 
                 let nar_hash = NixHashWithMode::Recursive(NixHash::new(
                     HashAlgo::Sha256,
diff --git a/tvix/store/src/import.rs b/tvix/store/src/import.rs
index bf80eb4b71b9..ade700a6c283 100644
--- a/tvix/store/src/import.rs
+++ b/tvix/store/src/import.rs
@@ -136,14 +136,15 @@ fn process_entry<BS: BlobService, DP: DirectoryPutter>(
     todo!("handle other types")
 }
 
-/// Imports the contents at a given Path into the tvix store.
+/// Ingests the contents at the given path into the tvix store,
+/// interacting with a [BlobService] and [DirectoryService].
+/// It returns the root node or an error.
 ///
-/// It doesn't register the contents at a Path in the store itself, that's up
-/// to the PathInfoService.
-//
-// returns the root node, or an error.
+/// It's not interacting with a [PathInfoService], it's up to the caller to
+/// possibly register it somewhere (and potentially rename it based on some
+/// naming scheme.
 #[instrument(skip(blob_service, directory_service), fields(path=?p))]
-pub fn import_path<BS: BlobService, DS: DirectoryService, P: AsRef<Path> + Debug>(
+pub fn ingest_path<BS: BlobService, DS: DirectoryService, P: AsRef<Path> + Debug>(
     blob_service: &mut BS,
     directory_service: &mut DS,
     p: P,
diff --git a/tvix/store/src/tests/import.rs b/tvix/store/src/tests/import.rs
index ed5154d6fec0..ef4ef74d09f7 100644
--- a/tvix/store/src/tests/import.rs
+++ b/tvix/store/src/tests/import.rs
@@ -1,7 +1,7 @@
 use super::utils::{gen_blob_service, gen_directory_service};
 use crate::blobservice::BlobService;
 use crate::directoryservice::DirectoryService;
-use crate::import::import_path;
+use crate::import::ingest_path;
 use crate::proto;
 use crate::tests::fixtures::DIRECTORY_COMPLICATED;
 use crate::tests::fixtures::*;
@@ -19,7 +19,7 @@ fn symlink() {
     )
     .unwrap();
 
-    let root_node = import_path(
+    let root_node = ingest_path(
         &mut gen_blob_service(),
         &mut gen_directory_service(),
         tmpdir.path().join("doesntmatter"),
@@ -43,7 +43,7 @@ fn single_file() {
 
     let mut blob_service = gen_blob_service();
 
-    let root_node = import_path(
+    let root_node = ingest_path(
         &mut blob_service,
         &mut gen_directory_service(),
         tmpdir.path().join("root"),
@@ -82,7 +82,7 @@ fn complicated() {
     let mut blob_service = gen_blob_service();
     let mut directory_service = gen_directory_service();
 
-    let root_node = import_path(&mut blob_service, &mut directory_service, tmpdir.path())
+    let root_node = ingest_path(&mut blob_service, &mut directory_service, tmpdir.path())
         .expect("must succeed");
 
     // ensure root_node matched expectations