kamcheungting-db opened a new pull request, #723: URL: https://github.com/apache/iceberg-cpp/pull/723
## What Second of the logging stack. Adds the abstract Logger interface (ShouldLog/Log/SetLevel/level/Flush/Noop), the owned LogMessage record (formatted text + source location + reserved attributes), and the process-wide default logger: a lock-free atomic level gate for the disabled path and a thread-local generation cache for the enabled path (no lock/refcount in steady state), swapped under a mutex via SetDefaultLogger/SetDefaultLevel. Seed default is the no-op logger; CerrLogger and spdlog come in later PRs. Also adds the .cc-only generated config.h used later for backend selection. ## Testing compiled and run with clang++ -std=c++23 -stdlib=libc++ (clang 18) — Noop singleton, seed default, set/get, generation-cache swaps, gate seeding, SetDefaultLevel sync, and nullptr→Noop all pass. Full ctest/spdlog-ON build needs a C++23 toolchain + network (gtest/spdlog) in CI. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
