vmware / splinterdb

High Performance Embedded Key-Value Store
https://splinterdb.org
Apache License 2.0
680 stars 57 forks source link

Replace all instances of platform_default_log() with CTEST_LOG_INFO() #543

Closed gapisback closed 1 year ago

gapisback commented 1 year ago

Also, minor fix in exec_worker_thread() in large_inserts_bugs_stress_test.c to avoid divide-by-0 error when elapsed_ns is too small.

-- Note to @rosenhouse : With this change, all unit-tests added newly by my shared-memory prototype work are changed to consistently use CTEST_LOG_INFO(). This brings them in-sync with recently added unit-test verbosity levels interfaces and behaviour.

Here is the confirmation:

agurajada/gabe/shmem-squash-logging-ovhds-poc

sdb-fdb-build:[16] $ grep -l -w platform_log *.c
sdb-fdb-build:[17] $ grep -l -w platform_default_log *.c
netlify[bot] commented 1 year ago

Deploy Preview for splinterdb canceled.

Name Link
Latest commit 337b0e5022a7ee6c88a8e6441db91df0471f0246
Latest deploy log https://app.netlify.com/sites/splinterdb/deploys/63ea7c9f18de81000846072c