Fix nightly CI regressions from #14478

Fix two build failures introduced by the accelerate-build PR (#14478):

1. **Release build (NDEBUG) compile error**: The SyncPoint cleanup
   listener in testharness.cc referenced SyncPoint::GetInstance() which
   is only declared in debug builds. Guard the listener with #ifndef
   NDEBUG since it's only needed for debug test sharding anyway.

2. **Folly-lite link error**: The USE_FOLLY_LITE cmake path
   unconditionally linked -lglog, but glog may not be installed as a
   system library. Use find_library() to only link glog when available.

Co-authored-by: Navi <navi@navibot.dev>

Author: xingbowang

CMakeLists.txt

@@ -1137,7 +1137,10 @@ if(USE_FOLLY_LITE)
   include_directories(${FMT_INCLUDE_DIR})
 
   add_definitions(-DUSE_FOLLY -DFOLLY_NO_CONFIG)
-  list(APPEND THIRDPARTY_LIBS glog)
+  find_library(GLOG_LIBRARY glog)
+  if(GLOG_LIBRARY)
+    list(APPEND THIRDPARTY_LIBS glog)
+  endif()
 endif()
 
 set(ROCKSDB_STATIC_LIB rocksdb${ARTIFACT_SUFFIX})

test_util/testharness.cc

@@ -13,9 +13,12 @@
 #include <string>
 #include <thread>
 
+#ifndef NDEBUG
 #include "test_util/sync_point.h"
+#endif
 
 namespace {
+#ifndef NDEBUG
 // Global gtest event listener that cleans up SyncPoint state after every
 // test. Many tests set SyncPoint callbacks with captured local variables
 // but forget to disable/clear them. Under sharded execution (multiple
@@ -45,6 +48,7 @@ static int RegisterSyncPointCleanup() noexcept {
 // NOLINTNEXTLINE(cppcoreguidelines-avoid-non-const-global-variables)
 [[maybe_unused]] static int sync_point_cleanup_registered_ =
     RegisterSyncPointCleanup();
+#endif  // !NDEBUG
 }  // namespace
 
 namespace ROCKSDB_NAMESPACE::test {