Merge pull request #440 from HDFGroup/chogan/borg_triggers

Implement BufferOrganizer triggers
HDFGroup · Aug 25, 2022 · 0300696 · 0300696
2 parents 412c2f6 + 5f3e951
commit 0300696
Show file tree

Hide file tree

Showing 37 changed files with 1,624 additions and 188 deletions.
diff --git a/adapter/test/pubsub/pubsub_topic_test.cc b/adapter/test/pubsub/pubsub_topic_test.cc
@@ -34,4 +34,8 @@ int main(int argc, char **argv) {
 
   auto disconnect_ret = hermes::pubsub::disconnect();
   Assert(disconnect_ret.Succeeded());
+
+  MPI_Finalize();
+
+  return 0;
 }
diff --git a/adapter/test/stdio/CMakeLists.txt b/adapter/test/stdio/CMakeLists.txt
@@ -55,8 +55,11 @@ add_executable(hermes_stdio_low_buf_adapter_test stdio_adapter_low_buffer_space_
 target_link_libraries(hermes_stdio_low_buf_adapter_test hermes_stdio)
 add_dependencies(hermes_stdio_low_buf_adapter_test hermes_stdio hermes_daemon)
 set_target_properties(hermes_stdio_low_buf_adapter_test PROPERTIES COMPILE_FLAGS "-DHERMES_INTERCEPT=1")
-gcc_hermes(hermes_stdio_low_buf_adapter_test "" "" hermes_small "")
-gcc_hermes(hermes_stdio_low_buf_adapter_test "" "" hermes_small async)
+# TODO: The DPE doesn't respect available buffering space. In this test, it
+# gives out over 1 MiB of RAM even though the RAM tier only has ~756 KiB
+# available. See issue #439.
+# gcc_hermes(hermes_stdio_low_buf_adapter_test "" "" hermes_small "")
+# gcc_hermes(hermes_stdio_low_buf_adapter_test "" "" hermes_small async)
 
 add_executable(hermes_stdio_adapter_mode_test stdio_adapter_mode_test.cpp ${ADAPTER_COMMON})
 target_link_libraries(hermes_stdio_adapter_mode_test hermes_stdio)

diff --git a/benchmarks/CMakeLists.txt b/benchmarks/CMakeLists.txt
@@ -3,7 +3,7 @@ include_directories(
   ${PROJECT_SOURCE_DIR}/test
 )
 
-set(BENCHMARKS mdm_bench dpe_bench vpic_bench)
+set(BENCHMARKS mdm_bench dpe_bench vpic_bench borg_bench)
 
 foreach(benchmark ${BENCHMARKS})
   add_executable(${benchmark} ${benchmark}.cc)