Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1384
Though some similar problems(indent, format, free Uri, etc) were not
pointed out, pls go through the code and fix all similar problems above.
---
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1384#discussion_r209506292
--- Diff: contrib/exthdfs/exthdfs.c ---
@@ -0,0 +1,469 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1333
+1
---
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1324
+1
---
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1314
+1
---
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1314#discussion_r154280568
--- Diff: src/backend/access/external/plugstorage.c ---
@@ -0,0 +1,533 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1310#discussion_r151854548
--- Diff:
src/backend/resourcemanager/resourcebroker/resourcebroker_LIBYARN.c ---
@@ -782,7 +782,7 @@ int handleRB2RM_ClusterReport(void
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1310#discussion_r151854472
--- Diff:
src/backend/resourcemanager/resourcebroker/resourcebroker_LIBYARN.c ---
@@ -773,7 +773,7 @@ int handleRB2RM_ClusterReport(void
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1280#discussion_r134641796
--- Diff: src/bin/gpcheckhdfs/gpcheckhdfs.c ---
@@ -6,9 +6,9 @@
* to you under the Apache License, Version 2.0 (the
* "Li
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1279
+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
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1277
@wangbincmss
Since hawq opened source and use apache JIRA to tracker all the changes, so
for every pull request please create a apache jira first for your pull request
like other
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1265#discussion_r127112526
--- Diff: depends/libhdfs3/src/client/HttpClient.h ---
@@ -0,0 +1,155
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1258
+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
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 se
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1226
I tested on my rhel7. After build rps rpm, there are 2 commands after the
rpm generate.
`
[INFO] + cd
/home/xsheng/hawq/incubator-hawq/ranger-plugin/target/rpm/hawq-ranger
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1250#discussion_r120274829
--- Diff:
ranger-plugin/service/src/main/java/org/apache/hawq/ranger/authorization/RangerHawqPluginResource.java
---
@@ -45,8 +47,43
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1250#discussion_r120044242
--- Diff:
ranger-plugin/service/src/main/java/org/apache/hawq/ranger/authorization/RangerHawqPluginResource.java
---
@@ -45,8 +47,43
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1250#discussion_r120044685
--- Diff:
ranger-plugin/service/src/main/java/org/apache/hawq/ranger/authorization/RangerHawqPluginResource.java
---
@@ -45,8 +47,43
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1250#discussion_r120043142
--- Diff:
ranger-plugin/service/src/main/java/org/apache/hawq/ranger/authorization/Utils.java
---
@@ -68,6 +77,30 @@ public static String
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1250#discussion_r120042673
--- Diff:
ranger-plugin/service/src/main/java/org/apache/hawq/ranger/authorization/RangerHawqPluginResource.java
---
@@ -45,8 +47,43
GitHub user stanlyxiang opened a pull request:
https://github.com/apache/incubator-hawq/pull/1248
HAWQ-1476. Augment enable-ranger-plugin.sh to support kerberos.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/stanlyxiang
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1245#discussion_r118834619
--- Diff: ranger-plugin/scripts/enable-ranger-plugin.sh ---
@@ -20,7 +20,7 @@
#
function usage() {
- echo "USAGE: e
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1235#discussion_r115425270
--- Diff: ranger-plugin/scripts/enable-ranger-plugin.sh ---
@@ -104,8 +157,18 @@ function get_hawq_password() {
done
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1235#discussion_r115424467
--- Diff: tools/bin/hawq_ctl ---
@@ -387,6 +387,10 @@ class HawqInit:
(self.GPHOME, self.standby_host_name, self.GPHOME
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1235#discussion_r115425328
--- Diff: ranger-plugin/scripts/enable-ranger-plugin.sh ---
@@ -237,9 +305,13 @@ main() {
if [[ $# -lt 1 ]]; then
usage
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1234#discussion_r115410444
--- Diff: src/backend/libpq/rangerrest.c ---
@@ -383,58 +383,100 @@ static size_t write_callback(char *contents, size_t
size, size_t nitems
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1234#discussion_r115410368
--- Diff: src/backend/libpq/rangerrest.c ---
@@ -383,58 +383,100 @@ static size_t write_callback(char *contents, size_t
size, size_t nitems
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1231
Merged into master. Close this PR.
---
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 stanlyxiang closed the pull request at:
https://github.com/apache/incubator-hawq/pull/1231
---
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
Github user stanlyxiang closed the pull request at:
https://github.com/apache/incubator-hawq/pull/1229
---
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
Github user stanlyxiang closed the pull request at:
https://github.com/apache/incubator-hawq/pull/1228
---
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
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1229
Merged into master. Close this PR.
---
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 stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1228
Merged into master. Close this PR.
---
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 stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1228#discussion_r114693788
--- Diff: tools/bin/hawq_ctl ---
@@ -737,25 +739,28 @@ class HawqStart:
logger.info("Segments started success
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1229#discussion_r114692327
--- Diff: tools/bin/hawqstate ---
@@ -135,12 +138,14 @@ def show_brief_status(hawq_site, segment_list,
standby_host):
else
GitHub user stanlyxiang opened a pull request:
https://github.com/apache/incubator-hawq/pull/1231
HAWQ-1452. Remove hawq_rps_address_suffix and hawq_rps_address_host iâ¦
â¦n hawq-site.xml to simplify configuration for RPS High Availability
You can merge this pull request
GitHub user stanlyxiang opened a pull request:
https://github.com/apache/incubator-hawq/pull/1229
HAWQ-1451. HAWQ state should be able to report the status of both RPSâ¦
⦠and standby RPS
You can merge this pull request into a Git repository by running:
$ git pull https
GitHub user stanlyxiang opened a pull request:
https://github.com/apache/incubator-hawq/pull/1228
HAWQ-1449. HAWQ start/stop cluster should be able to start/stop RPS oâ¦
â¦n standby node
You can merge this pull request into a Git repository by running:
$ git pull https
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1228
@interma @linwen @zhangh43 @ictmalili pls review, 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
GitHub user stanlyxiang opened a pull request:
https://github.com/apache/incubator-hawq/pull/1226
HAWQ-1447. Fix ranger build failure
@interma @zhangh43 @linwen @ictmalili pls review, thanks
You can merge this pull request into a Git repository by running:
$ git pull https
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1219
Merged into master. Close this PR.
---
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 stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1219
@linwen Yes. Currently only support percentage.
---
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 user stanlyxiang opened a pull request:
https://github.com/apache/incubator-hawq/pull/1219
HAWQ-1433. ALTER RESOURCE QUEUE DDL does not check the format of attrâ¦
â¦ibute MEMORY_CLUSTER_LIMIT and CORE_CLUSTER_LIMIT
You can merge this pull request into a Git repository
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1217
Merged into master. Close this PR.
---
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 stanlyxiang closed the pull request at:
https://github.com/apache/incubator-hawq/pull/1217
---
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
GitHub user stanlyxiang opened a pull request:
https://github.com/apache/incubator-hawq/pull/1217
HAWQ-1430. Update ranger related log level to avoid log flood
@interma @zhangh43 @ictmalili please help to review. thanks.
You can merge this pull request into a Git repository
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1188
We CAN copy. But better to GENERATE the version.json file.
---
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
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1186#discussion_r107836478
--- Diff: tools/bin/hawq_ctl ---
@@ -1041,14 +1041,14 @@ class HawqStop:
logger.info("Segments %s success
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1171#discussion_r106091426
--- Diff: src/test/feature/Ranger/test_ranger.cpp ---
@@ -24,19 +24,34 @@
#include "lib/command.h"
#include &quo
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1171#discussion_r106078845
--- Diff: ranger-plugin/.gitignore ---
@@ -1,2 +1,3 @@
target/
Makefile.global
+/bin/
--- End diff --
is there any
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1171#discussion_r106079103
--- Diff: src/test/feature/Ranger/test_ranger.cpp ---
@@ -24,19 +24,34 @@
#include "lib/command.h"
#include &quo
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1171#discussion_r106078042
--- Diff: src/test/feature/Ranger/test_ranger.cpp ---
@@ -24,19 +24,34 @@
#include "lib/command.h"
#include &quo
GitHub user stanlyxiang opened a pull request:
https://github.com/apache/incubator-hawq/pull/1166
HAWQ-1362. remove SYS_CATALINA_HOME to stay consistent with build cheâ¦
â¦ck CATALINA_HOME
You can merge this pull request into a Git repository by running:
$ git pull https
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1164
Merged into master. Close this PR.
---
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 stanlyxiang closed the pull request at:
https://github.com/apache/incubator-hawq/pull/1164
---
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
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1164#discussion_r104841513
--- Diff: ranger-plugin/conf/rps.properties ---
@@ -29,4 +29,7 @@ RPS_SHUTDOWN_PORT=8405
RANGER_HAWQ_INSTANCE=hawq
# version
GitHub user stanlyxiang opened a pull request:
https://github.com/apache/incubator-hawq/pull/1164
HAWQ-1362. Add RPS start/stop script in HAWQ start/stop script
please review, thanks.
You can merge this pull request into a Git repository by running:
$ git pull https
GitHub user stanlyxiang opened a pull request:
https://github.com/apache/incubator-hawq/pull/1147
HAWQ-1350. Fix ranger-plugin makefile error
@interma @ictmalili @linwen
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user stanlyxiang closed the pull request at:
https://github.com/apache/incubator-hawq/pull/1143
---
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
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1143
Merged into master. Close this PR.
---
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 stanlyxiang closed the pull request at:
https://github.com/apache/incubator-hawq/pull/1137
---
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
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1137
Merged into master. Close this PR.
---
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 stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1143#discussion_r102678227
--- Diff: config/tomcat.m4 ---
@@ -0,0 +1,59 @@
+dnl
+dnl tomcat.m4: Locates the Apache Tomcat/6.0.x and its scripts and jars.
+dnl
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1143
Yes. This is for ranger support only. In our work, we avoid to confuse
users with ranger and ranger-plugin. Because users should install ranger
(apache ranger) before use HAWQ
GitHub user stanlyxiang opened a pull request:
https://github.com/apache/incubator-hawq/pull/1143
HAWQ-1350. Add --enable-rps option to build ranger-plugin when build â¦
â¦hawq.
please help to review, thanks. @ictmalili @linwen @zhangh43 @interma @wcl14
You can merge
Github user stanlyxiang commented on the pull request:
https://github.com/apache/incubator-hawq/commit/b37f18768dceec5600f8a90f6f8b17045954629c#commitcomment-21010171
Agree with wengyanqing. And we'd better rebase multiple commits into one
commit if we are doing one thing, except
GitHub user stanlyxiang opened a pull request:
https://github.com/apache/incubator-hawq/pull/1137
HAWQ-1349. make uninstall failed for make target gpmodcat_uninstall
just a typo, please help to review. thanks. @ictmalili @zhangh43 @linwen
@interma
You can merge this pull request
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1120
Merged into master. Close this PR.
---
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 stanlyxiang closed the pull request at:
https://github.com/apache/incubator-hawq/pull/1120
---
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
GitHub user stanlyxiang opened a pull request:
https://github.com/apache/incubator-hawq/pull/1120
HAWQ-1329. pg_catalog view fallback failed.
@zhangh43 @linwen
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/stanlyxiang
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1106
+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
Github user stanlyxiang closed the pull request at:
https://github.com/apache/incubator-hawq/pull/1103
---
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
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1103
Merged into master. Close this PR.
---
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 stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1103
@zhangh43 @ictmalili @xunzhangplease help to review. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
GitHub user stanlyxiang opened a pull request:
https://github.com/apache/incubator-hawq/pull/1103
HAWQ-1292. Change GUC enable_ranger(bool) to a text GUC(hawq_acl_typeâ¦
â¦), which can allow other kinds of ACL.
You can merge this pull request into a Git repository by running
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1097
+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
Github user stanlyxiang closed the pull request at:
https://github.com/apache/incubator-hawq/pull/1095
---
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
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1095
merged into master. closed.
---
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
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1095
cc @ictmalili @interma
---
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
GitHub user stanlyxiang opened a pull request:
https://github.com/apache/incubator-hawq/pull/1095
HAWQ-1276. hawq should error out directly when ranger plugin service â¦
â¦is unavailable, not a warning.
You can merge this pull request into a Git repository by running:
$ git
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1094
+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
Github user stanlyxiang closed the pull request at:
https://github.com/apache/incubator-hawq/pull/1088
---
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
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1088
merged into master. closed.
---
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
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1090#discussion_r96338779
--- Diff:
ranger-plugin/integration/service/src/test/java/org/apache/hawq/ranger/integration/service/tests/RPSRequest.java
---
@@ -0,0 +1,41
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1090#discussion_r96338796
--- Diff:
ranger-plugin/integration/service/src/test/java/org/apache/hawq/ranger/integration/service/tests/RPSResponse.java
---
@@ -0,0 +1,23
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1090#discussion_r96338825
--- Diff:
ranger-plugin/integration/service/src/test/java/org/apache/hawq/ranger/integration/service/tests/Utils.java
---
@@ -0,0 +1,57
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1089
+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
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1088#discussion_r96181179
--- Diff: src/backend/libpq/rangerrest.c ---
@@ -366,7 +365,7 @@ static size_t write_callback(char *contents, size_t
size, size_t nitems
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1088#discussion_r96179729
--- Diff: src/backend/libpq/rangerrest.c ---
@@ -366,7 +365,7 @@ static size_t write_callback(char *contents, size_t
size, size_t nitems
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1087
+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
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1088
cc @interma @zhangh43 @linwen @ictmalili @wcl14
---
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 user stanlyxiang opened a pull request:
https://github.com/apache/incubator-hawq/pull/1088
HAWQ-1276. The error message is not friendly when ranger plugin serviâ¦
â¦ce is unavailable.
You can merge this pull request into a Git repository by running:
$ git pull https
Github user stanlyxiang commented on the issue:
https://github.com/apache/incubator-hawq/pull/1086
LGTM , +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
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1086#discussion_r96110581
--- Diff: .travis.yml ---
@@ -38,6 +39,7 @@ script:
- make -j$(sysctl -n hw.ncpu)
- make feature-test
- make feature-test
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1081#discussion_r95751617
--- Diff: src/backend/libpq/rangerrest.c ---
@@ -463,7 +464,12 @@ int check_privilege_from_ranger(List *request_list,
List *result_list
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1081#discussion_r95715330
--- Diff: src/backend/libpq/rangerrest.c ---
@@ -381,29 +449,38 @@ int call_ranger_rest(CURL_HANDLE curl_handle, const
char* request
GitHub user stanlyxiang opened a pull request:
https://github.com/apache/incubator-hawq/pull/1082
HAWQ-1256. non-superuser connect to db will do aclcheck through rangeâ¦
â¦r and curl handle haven't been initialized.
You can merge this pull request into a Git repository by running
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1081#discussion_r95528917
--- Diff: src/backend/libpq/rangerrest.c ---
@@ -68,51 +69,99 @@ static void getClientIP(char *remote_host
Github user stanlyxiang commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1081#discussion_r95531777
--- Diff: src/backend/libpq/rangerrest.c ---
@@ -381,29 +449,38 @@ int call_ranger_rest(CURL_HANDLE curl_handle, const
char* request
Github user stanlyxiang closed the pull request at:
https://github.com/apache/incubator-hawq/pull/1079
---
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
1 - 100 of 171 matches
Mail list logo