Expose build_config() in Python (#388)

Author: jparismorgan

apis/python/src/tiledb/vector_search/type_erased_module.cc

@@ -42,6 +42,7 @@
 #include "api/ivf_flat_index.h"
 #include "api/vamana_index.h"
 #include "detail/time/temporal_policy.h"
+#include "stats.h"
 
 namespace py = pybind11;
 
@@ -440,4 +441,6 @@ void init_type_erased_module(py::module_& m) {
       .def("id_type_string", &IndexIVFFlat::id_type_string)
       .def("px_type_string", &IndexIVFFlat::px_type_string)
       .def("dimensions", &IndexIVFFlat::dimensions);
+
+  m.def("build_config_string", []() { return build_config().dump(); });
 }

src/include/stats.h

@@ -222,42 +222,20 @@ static auto dump_logs = [](std::string filename,
   }
 };
 
-#ifdef JSON_LOGGING
-static auto config_log(const std::string& program_name) {
-  std::string uuid_;
-  char host_[16];
-  std::string date_;
-  std::size_t uuid_size_ = 24;
-
-  auto seed = std::random_device();
-  auto gen = std::mt19937(seed());
-  auto dis = std::uniform_int_distribution<int8_t>(97, 122);
-  uuid_.resize(uuid_size_);
-  std::generate(uuid_.begin(), uuid_.end(), [&] { return dis(gen); });
-
-  if (int e = gethostname(host_, sizeof(host_))) {
-    std::cerr << "truncated host name\n";
-    strncpy(host_, "ghost", 15);
-  }
-  {
-    std::stringstream ss;
-    std::time_t currentTime = std::time(nullptr);
-    std::string dateString = std::ctime(&currentTime);
-    dateString.erase(dateString.find('\n'));
-    ss << dateString;
-    date_ = ss.str();
-  }
+static auto build_config() {
+  // This is failing today, but could perhaps be added back in the future.
+  // char host_[16];
+  // if (int e = gethostname(host_, sizeof(host_))) {
+  //   std::cerr << "truncated host name\n";
+  //   strncpy(host_, "ghost", 15);
+  // }
 
   auto&& [major, minor, patch] = tiledb::version();
 
   json config = {
-      {"uuid", uuid_},
-      {"host", host_},
-      {"Program", program_name},
-      {"Build_date", CURRENT_DATETIME},
-      {"Run_date", date_},
-      {"cmake_source_dir", CMAKE_SOURCE_DIR},
-      {"Build", BUILD_TYPE},
+      {"CURRENT_DATETIME", CURRENT_DATETIME},
+      {"CMAKE_SOURCE_DIR", CMAKE_SOURCE_DIR},
+      {"BUILD_TYPE", BUILD_TYPE},
       {"Compiler",
        {{"CXX_COMPILER", IVF_HACK_CXX_COMPILER},
         {"CXX_COMPILER_ID", CXX_COMPILER_ID},
@@ -296,6 +274,5 @@ auto args_log(const Args& args) {
   }
   return arg_log;
 }
-#endif  // JSON_LOGGING
 
 #endif  // TDB_STATS_H

src/include/test/unit_stats.cc

@@ -27,17 +27,16 @@
  *
  * @section DESCRIPTION
  *
- * WIP.
  */
 
 #include <catch2/catch_all.hpp>
-#include <chrono>
-#include <iostream>
-#include <thread>
 
-#include "utils/logging.h"
-#include "utils/timer.h"
+#include "stats.h"
 
 TEST_CASE("stats: test test", "[stats]") {
   REQUIRE(true);
 }
+
+TEST_CASE("stats: test build_config", "[stats]") {
+  build_config();
+}