Skip to content

Commit 2823fa2

Browse files
Apply suggestions from code review
Co-authored-by: kirkrodrigues <[email protected]>
1 parent 00239a4 commit 2823fa2

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

components/core/tests/test-clp_s-delta-encode-log-order.cpp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,8 @@ constexpr size_t cNumEntries{3};
2323

2424
namespace {
2525
/**
26-
* This class is a simple implementation of the abstract clp_s::FilterClass that allows us to grab
27-
* the underlying `std::vector<BaseColumnReader*>` from a `SchemaReader`.
26+
* A simple implementation of `clp_s::FilterClass` that allows us to grab the underlying
27+
* `std::vector<BaseColumnReader*>` from a `SchemaReader`.
2828
*/
2929
class SimpleFilterClass : public clp_s::FilterClass {
3030
public:
@@ -101,6 +101,7 @@ TEST_CASE("clp-s-delta-encode-log-order", "[clp-s][delta-encode-log-order]") {
101101
REQUIRE(nullptr != log_event_idx_reader);
102102
REQUIRE(clp_s::NodeType::DeltaInteger == log_event_idx_reader->get_type());
103103
REQUIRE(nullptr != dynamic_cast<clp_s::DeltaEncodedInt64ColumnReader*>(log_event_idx_reader));
104+
104105
// Test forwards and backwards seeks on `DeltaEncodedInt64ColumnReader`.
105106
size_t i{start_index};
106107
for (size_t num_iterations{0ULL}; num_iterations < cNumEntries; ++num_iterations) {

0 commit comments

Comments
 (0)