Removed presto-geospatial module and moved the types and functions to presto-main

[pratyakshsharma]

Relevant slack discussion here - https://prestodb.slack.com/archives/C07JH9WMQ/p1651146423521239

Test plan - (Please fill in how you tested your changes)

Please make sure your submission complies with our Development, Formatting, and Commit Message guidelines. Don't forget to follow our attribution guidelines for any code copied from other projects.

Fill in the release notes towards the bottom of the PR description. See Release Notes Guidelines for details.

== RELEASE NOTES ==

General Changes
* Removed presto-geospatial module 
* Added the types and SqlFunctions defined by GeoPlugin to `BuiltInTypeAndFunctionNamespaceManager.java` for registering
* Moved the relevant classes to presto-main to avoid too many circular dependency issues which would have occurred if movement was done to presto-common rather
* Moved the relevant test classes to presto-main and presto-memory

[linux-foundation-easycla[bot]]

The committers listed above are authorized under a signed CLA.

• :white_check_mark: login: pratyakshsharma / name: Pratyaksh Sharma (3614ccb8e5a165d707f946567d6965c2ca9e1a75)

[pratyakshsharma]

@fgwang7w please take a look and add any relevant stakeholders.

cc @imjalpreet @agrawalreetika

[fgwang7w]

Hi @zhenxiao I've given 1st pass of code review, please help review as well. Hi @tdcmeehan This is a straight forward lib moving from one module to presto-main, I suppose there's a need to document the changes for release notes, any suggestion? Thanks

[pratyakshsharma]

@zhenxiao @tdcmeehan ping!

[tdcmeehan]

@pratyakshsharma please fix the merge conflicts, thanks!

[pratyakshsharma]

fixed the merge conflicts. Verified the changes using show functions; command and by invoking few simple functions as in the attached screenshot .

[pratyakshsharma]

@tdcmeehan let us merge this!

[arunthirupathi]

The release notes are not user facing. What should the user do ? User need not register GeoPlugin anymore if I understand it correctly. Can you please give me an updated release note ?

[pratyakshsharma]

@arunthirupathi sure. Actually the user facing release notes are present in the commit message body. Providing them as below -

1. presto-geospatial module has been removed and GeoPlugin class has been deleted.
2. User does not need to do anything now to make use of presto's geospatial functions. All the functions are auto registered, one can simply start using them after starting the server.

Does this look good?