David Phillips

I like this approach, since the plugin is a normal plugin module that gets built and tested as usual. Can you split the commits so that the example plugin is...

Thanks for starting the discussion on this, which prompted me to look at Temurin. They seem to produce high-quality Docker images, which look like a good option to replace the...

The CLI is using [JLine](https://github.com/jline/jline3). Could you see if their demo works or has the same problem?

Try running the CLI with JNA disabled: ``` java -Dorg.jline.terminal.jna=false -jar /usr/bin/trino ``` Thanks for testing.

Hmm, I wonder if their demo is using Jansi rather than JNA by default.

As a bit of data, I checked the Presto code base, where we previously converted from `checkNotNull` to `requireNonNull`. We have over 4600+ instances of `requireNonNull` and less than 25...

Is it possible to implement `org.joda.time.tz.Provider` using the new `java.time.zone.ZoneRulesProvider`? Has anyone done this?

Cassandra needs to implement `getMergeRowIdColumnHandle`

This is a real failure: ``` Caused by: java.lang.IllegalArgumentException: Unsupported local exchange partitioning MERGE [insert = hive:HivePartitioningHandle{buckets=1, hiveTypes=[]}] at io.trino.operator.exchange.LocalExchange.computeBufferCount(LocalExchange.java:366) at io.trino.operator.exchange.LocalExchange.(LocalExchange.java:98) at io.trino.sql.planner.LocalExecutionPlanner$Visitor.createLocalExchange(LocalExecutionPlanner.java:3573) at io.trino.sql.planner.LocalExecutionPlanner$Visitor.visitExchange(LocalExecutionPlanner.java:3459) at io.trino.sql.planner.LocalExecutionPlanner$Visitor.visitExchange(LocalExecutionPlanner.java:847) at io.trino.sql.planner.plan.ExchangeNode.accept(ExchangeNode.java:243)...

We might need some logic in `FaultTolerantQueryScheduler#createBucketToPartitionMap` to create the `bucketToPartitionMap` for the "Bucket to partition must be set before a partition function can be created" error. Though it would...