Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
  • Loading branch information
JaySon-Huang authored Aug 13, 2024
1 parent 9cdb1a3 commit 2baedd6
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,7 @@ MutableColumnPtr VectorColumnFromIndexReader::calcResultsByPack(
results_it++;
}

// insert <pack_id, [offset0, offset1, ...]>
column->insertData(
reinterpret_cast<const char *>(offsets_in_pack.data()),
offsets_in_pack.size() * sizeof(UInt32));
Expand Down Expand Up @@ -114,7 +115,7 @@ void VectorColumnFromIndexReader::read(MutableColumnPtr & column, size_t start_p
}
RUNTIME_CHECK(filled_result_rows == offset_in_pack);

// TODO: We could fill multiple rows if rowid is continuous.
// TODO(vector-index): We could fill multiple rows if rowid is continuous.
VectorIndexViewer::Key rowid = pack_start_rowid[pack_id] + offset_in_pack;
index->get(rowid, value);
column->insertData(reinterpret_cast<const char *>(value.data()), value.size() * sizeof(Float32));
Expand Down

0 comments on commit 2baedd6

Please sign in to comment.