Hello, We are trying to connect Pinot with Trino a...
# troubleshooting
t
Hello, We are trying to connect Pinot with Trino and we are getting this error
Copy code
No valid brokers found for backendentityview'
We got to know it is because, the trino-pinot connector doesn’t support mixed case table name. Is anything planned to support mixed case table names in the connector?
It is failing at this point https://github.com/trinodb/trino/blob/a66aaf10a4793d35604bec2174be9de806bb013c/plu[…]inot/src/main/java/io/trino/plugin/pinot/PinotSplitManager.java While it is trying to fetch the routing table from broker, it is using the tableName from tableHandle. In tableHandle, the table name is stored in lower case. because of which Trino is not able to find the routing table in Pinot.
k
@Elon ^^
e
Which version of trino and pinot are you using?
Ah, we have a pr to address this:
t
Which version of trino and pinot are you using?
We are using Trino v357 and Pinot 0.7.1
Ah, we have a pr to address this:
Yes. Saw that. Any estimates around when this would be available in Trino? Thank you!
@Elon
@Elon Are you still actively working on that PR? This is something we need to be able to use Trino with Pinot. If you are busy, we can plan to contribute the same.
e
Yes, I am actively working on it:) Right now this pr depends on https://github.com/trinodb/trino/pull/9098 - if it helps I can push a rebased version of it so you can use both. btw the pinot-0.8.0 connector is backwards compatible with pinot-0.6.0 and 0.7.1.
Then you can just clone the branch and apply the commits. We use the updated versions in production.
The pr as it is right now will not work with pinot 0.8.0, I will update you when I push the updated version (should be early this week, next few days)
t
That’d be great. Thanks @Elon !
e
Will be pushing it shortly and I'll follow up with you.
🙌 1