Loki connector broken since 477 (okhttp version mixup)

[b4sus]

Hello,

we are updating our trino from 475 to 478 and we bumped into problem with loki connector, specifically between version 476 and 477.

The error by starting trino:

1) [Guice/ErrorInCustomProvider]: NoClassDefFoundError: okhttp3/MediaType
  at LokiModule.getLokiClient(LokiModule.java:50)
  at LokiMetadata.<init>(LokiMetadata.java:58)
      \_ for 1st parameter lokiClient
  at LokiModule.configure(LokiModule.java:39)
  while locating LokiMetadata
  at LokiModule.configure(LokiModule.java:38)
  while locating ConnectorMetadata

Learn more:
  https://github.com/google/guice/wiki/ERROR_IN_CUSTOM_PROVIDER
Caused by: NoClassDefFoundError: okhttp3/MediaType
	at LokiClient.<clinit>(LokiClient.java:41)
	at LokiModule.getLokiClient(LokiModule.java:50)
	at LokiModule$$FastClassByGuice$$6646.GUICE$TRAMPOLINE(<generated>)
	at LokiModule$$FastClassByGuice$$6646.apply(<generated>)
	at ProviderMethod$FastClassProviderMethod.doProvision(ProviderMethod.java:

As far as i can tell, the problem comes from trino bumping the version of okhttp to version 5.x (was 4.12.0, bump happened between 476 and 477), but loki-client declares 4.12.0 (see here).

Currently we just disabled the loki connector.

We are using standard trino image from dockerhub.

b4sus

[ebyhr]

I guess this issue explains why Loki connector is broken in our product tests #26500
cc: @homar

[wendigo]

MediaType is still there but now okhttp is split into okhttp and okhttp-jvm so I guess it just lacks runtime dependency.