[Bug report] Gravitino-trino-connector is incompatible with Trino-435

[theoryxu]

Version

main branch

Describe what's wrong

Hello fellows

There are some problems when gravitino-trino-connector（0.4.0） is working with Trino-435. I guess Trino-435 has some new interfaces that the connector has not yet implemented. Error information is as follows.

Error message and/or stacktrace

How to reproduce

0.4.0 or above

Additional context

No response

[jerryshao]

@diqiu50 can you please take a look at this?

[diqiu50]

In the directory of the graviton-trino-connector plugin, some Trino jars have different versions than the Trino server.

ls   trino-*
-rw-r--r--@ 1 ice  staff   36737 Feb 28 16:14 trino-cache-426.jar
-rw-r--r--@ 1 ice  staff   20523 Feb 28 16:14 trino-matching-426.jar
-rw-r--r--  1 ice  staff  317755 Feb 28 16:24 trino-plugin-toolkit-426.jar
-rw-r--r--@ 1 ice  staff  976399 Feb 28 16:14 trino-spi-426.jar

I replaced the trino-plugin-toolkit-426.jar with trino-plugin-toolkit-435.jar, and now the graviton-connector works on Trino-435

So we need to find a way to solve this version issue.

[jerryshao]

I think you should not include trino dependencies, perhaps you may need to change the dependency from implementation to compileOnly?

[yuqi1129]

I think you should not include trino dependencies, perhaps you may need to change the dependency from implementation to compileOnly?

Yes, he is going to remove all trino-related jars from the runtime class path.

[theoryxu]

@diqiu50 Hi bro, just for your information. There is another case that shows a different error stack trace.

When the node is a worker rather than a coodirator, the error msg is the following:

[diqiu50]

Ok, I will test it later

[jerryshao]

@diqiu50 Hi bro, just for your information. There is another case that shows a different error stack trace.

When the node is a worker rather than a coodirator, the error msg is the following:

Can you please create a separate issue to track this bug? @theoryxu

[theoryxu]

@jerryshao, of course: #2428