Remove TransactionLogIteratorRace when -DNDEBUG

Author: igorcanadi

db/db_test.cc

@@ -5441,6 +5441,7 @@ TEST(DBTest, TransactionLogIterator) {
   } while (ChangeCompactOptions());
 }
 
+#ifndef NDEBUG // sync point is not included with DNDEBUG build
 TEST(DBTest, TransactionLogIteratorRace) {
   // Setup sync point dependency to reproduce the race condition of
   // a log file moved to archived dir, in the middle of GetSortedWalFiles
@@ -5485,6 +5486,7 @@ TEST(DBTest, TransactionLogIteratorRace) {
     }
   } while (ChangeCompactOptions());
 }
+#endif
 
 TEST(DBTest, TransactionLogIteratorMoveOverZeroFiles) {
   do {

db/file_indexer.h

@@ -16,7 +16,7 @@
 namespace rocksdb {
 
 class Comparator;
-class FileMetaData;
+struct FileMetaData;
 
 // The file tree structure in Version is prebuilt and the range of each file
 // is known. On Version::Get(), it uses binary search to find a potential file