felipepessoto commented on code in PR #13136:
URL: https://github.com/apache/gluten/pull/13136#discussion_r4406930607


##########
.github/workflows/util/delta-spark-ut/known-failures.txt:
##########
@@ -369,14 +369,10 @@ 
org.apache.spark.sql.delta.deletionvectors.DeletionVectorsWithPredicatePushdownS
 
org.apache.spark.sql.delta.deletionvectors.DeletionVectorsWithPredicatePushdownSuite#PredicatePushdown:
 Single deletion at the second row group. vectorizedReaderEnabled: true 
readColumnarBatchAsRows: true
 
org.apache.spark.sql.delta.deletionvectors.DeletionVectorsWithPredicatePushdownSuite#huge
 table: delete a small number of rows from tables of 2B rows with DVs
 
org.apache.spark.sql.delta.deletionvectors.DeletionVectorsWithPredicatePushdownSuite#huge
 table: read from tables of 2B rows with existing DV of many zeros
-org.apache.spark.sql.delta.generatedsuites.dataskipping.DataSkippingDeltaV1CheckpointV2JsonSuite#Data
 skipping handles aliasing for _metadata fields
-org.apache.spark.sql.delta.generatedsuites.dataskipping.DataSkippingDeltaV1CheckpointV2JsonSuite#Data
 skipping handles aliasing for _metadata fields - old behavior with DataFrame 
schema
 
org.apache.spark.sql.delta.generatedsuites.dataskipping.DataSkippingDeltaV1CheckpointV2JsonSuite#data
 skipping flags

Review Comment:
   Confirmed on current head `27e9bf905`: the [Delta 4.4.1 / Spark 4.1 
run](https://github.com/apache/gluten/actions/runs/40763107194) completed 
successfully. I checked the JUnit XML for all 12 removed entries under 
`org.apache.spark.sql.delta.generatedsuites.dataskipping`; every case executed 
and passed, with none skipped or missing.
   
   The native/bundle jobs and the shards containing those cases used merge 
`b89540fe` containing this head, with the same verified bundle artifact. All 
eight shard gates and the aggregate passed with zero regressions, zero 
now-passing baseline entries, and zero stale entries. The unchanged strict 
comparison also passes against the downloaded reports. This validates the 
baseline removals on the current Delta release, not only the historical 4.2.0 
run.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to