Description

After the Jetty upgrade, we can no longer run presto-on-spark native on JDK8/11. This is because presto-on-spark (pos) native relies on airlift.http-client to communicate between the JVM code and the Cpp sidecar.
But, with jetty upgrade, min JDK version is 17.

This PR replaces the jetty http client with okhttp in the presto-on-spark native codepath.

To do this it does 2 things

1. Replace direct usages of http-client , http-server from the pos native codepath
2. Remove dependency on drift-client and drift-transport-api which were added previously.

Below are exhaustive list of changes to achieve this

1. Remove http-client, http-server dependency

• We add new classes that replace the HttpClient functionality - OkHttpResponseHandler, OkHttpBaseResponse, (dto) OkHttpHeaderName.
• We also update other classes in com.facebook.presto.spark.execution.http which relied on http client metadata classes (headers, http status , header string constants) with either okhttp alternatives where available, or google/apache commons libs where okhttp alternative is not available.

2. Remove drift dependency

   • Add new inline class for handling http responses PagesResponseHandler.
     This enables deletion/cleanup of the
     • HttpRpcShuffleClient
     • HttpAndThriftRpcShuffleClientProvider,
     • HttpShuffleClientProvider
     • ThriftRpcShuffleClient
     • ThriftShuffleClientProvider

3. Finally we have also updated the pom Remove pom dependency on http-client, http-server, drift-client , drift-transport-api

Test Plan

CI tests

== NO RELEASE NOTE ==