Skip to content

Commit

Permalink
fix
Browse files Browse the repository at this point in the history
  • Loading branch information
Huang Wei committed Jun 28, 2024
1 parent 1d683dc commit 966b2d5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions hybridse/src/node/sql_node.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1188,6 +1188,7 @@ static absl::flat_hash_map<SqlNodeType, absl::string_view> CreateSqlNodeTypeToNa
{kCreateFunctionStmt, "kCreateFunctionStmt"},
{kCreateUserStmt, "kCreateUserStmt"},
{kAlterUserStmt, "kAlterUserStmt"},
{kRevokeStmt, "kRevokeStmt"},
{kGrantStmt, "kGrantStmt"},

Check warning on line 1192 in hybridse/src/node/sql_node.cc

View check run for this annotation

Codecov / codecov/patch

hybridse/src/node/sql_node.cc#L1191-L1192

Added lines #L1191 - L1192 were not covered by tests
{kDynamicUdfFnDef, "kDynamicUdfFnDef"},
{kDynamicUdafFnDef, "kDynamicUdafFnDef"},
Expand Down
2 changes: 1 addition & 1 deletion src/storage/index_organized_table.cc
Original file line number Diff line number Diff line change
Expand Up @@ -287,7 +287,7 @@ absl::Status IndexOrganizedTable::Put(uint64_t time, const std::string& value, c
std::optional<uint64_t> clustered_tsv;
std::map<uint32_t, std::map<int32_t, uint64_t>> ts_value_map;

Check warning on line 288 in src/storage/index_organized_table.cc

View check run for this annotation

Codecov / codecov/patch

src/storage/index_organized_table.cc#L287-L288

Added lines #L287 - L288 were not covered by tests
// we need two ref cnt
// 1. clustered and covering: put row ->DataBlock(i)
// 1. clustered and covering: put row -> DataBlock(i)
// 2. secondary: put pkeys+pts -> DataBlock(j)
uint32_t real_ref_cnt = 0, secondary_ref_cnt = 0;

Check warning on line 292 in src/storage/index_organized_table.cc

View check run for this annotation

Codecov / codecov/patch

src/storage/index_organized_table.cc#L292

Added line #L292 was not covered by tests
// cidx_inner_key_pair can get the clustered index
Expand Down

0 comments on commit 966b2d5

Please sign in to comment.