Minor fix to BEP key encoding

LifetimeEvents and BuildEvents were racing to who would get the entry first. This partitions the data.
TraceMachina · Dec 13, 2024 · 6d07adb · 6d07adb
1 parent 3f3d4e2
commit 6d07adb
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/nativelink-service/src/bep_server.rs b/nativelink-service/src/bep_server.rs
@@ -82,7 +82,7 @@ impl BepServer {
         let sequence_number = build_event.sequence_number;
 
         let store_key = StoreKey::Str(Cow::Owned(format!(
-            "BepEvent:{}:{}:{}",
+            "BepEvent:le:{}:{}:{}",
             &stream_id.build_id, &stream_id.invocation_id, sequence_number,
         )));
 
@@ -140,7 +140,7 @@ impl BepServer {
             store
                 .update_oneshot(
                     StoreKey::Str(Cow::Owned(format!(
-                        "BepEvent:{}:{}:{}",
+                        "BepEvent:be:{}:{}:{}",
                         &stream_id.build_id, &stream_id.invocation_id, sequence_number,
                     ))),
                     buf.freeze(),

diff --git a/nativelink-service/tests/bep_server_test.rs b/nativelink-service/tests/bep_server_test.rs
@@ -117,7 +117,7 @@ async fn publish_lifecycle_event_test() -> Result<(), Box<dyn std::error::Error>
     let sequence_number = request.clone().build_event.unwrap().sequence_number;
 
     let store_key = StoreKey::Str(Cow::Owned(format!(
-        "BepEvent:{}:{}:{}",
+        "BepEvent:le:{}:{}:{}",
         stream_id.clone().build_id,
         stream_id.clone().invocation_id,
         sequence_number
@@ -287,7 +287,7 @@ async fn publish_build_tool_event_stream_test() -> Result<(), Box<dyn std::error
                 .iter()
                 .map(|request| {
                     StoreKey::Str(Cow::Owned(format!(
-                        "BepEvent:{}:{}:{}",
+                        "BepEvent:be:{}:{}:{}",
                         stream_id.build_id,
                         stream_id.invocation_id,
                         request