Fix to the NA in the cluster calls

darrenge · Sep 6, 2024 · 86d8742 · 86d8742
1 parent 9a22ced
commit 86d8742
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/benchmark/BDN.benchmark/Cluster/RespClusterBench.cs b/benchmark/BDN.benchmark/Cluster/RespClusterBench.cs
@@ -2,6 +2,7 @@
 // Licensed under the MIT license.
 
 using System.Runtime.CompilerServices;
+using System.Runtime.InteropServices;
 using System.Text;
 using BenchmarkDotNet.Attributes;
 using Embedded.perftest;
@@ -48,6 +49,7 @@ public void SetupSingleInstance()
                 EnableCluster = true,
                 Port = Port,
                 CleanClusterConfig = true,
+                CheckpointDir = RuntimeInformation.IsOSPlatform(OSPlatform.Linux) ? "/tmp" : null
             };
             server = new EmbeddedRespServer(opt);
             session = server.GetRespSession();

diff --git a/test/BDNPerfTests/run_bdnperftest.ps1 b/test/BDNPerfTests/run_bdnperftest.ps1
@@ -77,14 +77,6 @@ param ($ResultsLine, $columnNum)
     $foundValue = $column[$columnNum].Trim(' us') 
     $foundValue = $column[$columnNum].Trim(' ns')  
 
-    # Just double check found results is not NA
-    if ($foundResultsValue -eq "NA") {
-        Write-Error -Message "**   << Data Error! >>  The  BDN benchmark Mean Value is not a valid value.  Mean value found: $dblfoundResultValue"
-        Write-Host "** "
-
-        return 0
-    }
-
     return $foundValue
 }