Update Transport with new API from Trino v406 - Githubissues

linkedin / transport

A framework for writing performant user-defined functions (UDFs) that are portable across a variety of engines including Apache Spark, Apache Hive, and Presto.

BSD 2-Clause "Simplified" License

297 stars 73 forks source link

Update Transport with new API from Trino v406 #126

Closed yiqiangin closed 1 year ago

yiqiangin commented 1 year ago

The code changes of this PR mainly include

Bumping the version of Trino used in Transport from v352 to v406
Replacing the classes defined in trino-main in v352 to the classes defined in trino-spi in v406
Replacing some classes of test infrastructure in Trino which are deprecated in v406 with the in-use classes

Tests:

unit tests are passed in all modules of Transport
major unit tests are passed in transportable-udfs-examples except several test cases fail because of the issue in QueryAssertions from Trino v406

weijiii commented 1 year ago

Curious is StdUdfWrapper going to still extend SqlScalarFunction with the new Trino SPI?