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})

db/db_compaction_test.cc

@@ -560,6 +560,7 @@ TEST_F(DBCompactionTest, UniversalReduceFileLockingRepickNothing) {
   ASSERT_OK(Flush());
   ASSERT_OK(dbfull()->TEST_WaitForCompact());
   ASSERT_TRUE(bottom_pri_compaction_attempt_repick);
+  Env::Default()->SetBackgroundThreads(0, Env::Priority::BOTTOM);
 }
 
 TEST_F(DBCompactionTest, SkipStatsUpdateTest) {
@@ -10643,6 +10644,7 @@ TEST_F(DBCompactionTest, BottomPriCompactionCountsTowardConcurrencyLimit) {
     sleeping_task_low.WaitUntilDone();
     compact_range_thread.join();
   }
+  env_->SetBackgroundThreads(0, Env::Priority::BOTTOM);
 }
 
 TEST_F(DBCompactionTest, BottommostFileCompactionAllowIngestBehind) {
@@ -11462,6 +11464,8 @@ TEST_F(DBCompactionTest, ReleaseCompactionDuringManifestWrite) {
 
   SyncPoint::GetInstance()->DisableProcessing();
   SyncPoint::GetInstance()->ClearAllCallBacks();
+  env_->SetBackgroundThreads(1, Env::Priority::LOW);
+  env_->SetBackgroundThreads(0, Env::Priority::BOTTOM);
 }
 
 TEST_F(DBCompactionTest, RecordNewestKeyTimeForTtlCompaction) {

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 {