refactor: rename src/server/rpc/storage.rs to src/server/rpc/service.rs (#513)

* refactor: rename src/server/rpc/storage.rs src/server/rpc/service.rs

* refactor: update references

src/commands/server.rs

@@ -6,7 +6,7 @@ use std::sync::Arc;
 use std::{env::VarError, path::PathBuf};
 
 use crate::server::http_routes;
-use crate::server::rpc::storage;
+use crate::server::rpc::service;
 
 use hyper::service::{make_service_fn, service_fn};
 use hyper::Server;
@@ -50,7 +50,7 @@ pub enum Error {
 
     #[snafu(display("Error serving RPC: {}", source))]
     ServingRPC {
-        source: crate::server::rpc::storage::Error,
+        source: crate::server::rpc::service::Error,
     },
 }
 
@@ -107,7 +107,7 @@ pub async fn main() -> Result<()> {
         .await
         .expect("failed to bind server");
 
-    let grpc_server = storage::make_server(socket, storage.clone(), executor);
+    let grpc_server = service::make_server(socket, storage.clone(), executor);
 
     info!("gRPC server listening on http://{}", grpc_bind_addr);
 

src/server/rpc.rs

@@ -11,4 +11,4 @@ pub(crate) const TAG_KEY_FIELD: &[u8] = &[255];
 pub mod data;
 pub mod expr;
 pub mod input;
-pub mod storage;
+pub mod service;

src/server/rpc/service.rs

@@ -1736,7 +1736,7 @@ mod tests {
         // Note we don't include the actual line / column in the
         // expected panic message to avoid needing to update the test
         // whenever the source code file changed.
-        let expected_error = "panicked at 'This is a test panic', src/server/rpc/storage.rs";
+        let expected_error = "panicked at 'This is a test panic', src/server/rpc/service.rs";
         assert!(
             captured_logs.contains(expected_error),
             "Logs did not contain expected panic message '{}'. They were\n{}",