David Stryker

Results 34 comments of David Stryker

I pushed a commit that fixed all of the fault-tolerant tests by adding a method to MergePartitioningHandle to produce the FaultTolerantPartitioningScheme, and another to produce the TaskSource needed by the...

I pushed a commit that fixes (most of) those delta lake tests that failed because they tried to delete or update rows whose partition column value was null. The bug...

I pushed a commit that fixes all the Kudu smoke test failures.

A bunch of test failures have been fixed by the last three commits. Here are the remain test failures that are related to the changes in this PR: ### Iceberg...

The latest complete list of failures, including duplicates: ``` ./11_pt (default, suite-8-non-generic, ).txt INFO: FAILURE / io.trino.tests.product.hive.TestHiveTransactionalTable.testAcidUpdatePartitioned ./11_pt (default, suite-8-non-generic, ).txt INFO: FAILURE / io.trino.tests.product.hive.TestHiveTransactionalTable.testCorrectSelectCountStar ./11_pt (default, suite-8-non-generic, ).txt INFO:...

Delta Lake tests that test file counts after update are failing because `DeltaLakeMergeSink.storeMergedRows` is using `MergePage.createDeleteAndInsertPages` to break the changes up into insert and delete pages, and subsequently treating each...

@alexjo2144 showed me how to decode Delta Lake files, and I quickly found the bug with tests like `testVacuum`: storing data files with zero rows. I added a commit to...

The most recent commit - - [ix predicate pushdown in merge](https://github.com/trinodb/trino/pull/13926/commits/8dd1215608dada16e73659a0a8dc4353f6fa71ae) - - fixes all of the pushdown problems. However, it breaks all the Kudu merge tests because the `KuduRecordSet`...

The latest commit fixes the regression of Kudu tests. Here is the complete list of current test failures, not including duplicates. The next steps are to fix the Hive row...

@electrum and I fixed bugs in the commit adding Hive update layout support, and I fixed a bug in comparing partition bucket maps. After these changes, we have only 6...