Skip to content

Commit

Permalink
resolve comments
Browse files Browse the repository at this point in the history
Signed-off-by: Alex Chi <[email protected]>
  • Loading branch information
skyzh committed Jan 13, 2024
1 parent f57e11e commit cb9c5ab
Show file tree
Hide file tree
Showing 3 changed files with 21 additions and 16 deletions.
2 changes: 1 addition & 1 deletion src/type/type.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -293,7 +293,7 @@ auto Type::GetData(const Value &val __attribute__((unused))) const -> const char

// Get the length of the variable length data
auto Type::GetStorageSize(const Value &val __attribute__((unused))) const -> uint32_t {
throw NotImplementedException("GetLength not implemented");
throw NotImplementedException("GetStorageSize not implemented");
}

} // namespace bustub
20 changes: 20 additions & 0 deletions test/sql/index.slt
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
statement ok
CREATE TABLE t1(v1 integer, v2 integer);

statement ok
INSERT INTO t1 VALUES (0, 0), (1, 1), (2, 2);

statement ok
CREATE INDEX t1v2a ON t1 (v2);

statement ok
CREATE INDEX t1v2b ON t1 USING bplustree (v2);

statement ok
CREATE INDEX t1v2c ON t1 USING hash (v2);

statement ok
CREATE INDEX t1v2d ON t1 USING stl_ordered (v2);

statement ok
CREATE INDEX t1v2e ON t1 USING stl_unordered (v2);
15 changes: 0 additions & 15 deletions test/sql/vector.slt
Original file line number Diff line number Diff line change
Expand Up @@ -3,18 +3,3 @@ CREATE TABLE t1(v1 VECTOR(3), v2 integer);

statement ok
INSERT INTO t1 VALUES (ARRAY [1.0, 1.0, 1.0], 232), (ARRAY [2.0, 1.0, 1.0], 233), (ARRAY [3.0, 1.0, 1.0], 234), (ARRAY [4.0, 1.0, 1.0], 235);

statement ok
CREATE INDEX t1v2a ON t1 (v2);

statement ok
CREATE INDEX t1v2b ON t1 USING bplustree (v2);

statement ok
CREATE INDEX t1v2c ON t1 USING hash (v2);

statement ok
CREATE INDEX t1v2d ON t1 USING stl_ordered (v2);

statement ok
CREATE INDEX t1v2e ON t1 USING stl_unordered (v2);

0 comments on commit cb9c5ab

Please sign in to comment.