Github user ChrisSandison commented on the issue:
https://github.com/apache/drill/pull/1201
@cgivre added comments
---
Github user ChrisSandison commented on the issue:
https://github.com/apache/drill/pull/1201
@cgivre do you mean including that in this PR?
---
Github user ChrisSandison commented on the issue:
https://github.com/apache/drill/pull/1201
@cgivre I agree. Would you be able to open a JIRA ticket with any details
of the work that @k255 was doing for it?
---
Github user ChrisSandison commented on the issue:
https://github.com/apache/drill/pull/1201
@cgivre I'm also needing to use ST_DWithin
(https://postgis.net/docs/ST_DWithin.html). I'm going to add it to this PR.
---
Github user ChrisSandison commented on the issue:
https://github.com/apache/drill/pull/1201
@cgivre I had some issues between this and a previous branch I was working
on, but I fixed it by upgrade the base image that I was building it on. I ended
up building it on maven:3.5.3-jdk-8
---
GitHub user ChrisSandison opened a pull request:
https://github.com/apache/drill/pull/1034
Adding asGeoJSON function
Fixes #5960
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ChrisSandison/drill DRILL-5960
Alternatively you
GitHub user ChrisSandison opened a pull request:
https://github.com/apache/drill/pull/1036
Adding ST_AsJSON functionality
Fixes #DRILL-5962
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ChrisSandison/drill DRILL-5962
Github user ChrisSandison commented on a diff in the pull request:
https://github.com/apache/drill/pull/1036#discussion_r151142574
--- Diff:
contrib/gis/src/main/java/org/apache/drill/exec/expr/fn/impl/gis/STAsJSON.java
---
@@ -0,0 +1,58 @@
+/**
+ * Licensed to the Apache
Github user ChrisSandison commented on a diff in the pull request:
https://github.com/apache/drill/pull/1034#discussion_r152008246
--- Diff:
contrib/gis/src/main/java/org/apache/drill/exec/expr/fn/impl/gis/STAsGeoJson.java
---
@@ -0,0 +1,63 @@
+/*
+ * Licensed to the
GitHub user ChrisSandison opened a pull request:
https://github.com/apache/drill/pull/1044
DRILL-5980: Making queryType param for REST client case-insensitive
Fixes #5980
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user ChrisSandison commented on a diff in the pull request:
https://github.com/apache/drill/pull/258#discussion_r178619522
--- Diff:
contrib/gis/src/main/java/org/apache/drill/exec/expr/fn/impl/gis/STXFunc.java
---
@@ -0,0 +1,64 @@
+/**
+ * Licensed to the Apache
Github user ChrisSandison commented on a diff in the pull request:
https://github.com/apache/drill/pull/258#discussion_r178652351
--- Diff:
contrib/gis/src/main/java/org/apache/drill/exec/expr/fn/impl/gis/STUnionAggregate.java
---
@@ -0,0 +1,114 @@
+/**
+ * Licensed to
GitHub user ChrisSandison opened a pull request:
https://github.com/apache/drill/pull/1201
DRILL-4091: Support for additional gis operations in gis contrib module
This based off of PR https://github.com/apache/drill/pull/258. I have
attempted to address the comments from the PR
13 matches
Mail list logo
