[fix][cp] Fix Parquet read with an isNull filter on nested array

bolt/dwio/parquet/reader/PageReader.cpp

@@ -1153,6 +1153,10 @@ void PageReader::skip(int64_t numRows) {
   }
   firstUnvisited_ += numRows;
 
+  if (toSkip == 0) {
+    return;
+  }
+
   // Skip nulls
   toSkip = skipNulls(toSkip);
 

bolt/dwio/parquet/tests/reader/ParquetTableScanTest.cpp

@@ -432,7 +432,6 @@ TEST_F(ParquetTableScanTest, decimalSubfieldFilter) {
       "Scalar function signature is not supported: eq(DECIMAL(5, 2), DECIMAL(5, 1))");
 }
 
-// Core dump is fixed.
 TEST_F(ParquetTableScanTest, map) {
   auto vector = makeMapVector<StringView, StringView>({{{"name", "gluten"}}});
 
@@ -461,7 +460,6 @@ TEST_F(ParquetTableScanTest, nullMap) {
   assertSelectWithFilter({"i", "c"}, {}, "", "SELECT i, c FROM tmp");
 }
 
-// Core dump is fixed.
 TEST_F(ParquetTableScanTest, singleRowStruct) {
   auto vector = makeArrayVector<int32_t>({{}});
   loadData(
@@ -590,6 +588,21 @@ TEST_F(ParquetTableScanTest, reqArrayLegacy) {
       "SELECT UNNEST(array[array['a', 'b'], array[], array['c', 'd']])");
 }
 
+TEST_F(ParquetTableScanTest, filterOnNestedArray) {
+  loadData(
+      getExampleFilePath("struct_of_array.parquet"),
+      ROW({"struct"},
+          {ROW({"a0", "a1"}, {ARRAY(VARCHAR()), ARRAY(INTEGER())})}),
+      makeRowVector(
+          {"unused"},
+          {
+              makeFlatVector<int32_t>({}),
+          }));
+
+  assertSelectWithFilter(
+      {"struct"}, {}, "struct.a0 is null", "SELECT ROW(NULL, NULL)");
+}
+
 TEST_F(ParquetTableScanTest, readAsLowerCase) {
   auto plan = PlanBuilder(pool_.get())
                   .tableScan(ROW({"a"}, {BIGINT()}), {}, "")