[GitHub] incubator-hawq issue #1353: HAWQ-1605. Support INSERT in PXF JDBC plugin

2018-06-08 Thread denalex
Github user denalex commented on the issue: https://github.com/apache/incubator-hawq/pull/1353 @sansanichfb -- can you please approve the PR if it looks good to you ? We are planning to commit it soon. ---

[GitHub] incubator-hawq issue #1353: HAWQ-1605. Support INSERT in PXF JDBC plugin

2018-06-08 Thread leskin-in
Github user leskin-in commented on the issue: https://github.com/apache/incubator-hawq/pull/1353 Hi @sansanichfb, You have mentioned some questions that were not replied. I am a bit confused; could you clarify what exactly is unclear or should be corrected? I have also lef

[GitHub] incubator-hawq issue #1353: HAWQ-1605. Support INSERT in PXF JDBC plugin

2018-06-06 Thread sansanichfb
Github user sansanichfb commented on the issue: https://github.com/apache/incubator-hawq/pull/1353 Hi @kapustor , Sorry for the late reply. I don't have any new comments, there are couple old comments, which weren't replied. Apart from that - looks good. ---

[GitHub] incubator-hawq issue #1353: HAWQ-1605. Support INSERT in PXF JDBC plugin

2018-05-25 Thread kapustor
Github user kapustor commented on the issue: https://github.com/apache/incubator-hawq/pull/1353 Hi @sansanichfb, Any news on this PR? ---

[GitHub] incubator-hawq issue #1353: HAWQ-1605. Support INSERT in PXF JDBC plugin

2018-05-04 Thread kapustor
Github user kapustor commented on the issue: https://github.com/apache/incubator-hawq/pull/1353 Hi @sansanichfb We've removed the notification about the consistency and transactions from the error text. Thank you! ---

[GitHub] incubator-hawq issue #1353: HAWQ-1605. Support INSERT in PXF JDBC plugin

2018-04-24 Thread sansanichfb
Github user sansanichfb commented on the issue: https://github.com/apache/incubator-hawq/pull/1353 @kapustor we can't fail safely when something wrong happens with at least one fragment, without dealing with the complexity of distributed transactions. So transaction handling doesn'

[GitHub] incubator-hawq issue #1353: HAWQ-1605. Support INSERT in PXF JDBC plugin

2018-04-21 Thread kapustor
Github user kapustor commented on the issue: https://github.com/apache/incubator-hawq/pull/1353 Hi @sansanichfb! > Also, I am wondering, which target database patch was tested against? The transactional approach doesn't seem clear to me - if DB doesn't support transactions -

[GitHub] incubator-hawq issue #1353: HAWQ-1605. Support INSERT in PXF JDBC plugin

2018-04-18 Thread sansanichfb
Github user sansanichfb commented on the issue: https://github.com/apache/incubator-hawq/pull/1353 Good job leskin-in! I left some comments. Also, I am wondering, which target database patch was tested against? The transactional approach doesn't seem clear to me - if DB doesn't suppor

[GitHub] incubator-hawq issue #1353: HAWQ-1605. Support INSERT in PXF JDBC plugin

2018-04-09 Thread kongyew
Github user kongyew commented on the issue: https://github.com/apache/incubator-hawq/pull/1353 **This JDBC profile is not officially supported** by Greenplum since it is not tested at all. Use it at your own risks. The interface maybe changed in the future since the current profile

[GitHub] incubator-hawq issue #1353: HAWQ-1605. Support INSERT in PXF JDBC plugin

2018-04-09 Thread deem0n
Github user deem0n commented on the issue: https://github.com/apache/incubator-hawq/pull/1353 Yep, our customers have plenty different data sources, mostly JDBC compliant. This feature makes GPDB a very flexible and friendly player in corporate IT landscape. ---

[GitHub] incubator-hawq issue #1353: HAWQ-1605. Support INSERT in PXF JDBC plugin

2018-04-09 Thread mebelousov
Github user mebelousov commented on the issue: https://github.com/apache/incubator-hawq/pull/1353 This is great feature. We really need to upload data from Greenplum to other databases. ---