From 4a1a5807d8f1ec17c94b67f6761be4d7406541e3 Mon Sep 17 00:00:00 2001 From: a8chakra Date: Fri, 6 Jan 2023 13:19:59 -0500 Subject: [PATCH] [1100] Inconsistent parameter order in RelsStore --- src/include/storage/store/rels_store.h | 4 ++-- src/storage/wal_replayer.cpp | 4 ++-- test/copy_csv/copy_csv_lists_test.cpp | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/include/storage/store/rels_store.h b/src/include/storage/store/rels_store.h index 7e0af6685d5..3cf19aeb6a7 100644 --- a/src/include/storage/store/rels_store.h +++ b/src/include/storage/store/rels_store.h @@ -20,8 +20,8 @@ class RelsStore { // TODO(Ziyi): other getters requires (direction, nodeID, relID) but this one is requiring // (direction, relID, nodeID). - inline Column* getRelPropertyColumn(RelDirection relDirection, table_id_t relTableID, - table_id_t boundNodeTableID, uint64_t propertyIdx) const { + inline Column* getRelPropertyColumn(RelDirection relDirection, table_id_t boundNodeTableID, + table_id_t relTableID, uint64_t propertyIdx) const { return relTables.at(relTableID) ->getPropertyColumn(relDirection, boundNodeTableID, propertyIdx); } diff --git a/src/storage/wal_replayer.cpp b/src/storage/wal_replayer.cpp index 803e761b317..9b34492c686 100644 --- a/src/storage/wal_replayer.cpp +++ b/src/storage/wal_replayer.cpp @@ -156,7 +156,7 @@ void WALReplayer::replayWALRecord(WALRecord& walRecord) { storageStructureID.columnFileID.relPropertyColumnID.relNodeTableAndDir; Column* column = storageManager->getRelsStore().getRelPropertyColumn(relNodeTableAndDir.dir, - relNodeTableAndDir.relTableID, relNodeTableAndDir.srcNodeTableID, + relNodeTableAndDir.srcNodeTableID, relNodeTableAndDir.relTableID, storageStructureID.columnFileID.relPropertyColumnID.propertyID); diskOverflowFile = reinterpret_cast(column)->getDiskOverflowFile(); @@ -368,7 +368,7 @@ VersionedFileHandle* WALReplayer::getVersionedFileHandleIfWALVersionAndBMShouldB storageStructureID.columnFileID.relPropertyColumnID.relNodeTableAndDir; Column* column = storageManager->getRelsStore().getRelPropertyColumn(relNodeTableAndDir.dir, - relNodeTableAndDir.relTableID, relNodeTableAndDir.srcNodeTableID, + relNodeTableAndDir.srcNodeTableID, relNodeTableAndDir.relTableID, storageStructureID.columnFileID.relPropertyColumnID.propertyID); return storageStructureID.isOverflow ? reinterpret_cast(column) diff --git a/test/copy_csv/copy_csv_lists_test.cpp b/test/copy_csv/copy_csv_lists_test.cpp index ccb19b3acef..03aeaa6ceb0 100644 --- a/test/copy_csv/copy_csv_lists_test.cpp +++ b/test/copy_csv/copy_csv_lists_test.cpp @@ -69,7 +69,7 @@ TEST_F(TinySnbListTest, RelPropertyColumnWithList) { catalog->getReadOnlyVersion()->getNodeTableIDFromName("person"); auto& property = catalog->getReadOnlyVersion()->getRelProperty(tableID, "places"); auto col = graph->getRelsStore().getRelPropertyColumn( - RelDirection::FWD, tableID, nodeTablesForAdjColumnAndProperties, property.propertyID); + RelDirection::FWD, nodeTablesForAdjColumnAndProperties, tableID, property.propertyID); ASSERT_TRUE(CheckEquals({"wwAewsdndweusd", "wek"}, col->readValue(0))); ASSERT_TRUE(CheckEquals({"anew", "jsdnwusklklklwewsd"}, col->readValue(1))); ASSERT_TRUE(CheckEquals({"awndsnjwejwen", "isuhuwennjnuhuhuwewe"}, col->readValue(5)));