[GitHub] incubator-hawq issue #1256: HAWQ-1485. fix exception of decryptPassword twic...

2017-06-14 Thread interma
Github user interma commented on the issue: https://github.com/apache/incubator-hawq/pull/1256 @linwen @zhangh43 @stanlyxiang help to review again, thanks! --- 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] incubator-hawq pull request #1256: HAWQ-1485. fix exception of decryptPasswo...

2017-06-14 Thread interma
Github user interma commented on a diff in the pull request: https://github.com/apache/incubator-hawq/pull/1256#discussion_r122109467 --- Diff: ranger-plugin/admin-plugin/src/main/java/org/apache/hawq/ranger/service/RangerServiceHawq.java --- @@ -112,7 +119,7 @@ private void

[GitHub] incubator-hawq pull request #1256: HAWQ-1485. fix exception of decryptPasswo...

2017-06-14 Thread interma
GitHub user interma opened a pull request: https://github.com/apache/incubator-hawq/pull/1256 HAWQ-1485. fix exception of decryptPassword twice in lookupResource() In my earlier PR, *decryptPassword()* is called in the beginning of *lookupResource()*, but *BaseClient* (a ranger

[GitHub] incubator-hawq pull request #1255: HAWQ-1486. Catch error out on NULL condit...

2017-06-14 Thread Quikling
GitHub user Quikling opened a pull request: https://github.com/apache/incubator-hawq/pull/1255 HAWQ-1486. Catch error out on NULL condition for churl_init_download Please review: @sansanichfb & @shivzone You can merge this pull request into a Git repository by running: $ git

[jira] [Created] (HAWQ-1486) PANIC accessing PXF HDFS table

2017-06-14 Thread John Gaskin (JIRA)
John Gaskin created HAWQ-1486: - Summary: PANIC accessing PXF HDFS table Key: HAWQ-1486 URL: https://issues.apache.org/jira/browse/HAWQ-1486 Project: Apache HAWQ Issue Type: Bug

[GitHub] incubator-hawq issue #1254: HAWQ-1373 - Added feature to reload GUC values u...

2017-06-14 Thread outofmem0ry
Github user outofmem0ry commented on the issue: https://github.com/apache/incubator-hawq/pull/1254 @linwen @stanlyxiang @radarwave - Thank you for the comments and suggestions. @radarwave - Please see the comments inline 1. I think we should keep the legacy command

[GitHub] incubator-hawq issue #1254: HAWQ-1373 - Added feature to reload GUC values u...

2017-06-14 Thread radarwave
Github user radarwave commented on the issue: https://github.com/apache/incubator-hawq/pull/1254 Thanks @outofmem0ry to contribute this, please check below comments. 1. I think we should keep the legacy command to do reload in case it's hard coded in some user cases, print a

[GitHub] incubator-hawq pull request #1253: HAWQ-1485. Use user/password instead of c...

2017-06-14 Thread interma
Github user interma closed the pull request at: https://github.com/apache/incubator-hawq/pull/1253 --- 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 this feature enabled and wishes so, or if the

[jira] [Resolved] (HAWQ-1480) Packing a core file in hawq

2017-06-14 Thread Lin Wen (JIRA)
[ https://issues.apache.org/jira/browse/HAWQ-1480?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Lin Wen resolved HAWQ-1480. --- Resolution: Fixed Fix Version/s: 2.3.0.0-incubating > Packing a core file in hawq >

[GitHub] incubator-hawq issue #1254: HAWQ-1373 - Added feature to reload GUC values u...

2017-06-14 Thread stanlyxiang
Github user stanlyxiang commented on the issue: https://github.com/apache/incubator-hawq/pull/1254 If we add "hawq reload-config", I think there is no need for "hawq stop cluster -u". 2 commands have same function make me more confused. --- If your project is set up for it, you can

[GitHub] incubator-hawq issue #1254: HAWQ-1373 - Added feature to reload GUC values u...

2017-06-14 Thread linwen
Github user linwen commented on the issue: https://github.com/apache/incubator-hawq/pull/1254 Shubham, I think what you've done in this PR is to add a command for hawq, which can reload GUC configs without restarting the system. Currently, this is done by this command "hawq stop

[GitHub] incubator-hawq issue #1253: HAWQ-1485. Use user/password instead of credenti...

2017-06-14 Thread zhangh43
Github user zhangh43 commented on the issue: https://github.com/apache/incubator-hawq/pull/1253 +1 --- 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 this feature enabled and wishes so, or if