Piotr Findeisen

Flaky test TestDriver.testUnblocksOnTimeout

2

https://github.com/trinodb/trino/actions/runs/7357271602/job/20028850348?pr=20244 ``` Error: Tests run: 163, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 20.69 s

Hive DELETE/UPDATE/MERGE on ACID/transactional tables produces incorrect results

11

https://github.com/trinodb/trino/actions/runs/3541969763/jobs/5948031373 ``` 2022-11-24T18:22:07.9630199Z tests | 2022-11-25 00:07:07 INFO: FAILURE / io.trino.tests.product.hive.TestHiveMerge.testMergeUpdateWithVariousLayouts [true, ] (Groups: hive_transactional) took 5.2 seconds 2022-11-24T18:22:07.9645101Z tests | 2022-11-25 00:07:07 SEVERE: Failure cause: 2022-11-24T18:22:07.9645609Z tests | java.lang.AssertionError:...

`No catalog '...'` sporadic failures when using dynamic catalogs

4

eg https://github.com/trinodb/trino/actions/runs/7474305862/job/20341354556?pr=20320 ``` Error: io.trino.plugin.iceberg.TestIcebergPartitionEvolution -- Time elapsed: 64.18 s createQueryRunner:40 » QueryFailed No catalog 'iceberg' Error: TestIcebergMigrateProcedure.testMigrateUnsupportedColumnType:311->AbstractTestQueryFramework.assertUpdate:414->AbstractTestQueryFramework.assertUpdate:419 » QueryFailed No catalog 'hive' Error: TestIcebergOrcWithBloomFilters>BaseOrcWithBloomFiltersTest.testOrcBloomFilterIsWrittenDuringInsert:67->AbstractTestQueryFramework.assertUpdate:414->AbstractTestQueryFramework.assertUpdate:419 » QueryFailed No catalog 'iceberg'...

Flaky test TestThrottledAsyncQueue.testThrottle: Expecting value to be false but was true

3

https://github.com/trinodb/trino/actions/runs/7847471851/job/21416973734?pr=20648 ``` Error: Tests run: 10, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 3.225 s

Improve 'Expecting code to raise a throwable' when lambda returns a value

4

`assertThatThrownBy` throws "Expecting code to raise a throwable" when called code doesn't throw. When test assertion fails, it's tempting to know what did the code do instead. Enhance "Expecting code...

Builtin mapper for ZonedDateTime discards time zone information

44

There exists a built-in mapper for `ZonedDateTime` which looks like SQL's `TIMESTAMP WITH TIME ZONE` is directly supported, but the mapper does not do that. I think exposing a mapper...

Except in test code.

Remove default insert layout inferred from table properties

1

Picking write layout based on read partitioning is a heuristic, but it's better to leave the responsibility to the connector. We don't have such heuristic for updates or execute and...

Improve planning for table function without partitioning

2

Before the change planner would insert a gather exchange below such a table function, preventing distributed execution. - Extracted from https://github.com/trinodb/trino/pull/21558 / https://github.com/trinodb/trino/pull/21378

Fix table function execution without partitioning (v2)

9

Previously, when table function did not declare partitioning it would run single-threaded and first buffer all data in memory, like a one big WINDOW. After the change, the local execution...