Pratyaksh Sharma comments

Results 93 comments of


                                            Pratyaksh Sharma

Removed unsupported data types on read/write paths in jdbc connectors

@NikhilCollooru Can you please take a pass.

Removed unsupported data types on read/write paths in jdbc connectors

> We actually have dependency of this client to fall back from TIMESTAMP_WITH_TIME_ZONE to TIMESTAMP (or TIME_WITH_TIME_ZONE to TIME). Deleting these two types could cause regression. @highker Where do we...

Removed unsupported data types on read/write paths in jdbc connectors

@highker ping.

Removed unsupported data types on read/write paths in jdbc connectors

Got you, closing this PR. Thank you for reviewing.

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

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

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

@zhenxiao @tdcmeehan ping!

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

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

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

@tdcmeehan let us merge this!

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

@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...

added getScheme() implementation in PrestoS3FileSystem.java

@imjalpreet Can you please take the first pass here?