This could be done in a single file, but by splitting this up it
will allows contributors and maintainers to incrementally re-
enable the modernizer on each module since it will cause lots of
code change. It will also introduce lots of opportunities for new
contributors to get involved with Presto.

- Spark requires certain versions of jersey, javax.rs.ws-api and
javax.serlet-api in order to run. This change ensures that we use
the correct library versions at runtime. The versions chosen
correspond to those used in Spark 2.4.4 which is the spark version
inside of Presto's spark-core dependency
- Use java 17 bytecode for PoS integration tests

This change expands the OVERRIDE_JDK_DIR environment
configuration to the tempto runner in addition to the existing
behavior for the presto coordinator and workers inside of the
product tests suite. run_tempto.sh was updated to detect the
overridden jdk volume when executing in order to make the
tests run on a compatible JDK since product tests compiles
to java 17 bytecode by default

This makes URI compliance checks in the REST catalog server
more lax during tests in order to comply with the request URIs
generated by the Iceberg rest catalog spec

Due to plugin and dependency updates, some modules' surefire
execution picks up a junit runner instead of the testng one.

This changes forces the use of testng in all presto modules.

presto-testing-mysql still depends on io.airlift:units instead of
com.facebook.airlift:units. This causes issues for the tests which
set MySqlOptions since the builder type only accepts units. Rather than
include the io.airlift:units dependency, this commit removes the use of
the builder options which accept a Duration type in order to allow the
project to compile.

This commit can be reverted if we release a new version of
presto-testing-mysql with the new com.facebook.airlift:units dependency.

Airlift's HttpUriBuilder expects that uris passed to it are
not yet encoded. However, because we use a semicolon ';' to
delimit function names and parameters in the function server
the semicolon which is encoded to %3B gets double-encoded to
%253B by the URI builder (% encodes to %25). Older versions
of Jetty allow this sequence in the URI. Newer servers however
reject this request encoded with %25xxx with an
error stating "AMBIGUOUS_PATH_PARAMETER".

If we avoid using the HttpUriBuilder, we don't double-encode
the semicolon, and the requests will work against the new
server

- remove airline from classpath by excluding it from com.teradata.tpcds:tpcds
- force presto-product-tests to shade airline by disabling
the generation of the dependency-reduced-pom
- change message() to body() in StatementClient to get correct error messages

downgrade guice to 6.0.0 for Java 8 compat

Downgrade guice to 6.0.0. replace all jakarta.inject.Provider with
javax.inject.Provider. Update airlift version to one with guice 6.0.0.
This makes it easier to integrate libraries that use javax with Presto.

This diff contains multiple changes that go together
1. We remove the dependency on presto-main module as it is targetted to compile
   for JDK17 but presto-on-spark* modules compile to JDK8/11.
2. Removing dependency on presto-main means that we have to figure out a solution
   to provide the missing functionality on which we depended. The only way to
   to do this is to duplicate the necessary classes and their dependencies
3. Next, we now have to override http-client,http-server and discovery to be
   compatible with JDK8/11. For this we pin the airlift version to 0.216 and the
   corresponding jetty version to 9.8.45.vXXXXX
4. Next we still need to do some amount of migration of classes as we moved from
   using javax -> jakarta and from io.airlift to com.facebook.airlift. Note that
   because for some of the airlift jars we still use 0.216 so it expects old
   io.airlift classes. So we retain both io.airlift and com.facebook.airlift
5. To manage the duplication we also exclude some classes from duplicate class check
   as the signature of the classes is the same