Add file error handling and structure handlers factory method

cpp/backend/common/BUILD

@@ -35,6 +35,10 @@ cc_library(
     deps = [
         ":page_id",
         ":page",
+        "//common:status_util",
+        "@com_google_absl//absl/status:status",
+        "@com_google_absl//absl/status:statusor",
+        "@com_google_absl//absl/strings:str_format",
     ],
     visibility = ["//backend:__subpackages__"],
 )
@@ -45,6 +49,7 @@ cc_test(
     deps = [
         ":file",
         "//common:file_util",
+        "//common:status_test_util",
         "@com_google_googletest//:gtest_main",
     ],
 )
@@ -55,7 +60,9 @@ cc_binary(
     deps = [
         ":file",
         "//common:file_util",
+        "//common:status_test_util",
         "@com_github_google_benchmark//:benchmark_main",
+        "@com_google_absl//absl/status:statusor",
         "//third_party/gperftools:profiler",
     ],
     testonly = True,
@@ -99,6 +106,9 @@ cc_library(
         ":file",
         ":page_id",
         "//common:memory_usage",
+        "//common:status_util",
+        "@com_google_absl//absl/status:status",
+        "@com_google_absl//absl/status:statusor",
         "@com_google_absl//absl/container:flat_hash_map",
     ],
     visibility = ["//backend:__subpackages__"],
@@ -110,6 +120,7 @@ cc_test(
     deps = [
         ":page",
         ":page_pool",
+        "//common:status_test_util",
         "@com_google_googletest//:gtest_main",
     ],
 )
@@ -121,7 +132,11 @@ cc_binary(
         ":access_pattern",
         ":eviction_policy",
         ":page_pool",
+        "//common:status_test_util",
         "@com_github_google_benchmark//:benchmark_main",
+        "@com_google_absl//absl/status:status",
+        "@com_google_absl//absl/status:statusor",
         "//third_party/gperftools:profiler",
     ],
+    testonly = True,
 )

cpp/backend/common/access_pattern.h

@@ -24,7 +24,7 @@ class Sequential {
   std::size_t next_;
 };
 
-// Simulates an uniformely distributed access pattern to a range of
+// Simulates an uniformly distributed access pattern to a range of
 // [0,...,size).
 class Uniform {
  public:

cpp/backend/common/eviction_policy.cc

@@ -11,7 +11,7 @@ namespace carmen::backend {
 
 namespace {
 
-// Selects a alement for the given set to be evicted according to the provided
+// Selects an element for the given set to be evicted according to the provided
 // eviction pattern. The EvictionPattern needs to provide a Next() function
 // providing a random slot to be evicted. The next higher value present in the
 // set is then chosen to be evicted.
@@ -106,7 +106,7 @@ void LeastRecentlyUsedEvictionPolicy::Read(std::size_t position) {
 }
 
 void LeastRecentlyUsedEvictionPolicy::Written(std::size_t position) {
-  // This policy does not distinguish between read an writes.
+  // This policy does not distinguish between read and writes.
   Read(position);
 }
 

cpp/backend/common/eviction_policy_benchmark.cc

@@ -59,7 +59,7 @@ BENCHMARK(BM_UniformWriteTest<RandomEvictionPolicy>)
 BENCHMARK(BM_UniformWriteTest<LeastRecentlyUsedEvictionPolicy>)
     ->Range(kMinPoolSize, kMaxPoolSize);
 
-// Evalutes the performance of removing elements from the pool.
+// Evaluates the performance of removing elements from the pool.
 template <EvictionPolicy Policy>
 void BM_UniformRemoveTest(benchmark::State& state) {
   auto pool_size = state.range(0);
@@ -81,7 +81,7 @@ BENCHMARK(BM_UniformRemoveTest<RandomEvictionPolicy>)
 BENCHMARK(BM_UniformRemoveTest<LeastRecentlyUsedEvictionPolicy>)
     ->Range(kMinPoolSize, kMaxPoolSize);
 
-// Evalutes the performance of selecting pages to be evicted.
+// Evaluates the performance of selecting pages to be evicted.
 template <EvictionPolicy Policy>
 void BM_GetPageToEvictTest(benchmark::State& state) {
   auto pool_size = state.range(0);