[GitHub] drill issue #258: DRILL-4091: Support for additional gis operations in gis c...

2017-10-27 Thread k255
Github user k255 commented on the issue: https://github.com/apache/drill/pull/258 It's good that now there's commiter which is aware of the GIS context! The list of functions added in this PR is as follows: ST_Buffer, ST_Contains, ST_Crosses, ST_Difference, ST_Disjoint

[GitHub] drill issue #258: DRILL-4091: Support for additional gis operations in gis c...

2017-10-27 Thread k255
Github user k255 commented on the issue: https://github.com/apache/drill/pull/258 @amansinha100 better later than never! The PR is updated now. @cgivre offered that he could help reviewing this. @joeauty probably in further development we can consider adding geojson support

[GitHub] drill pull request: DRILL-4091: Support for additional gis operati...

2016-02-11 Thread k255
Github user k255 commented on the pull request: https://github.com/apache/drill/pull/258#issuecomment-182857803 Aggregate version of st_union allows merging geometries using 'group by' --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] drill pull request: DRILL-4091: Support for additional gis operati...

2016-01-19 Thread k255
Github user k255 commented on the pull request: https://github.com/apache/drill/pull/258#issuecomment-172858385 Added new functionality to transform spatial reference of geometries (SRID) based on Proj4J. Usage: ST_Transform(geom, srcSRID, tgtSRID) This lets you transform

[GitHub] drill pull request: DRILL-4091: Support for additional gis operati...

2015-11-16 Thread k255
Github user k255 commented on the pull request: https://github.com/apache/drill/pull/258#issuecomment-157026760 this extends DRILL-3914 functionality --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] drill pull request: DRILL-4091: Support for additional gis operati...

2015-11-16 Thread k255
GitHub user k255 opened a pull request: https://github.com/apache/drill/pull/258 DRILL-4091: Support for additional gis operations in gis contrib module Support for commonly used gis functions in gis contrib module: relate, contains, crosses, intersects, touches, difference

[GitHub] drill pull request: [DRILL-3914]: support for geospatial query fun...

2015-10-16 Thread k255
Github user k255 commented on a diff in the pull request: https://github.com/apache/drill/pull/191#discussion_r42217546 --- Diff: contrib/gis/src/main/java/org/apache/drill/exec/expr/fn/impl/gis/STAsText.java --- @@ -0,0 +1,58 @@ +/** + * Licensed to the Apache Software

[GitHub] drill pull request: [DRILL-3914]: support for geospatial query fun...

2015-10-15 Thread k255
Github user k255 commented on the pull request: https://github.com/apache/drill/pull/191#issuecomment-148369490 Fixed bug with complex geometries caused by to small buffer. Now it's possible to build more complex geometries.Would be nice if somebody could check the way I handled

[GitHub] drill pull request: [DRILL-3914]: support for geospatial query fun...

2015-10-08 Thread k255
GitHub user k255 opened a pull request: https://github.com/apache/drill/pull/191 [DRILL-3914]: support for geospatial query functionality Sample dataset is provided on classpath, after building from fork repository, you can query it like: select * from cp.`sample-data/CA

[GitHub] drill pull request: [DRILL-3914]: support for geospatial query fun...

2015-10-08 Thread k255
Github user k255 commented on the pull request: https://github.com/apache/drill/pull/191#issuecomment-146679625 I added some general tests to check if geometry functions work as expected. I'm happy that you like it. Currently it's quite simple but it can grow. One