fix internal tests

yma11 · Aug 8, 2024 · 959f214 · 959f214
1 parent cf8e821
commit 959f214
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 0 deletions.
diff --git a/velox/dwio/parquet/reader/RepeatedColumnReader.cpp b/velox/dwio/parquet/reader/RepeatedColumnReader.cpp
@@ -211,6 +211,13 @@ void MapColumnReader::read(
   elementReader_->seekTo(childTargetReadOffset_, false);
 }
 
+void MapColumnReader::filterRowGroups(
+    uint64_t rowGroupSize,
+    const dwio::common::StatsContext& context,
+    dwio::common::FormatData::FilterRowGroupsResult& result) const {
+  // empty placeholder to avoid incorrect calling on parent's impl
+}
+
 ListColumnReader::ListColumnReader(
     const TypePtr& requestedType,
     const std::shared_ptr<const dwio::common::TypeWithId>& fileType,
@@ -306,4 +313,11 @@ void ListColumnReader::read(
   child_->seekTo(childTargetReadOffset_, false);
 }
 
+void ListColumnReader::filterRowGroups(
+    uint64_t rowGroupSize,
+    const dwio::common::StatsContext& context,
+    dwio::common::FormatData::FilterRowGroupsResult& result) const {
+  // empty placeholder to avoid incorrect calling on parent's impl
+}
+
 } // namespace facebook::velox::parquet
diff --git a/velox/dwio/parquet/reader/RepeatedColumnReader.h b/velox/dwio/parquet/reader/RepeatedColumnReader.h
@@ -97,6 +97,11 @@ class MapColumnReader : public dwio::common::SelectiveMapColumnReader {
   /// supplied before receiving new lengths.
   void skipUnreadLengths();
 
+  void filterRowGroups(
+      uint64_t rowGroupSize,
+      const dwio::common::StatsContext&,
+      dwio::common::FormatData::FilterRowGroupsResult&) const override;
+
  private:
   RepeatedLengths lengths_;
   RepeatedLengths keyLengths_;
@@ -148,6 +153,11 @@ class ListColumnReader : public dwio::common::SelectiveListColumnReader {
   /// supplied before receiving new lengths.
   void skipUnreadLengths();
 
+  void filterRowGroups(
+      uint64_t rowGroupSize,
+      const dwio::common::StatsContext&,
+      dwio::common::FormatData::FilterRowGroupsResult&) const override;
+
  private:
   RepeatedLengths lengths_;
   arrow::LevelInfo levelInfo_;