Jaeho Yoo

Would be great if there was a trino version of such great gateway !

Just a heads up @ccchenhe @ahackett, this issue has been fixed and uses `/ui/api/stats` for jdbc/http in trinodb/trino-gateway https://github.com/trinodb/trino-gateway/commit/11b2dd2fa490d4a626620a1d92266a485b86530e#diff-a3a976d789379cd160fbc2e9281073d96aa6e682e9d9e9ab67883992acaeeab2 For [http](https://github.com/trinodb/trino-gateway/blob/main/gateway-ha/src/main/java/io/trino/gateway/ha/clustermonitor/ClusterStatsHttpMonitor.java#L93-L94) and [jdbc](https://github.com/trinodb/trino-gateway/blob/main/gateway-ha/src/main/java/io/trino/gateway/ha/clustermonitor/ClusterStatsJdbcMonitor.java#L38-L41) it uses username/pwd set in backendStateConfiguration

> You would need to inject every single linked class. That is not normally recommended. What are you trying to do? Yeah that was the issue. I had to add...

> only use jvm types in your abstract class. Hmm.. I don't quite understand this part. is there any links that you can share what you mean by jvm types?

Thx for very interesting feature. We were expecting this kind of feature to limit maximum number of partitions. - https://trinodb.slack.com/archives/CP1MUNEUX/p1646707920591449?thread_ts=1645502073.803079&cid=CP1MUNEUX Before this, we were using Bytebuddy to Hook HivePartitionManager class...

Hi, it would be great if this PR gets merged !

> We're not using JDBC here so we only need this feature for the Trino CLI, but having support for extra headers in the code makes implementing this for JDBC...

FYI, I just tested the approach of using `X-Trino-Client-Tags` with trino-gateway which works brilliantly so either way works :) Gateway provides routing-rules engine option for those circumstance where it is...

after reviewing [golang url](https://github.com/golang/go/blob/go1.15/src/net/url/url.go#L358) code, i take back second suggestion as URL schema has UserInfo. It suggests general form as `[scheme:][//[userinfo@]host][/]path[?query][#fragment]`, so separating `default user` with `server addr` would be...

@losipiuk Hey, am back to work with Presto :) While I was working with query callback, I was wondering WDUT about this idea of changing header constants to public? `prestoUserHeader`...