[GitHub] incubator-hawq pull request #1171: HAWQ-1359. Add test cases for Ranger supp...

2017-03-15 Thread linwen
Github user linwen closed the pull request at: https://github.com/apache/incubator-hawq/pull/1171 --- 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

[GitHub] incubator-hawq pull request #1171: HAWQ-1359. Add test cases for Ranger supp...

2017-03-15 Thread stanlyxiang
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 "lib/gpfdist.h"

[GitHub] incubator-hawq pull request #1171: HAWQ-1359. Add test cases for Ranger supp...

2017-03-14 Thread linwen
Github user linwen commented on a diff in the pull request: https://github.com/apache/incubator-hawq/pull/1171#discussion_r106084037 --- Diff: src/test/feature/Ranger/test_ranger.cpp --- @@ -24,19 +24,34 @@ #include "lib/command.h" #include "lib/gpfdist.h"

[GitHub] incubator-hawq pull request #1171: HAWQ-1359. Add test cases for Ranger supp...

2017-03-14 Thread linwen
Github user linwen commented on a diff in the pull request: https://github.com/apache/incubator-hawq/pull/1171#discussion_r106080149 --- Diff: src/test/feature/lib/sql_util.cpp --- @@ -224,7 +225,9 @@ const string SQLUtility::generateSQLFile(const string , bool usingDefaul

[GitHub] incubator-hawq pull request #1171: HAWQ-1359. Add test cases for Ranger supp...

2017-03-14 Thread stanlyxiang
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] incubator-hawq pull request #1171: HAWQ-1359. Add test cases for Ranger supp...

2017-03-14 Thread stanlyxiang
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 "lib/gpfdist.h"

[GitHub] incubator-hawq pull request #1171: HAWQ-1359. Add test cases for Ranger supp...

2017-03-14 Thread stanlyxiang
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 "lib/gpfdist.h"

[GitHub] incubator-hawq pull request #1171: HAWQ-1359. Add test cases for Ranger supp...

2017-03-14 Thread linwen
GitHub user linwen opened a pull request: https://github.com/apache/incubator-hawq/pull/1171 HAWQ-1359. Add test cases for Ranger support, combinations of differe… Please review, thanks! You can merge this pull request into a Git repository by running: $ git pull