ARUNACHALAM THIRUPATHI

Presto supports lot of types and ever evolving. for example, GeometryTypes and so on, so this would put the type system in a confused state. So I recommend not to...

One minor comment, it is ok, if you want to defer it.

Shouldn't the right fix be removing the check here https://github.com/Alluxio/alluxio/blob/master/core/client/hdfs/src/main/java/alluxio/hadoop/LocalCacheFileSystem.java#L91 The supported scheme is not used beyond that. Upgrading the dependency after the fix is released. CC: @rongrong who is...

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

Do you know how Trino supports this ? Looks like Trino is not using jayway library. I looked at Jayway library and looks like no commits for the past 3...

When there is an update, Please ping me directly and I will take a look at it again.

I discussed with few Presto folks. The recommendation is to remove the session property, default it to Presto, when the Presto parser fails, use the jayway.

There are conflicts in this branch, can you please rebase and push. I need to run some verification tests (this will take couple of hours of my time) and once...

Looks like there are merge conflicts.

Can you please squash your commits into one as well? Apologies, did not notice there were lot of commits.