Github user kdunn926 commented on the issue:
https://github.com/apache/incubator-hawq/pull/700
Well, @ivannovick - looks like we can either merge this code in the [GPDB
PR](https://github.com/greenplum-db/postgis/pull/1) or leave yet another
differentiating product feature on the
Github user edespino commented on the issue:
https://github.com/apache/incubator-hawq/pull/700
@kdunn926 - As this point since the code is GPLv2, this PR is not
compatible with the Apache HAWQ (incubating) code base. It makes sense to
close this PR and find an alternate (non-Apache)
Github user kdunn926 commented on the issue:
https://github.com/apache/incubator-hawq/pull/700
@liming01, agreed, performance will not be optimal without indexes. I
believe there is value in this PR regardless, until
[HAWQ-303](https://issues.apache.org/jira/browse/HAWQ-303) is
Github user liming01 commented on the issue:
https://github.com/apache/incubator-hawq/pull/700
@kdunn926, if spatial index can not support, then the query performance
will be very slow. What we can offer will be far away from user's expectation.
Need to think a workaround.
---
If
Github user kdunn926 commented on the issue:
https://github.com/apache/incubator-hawq/pull/700
@liming01 - you are correct, because HAWQ doesn't support indexes, this
work has the same limitation. All the spatial operation tests completed
successfully, with the exception of spatial
Github user liming01 commented on the issue:
https://github.com/apache/incubator-hawq/pull/700
@kdunn926, one question:
Because hawq doesn't support index at present, how can you create spatial
index? Where to store the index data? Did you test it(the statement CREATE
INDEX is
Github user changleicn commented on the issue:
https://github.com/apache/incubator-hawq/pull/700
It can be planned after the first release.
---
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 not have
Github user vVineet commented on the issue:
https://github.com/apache/incubator-hawq/pull/700
@changleicn - what do you suggest as the next steps on this?
---
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
Github user kdunn926 commented on the issue:
https://github.com/apache/incubator-hawq/pull/700
That makes sense. I think it would be great to target the same code base as
the one GPDB uses (original PR) so both projects benefit from features without
having to keep two repos in sync
Github user changleicn commented on the issue:
https://github.com/apache/incubator-hawq/pull/700
Great to have this feature in hawq.
I just checked the license (http://postgis.net/), it is GPLv2 which is not
compatible with apache license. This is similar to PL/R. so I think
Github user kdunn926 commented on the issue:
https://github.com/apache/incubator-hawq/pull/700
Hey @xunzhang -
1. I'm validating the build procedure right now. I'll provide it once I've
replicated it again.
2. To install in my HAWQ sandbox, I actually used a modified
Github user xunzhang commented on the issue:
https://github.com/apache/incubator-hawq/pull/700
Hi @kdunn926 ,
1. Does this pull request contain the logic of configuration and building
`Postgis` when compiling hawq? If so, can you separate this part into another
commit for
12 matches
Mail list logo
