Add Presto Java dev container

[mblanco-denodo]

Description

Added a dev container to develop Presto Java

Motivation and Context

Ease Presto development

Impact

Test Plan

Manual testing:

• Created dev container in Intellij IDEA
• Compile && mvn clean install both in java 8 and java 17 in terminal and IDEA
• Build docs

Contributor checklist

• Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
• PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
• Documented new properties (with its default value), SQL syntax, functions, or other functionality.
• If release notes are required, they follow the release notes guidelines.
• Adequate tests were added if applicable.
• CI passed.

Release Notes

Please follow release notes guidelines and fill in the release notes below.

== NO RELEASE NOTE ==

[ZacBlanco]

This is great. I think we should consider making this container also compatible with the presto CPP development to have a unified experience.

I wonder if we could get some of the steps from presto-native-execution/scripts/setup-ubuntu to be run here as well.

Also, should we consider using something like sdkman rather than the system installs for java?

[ZacBlanco]

what's this for?

[mblanco-denodo]

Doc prerrequisites https://github.com/prestodb/presto/blob/master/presto-docs/README.md

[ZacBlanco]

This is quite a large install. Is installing tex actually required?

[mblanco-denodo]

Yes, its a large install but it is in the doc prerrequisites: https://github.com/prestodb/presto/blob/master/presto-docs/README.md

[ZacBlanco]

The default build does not really need the latex bits. You need them for the PDF output, but it is seldom used. I would opt to leave it out, just to keep the base image small

[mblanco-denodo]

Hi! I've removed that from the container, can you review again please?

[mblanco-denodo]

For presto native I am going to create another PR. In my experience it is better to have separate dev containers.

[steveburnett]

@mblanco-denodo, there have been some fixes to the CI tests since the last activity on this PR. If you rebase, these pending tests might pass.

[mblanco-denodo]

For presto native I am going to create another PR. In my experience it is better to have separate dev containers.

This is the one i've created some time ago for C++. Might need a refactor as the requisites for prestissimo are continuously evolving. Can you take a look into that as well?
#25358