clean

kuzudb · Apr 9, 2024 · 5840b68 · 5840b68
1 parent 1e3cdc2
commit 5840b68
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 65 deletions.
diff --git a/src/include/storage/wal/wal_record.h b/src/include/storage/wal/wal_record.h
@@ -28,8 +28,6 @@ enum class DBFileType : uint8_t {
     METADATA = 2,
 };
 
-std::string dbFileTypeToString(DBFileType dbFileType);
-
 // DBFileID start with 1 byte type and 1 byte isOverflow field followed with additional
 // bytes needed by the different log types. We don't need these to be byte aligned because they are
 // not stored in memory. These are used to serialize and deserialize log entries.
@@ -54,16 +52,13 @@ enum class WALRecordType : uint8_t {
     COMMIT_RECORD = 3,
     CATALOG_RECORD = 4,
     CREATE_TABLE_RECORD = 6,
-    CREATE_REL_TABLE_GROUP_RECORD = 7,
     CREATE_RDF_GRAPH_RECORD = 8,
     COPY_TABLE_RECORD = 19,
     DROP_TABLE_RECORD = 20,
     DROP_PROPERTY_RECORD = 21,
     ADD_PROPERTY_RECORD = 22,
 };
 
-std::string walRecordTypeToString(WALRecordType walRecordType);
-
 struct WALRecord {
     WALRecordType type;
 

diff --git a/src/storage/wal/wal_record.cpp b/src/storage/wal/wal_record.cpp
@@ -179,23 +179,6 @@ std::unique_ptr<AddPropertyRecord> AddPropertyRecord::deserialize(
     return retVal;
 }
 
-std::string dbFileTypeToString(DBFileType dbFileType) {
-    switch (dbFileType) {
-    case DBFileType::DATA: {
-        return "DATA";
-    }
-    case DBFileType::METADATA: {
-        return "METADATA";
-    }
-    case DBFileType::NODE_INDEX: {
-        return "NODE_INDEX";
-    }
-    default: {
-        KU_UNREACHABLE;
-    }
-    }
-}
-
 DBFileID DBFileID::newDataFileID() {
     return DBFileID{DBFileType::DATA};
 }
@@ -210,46 +193,5 @@ DBFileID DBFileID::newPKIndexFileID(common::table_id_t tableID) {
     return retVal;
 }
 
-std::string walRecordTypeToString(WALRecordType walRecordType) {
-    switch (walRecordType) {
-    case WALRecordType::PAGE_UPDATE_OR_INSERT_RECORD: {
-        return "PAGE_UPDATE_OR_INSERT_RECORD";
-    }
-    case WALRecordType::TABLE_STATISTICS_RECORD: {
-        return "TABLE_STATISTICS_RECORD";
-    }
-    case WALRecordType::COMMIT_RECORD: {
-        return "COMMIT_RECORD";
-    }
-    case WALRecordType::CATALOG_RECORD: {
-        return "CATALOG_RECORD";
-    }
-    case WALRecordType::CREATE_TABLE_RECORD: {
-        return "CREATE_TABLE_RECORD";
-    }
-    case WALRecordType::CREATE_REL_TABLE_GROUP_RECORD: {
-        return "REL_TABLE_GROUP_RECORD";
-    }
-    case WALRecordType::CREATE_RDF_GRAPH_RECORD: {
-        return "CREATE_RDF_GRAPH_RECORD";
-    }
-    case WALRecordType::COPY_TABLE_RECORD: {
-        return "COPY_TABLE_RECORD";
-    }
-    case WALRecordType::DROP_TABLE_RECORD: {
-        return "DROP_TABLE_RECORD";
-    }
-    case WALRecordType::ADD_PROPERTY_RECORD: {
-        return "ADD_PROPERTY_RECORD";
-    }
-    case WALRecordType::DROP_PROPERTY_RECORD: {
-        return "DROP_PROPERTY_RECORD";
-    }
-    default: {
-        KU_UNREACHABLE;
-    }
-    }
-}
-
 } // namespace storage
 } // namespace kuzu
diff --git a/src/storage/wal_replayer.cpp b/src/storage/wal_replayer.cpp
@@ -108,8 +108,7 @@ void WALReplayer::replayWALRecord(WALRecord& walRecord,
         replayAddPropertyRecord(walRecord);
     } break;
     default:
-        throw RuntimeException("Unrecognized WAL record type inside WALReplayer::replay. type: " +
-                               walRecordTypeToString(walRecord.type));
+        KU_UNREACHABLE;
     }
 }