Improvements on transport-trino-plugin

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

Improvements on transport-trino-plugin #133

Closed yiqiangin closed 1 year ago

yiqiangin commented 1 year ago

Code Changes

The code changes are made in transport-trino-plugin part to address some remaining comments in https://github.com/linkedin/transport/pull/128
If there are subdirectories for UDFs from different UDF MPs, creating a TransportUDFClassLoader to load UDF classes for each subdirectory in TransportConnector.

Tests

unit tests
manual test with local Trino server

yiqiangin commented 1 year ago

@xkrogen and @weijiii Could you help review this PR?

weijiii commented 1 year ago

Thanks for the effort! Taking a look.