Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1224#discussion_r113817022
--- Diff:
pxf/pxf-hive/src/main/java/org/apache/hawq/pxf/plugins/hive/HiveDataFragmenter.java
---
@@ -466,7 +466,14 @@ private boolean
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1224#discussion_r113578883
--- Diff:
pxf/pxf-hive/src/main/java/org/apache/hawq/pxf/plugins/hive/HiveDataFragmenter.java
---
@@ -466,7 +466,14 @@ private boolean
GitHub user kavinderd opened a pull request:
https://github.com/apache/incubator-hawq/pull/1224
HAWQ-1440. Support ANALYZE for all Hive External Tables
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/kavinderd/incubator-hawq
Github user kavinderd closed the pull request at:
https://github.com/apache/incubator-hawq/pull/1189
---
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 user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1216#discussion_r90746
--- Diff:
pxf/pxf-api/src/main/java/org/apache/hawq/pxf/api/utilities/Utilities.java ---
@@ -209,7 +209,8 @@ public static boolean useAggBridge
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1189#discussion_r108538619
--- Diff: src/backend/executor/nodeAgg.c ---
@@ -1950,6 +1950,21 @@ ExecInitAgg(Agg *node, EState *estate, int eflags)
* initialize child
GitHub user kavinderd opened a pull request:
https://github.com/apache/incubator-hawq/pull/1189
HAWQ-1409. Send AGG-TYPE header to PXF
This change is mean to be a proof of concept that pushing down
aggregate function information from HAWQ to the underlying external
storage
Github user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/1104
+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
Github user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/808
@huor @paul-guo- Can you look at this PR when you get a chance. I need one
more +1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1076#discussion_r98277093
--- Diff:
pxf/pxf-hive/src/main/java/org/apache/hawq/pxf/plugins/hive/utilities/HiveUtilities.java
---
@@ -432,32 +447,144 @@ public static
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1076#discussion_r98115361
--- Diff: src/bin/gpfusion/gpbridgeapi.c ---
@@ -68,7 +68,6 @@ void free_token_resources(PxfInputData *inputData);
Datum gpbridge_import
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1076#discussion_r98115518
--- Diff: src/backend/catalog/external/externalmd.c ---
@@ -631,3 +630,79 @@ static int ComputeTypeMod(Oid typeOid, const char
*colname, int
Github user kavinderd closed the pull request at:
https://github.com/apache/incubator-hawq/pull/1075
---
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 user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1102#discussion_r98057448
--- Diff: pxf/pxf-service/src/scripts/pxf-service ---
@@ -193,39 +210,83 @@ function doInit()
return 0
fi
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1102#discussion_r98055597
--- Diff: pxf/pxf-service/src/scripts/pxf-service ---
@@ -40,6 +34,29 @@ else
source $env_script
fi
+pxf_user=$PXF_USER
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1102#discussion_r98051363
--- Diff: pxf/Makefile ---
@@ -65,3 +75,9 @@ doc:
.PHONY: tomcat
tomcat:
./gradlew tomcatRpm $(BUILD_PARAMS)
+
+install
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1102#discussion_r98055816
--- Diff: pxf/pxf-service/src/scripts/pxf-service ---
@@ -193,39 +210,83 @@ function doInit()
return 0
fi
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1102#discussion_r98056223
--- Diff: pxf/pxf-service/src/scripts/pxf-service ---
@@ -193,39 +210,83 @@ function doInit()
return 0
fi
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1102#discussion_r98058834
--- Diff: pxf/pxf-service/src/scripts/pxf-service ---
@@ -193,39 +210,83 @@ function doInit()
return 0
fi
Github user kavinderd closed the pull request at:
https://github.com/apache/incubator-hawq/pull/1064
---
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 user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1075#discussion_r95034235
--- Diff:
pxf/pxf-service/src/main/java/org/apache/hawq/pxf/service/utilities/SecuredHDFS.java
---
@@ -53,6 +53,10 @@
public static void
GitHub user kavinderd opened a pull request:
https://github.com/apache/incubator-hawq/pull/1075
HAWQ-762. Login to kerberos if credentials are no longer valid
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/kavinderd/incubator
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1075#discussion_r95010426
--- Diff:
pxf/pxf-service/src/main/java/org/apache/hawq/pxf/service/utilities/SecuredHDFS.java
---
@@ -53,6 +53,10 @@
public static void
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1064#discussion_r94677406
--- Diff:
pxf/pxf-hive/src/main/java/org/apache/hawq/pxf/plugins/hive/HiveORCSerdeResolver.java
---
@@ -102,11 +104,19 @@ void initSerde
GitHub user kavinderd opened a pull request:
https://github.com/apache/incubator-hawq/pull/1064
HAWQ-1215. Support Complextypes with HiveORC
Preliminary PR for complex type support in Hive
You can merge this pull request into a Git repository by running:
$ git pull https
Github user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/1039
So the delimiter we were previously explicitly asking for in the DDL will
just be retrieved internally?
---
If your project is set up for it, you can reply to this email and have your
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1039#discussion_r91353048
--- Diff:
pxf/pxf-hive/src/main/java/org/apache/hawq/pxf/plugins/hive/HiveResolver.java
---
@@ -79,7 +79,7 @@
private String serdeName
Github user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/1021
Nice work
---
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 kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1002#discussion_r88318669
--- Diff: src/backend/access/transam/varsup.c ---
@@ -474,74 +483,128 @@ ResetExternalObjectId(void)
/*
* master_highest_used_oid
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1002#discussion_r88310416
--- Diff: src/backend/access/transam/varsup.c ---
@@ -474,74 +483,128 @@ ResetExternalObjectId(void)
/*
* master_highest_used_oid
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/981#discussion_r88100166
--- Diff: src/backend/access/external/pxffilters.c ---
@@ -496,23 +629,116 @@ opexpr_to_pxffilter(OpExpr *expr, PxfFilterDesc
*filter
Github user kavinderd closed the pull request at:
https://github.com/apache/incubator-hawq/pull/969
---
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 user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/969#discussion_r84760743
--- Diff:
pxf/pxf-hive/src/main/java/org/apache/hawq/pxf/plugins/hive/HiveFilterBuilder.java
---
@@ -61,8 +61,11 @@ public HiveFilterBuilder
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/969#discussion_r84744935
--- Diff:
pxf/pxf-api/src/main/java/org/apache/hawq/pxf/api/FilterParser.java ---
@@ -279,25 +296,110 @@ int safeToInt(Long value) throws
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/969#discussion_r84528951
--- Diff:
pxf/pxf-hbase/src/main/java/org/apache/hawq/pxf/plugins/hbase/HBaseFilterBuilder.java
---
@@ -229,8 +233,20 @@ private
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/968#discussion_r84334739
--- Diff: src/backend/access/external/pxffilters.c ---
@@ -406,6 +406,22 @@ pxf_serialize_filter_list(List *expressionItems
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/968#discussion_r84333580
--- Diff: src/backend/access/external/test/pxffilters_test.c ---
@@ -507,6 +542,23 @@ void test__pxf_serialize_filter_list__oneFilter(void
**state
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/968#discussion_r8414
--- Diff: src/backend/access/external/test/pxffilters_test.c ---
@@ -323,6 +323,30 @@ OpExpr* build_op_expr(void* left, void* right, int op
GitHub user kavinderd opened a pull request:
https://github.com/apache/incubator-hawq/pull/969
HAWQ-1100. Support for new filter string
- constant data type is now of the form "csd"
You can merge this pull request into a Git repository by running:
$ git pull https://
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/958#discussion_r83111789
--- Diff: src/backend/access/external/pxffilters.c ---
@@ -31,14 +31,14 @@
#include "utils/guc.h"
#include "ut
Github user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/959
+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
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/959#discussion_r83110560
--- Diff: src/bin/psql/describe.c ---
@@ -4316,10 +4316,10 @@ describePxfTable(const char *profile, const char
*pattern, bool verbose
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/959#discussion_r83109712
--- Diff: src/bin/psql/describe.c ---
@@ -4316,10 +4316,10 @@ describePxfTable(const char *profile, const char
*pattern, bool verbose
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/958#discussion_r83108994
--- Diff: src/backend/access/external/pxffilters.c ---
@@ -135,7 +135,40 @@ dbop_pxfop_map pxf_supported_opr[] =
{1097 /* date_gt
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/906#discussion_r83103736
--- Diff:
pxf/pxf-hive/src/main/java/org/apache/hawq/pxf/plugins/hive/HiveDataFragmenter.java
---
@@ -404,24 +411,41 @@ private String
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/906#discussion_r83101774
--- Diff:
pxf/pxf-hbase/src/main/java/org/apache/hawq/pxf/plugins/hbase/HBaseFilterBuilder.java
---
@@ -122,25 +130,23 @@ public Filter
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/906#discussion_r83100399
--- Diff:
pxf/pxf-hbase/src/main/java/org/apache/hawq/pxf/plugins/hbase/HBaseFilterBuilder.java
---
@@ -52,12 +52,15 @@
*/
public class
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/906#discussion_r83098043
--- Diff:
pxf/pxf-api/src/main/java/org/apache/hawq/pxf/api/FilterParser.java ---
@@ -396,4 +387,12 @@ private Operation reverseOp(Operation
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/906#discussion_r83097937
--- Diff:
pxf/pxf-api/src/main/java/org/apache/hawq/pxf/api/FilterParser.java ---
@@ -396,4 +387,12 @@ private Operation reverseOp(Operation
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/906#discussion_r83097391
--- Diff:
pxf/pxf-api/src/main/java/org/apache/hawq/pxf/api/FilterParser.java ---
@@ -396,4 +387,12 @@ private Operation reverseOp(Operation
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/906#discussion_r82033272
--- Diff:
pxf/pxf-api/src/main/java/org/apache/hawq/pxf/api/FilterParser.java ---
@@ -396,4 +387,12 @@ private Operation reverseOp(Operation
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/906#discussion_r82033359
--- Diff:
pxf/pxf-api/src/main/java/org/apache/hawq/pxf/api/FilterParser.java ---
@@ -220,6 +193,24 @@ public Object parse(String filter) throws
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/906#discussion_r82032792
--- Diff:
pxf/pxf-hbase/src/main/java/org/apache/hawq/pxf/plugins/hbase/HBaseFilterBuilder.java
---
@@ -71,14 +74,19 @@ public HBaseFilterBuilder
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/906#discussion_r82032692
--- Diff:
pxf/pxf-hbase/src/main/java/org/apache/hawq/pxf/plugins/hbase/HBaseFilterBuilder.java
---
@@ -71,14 +74,19 @@ public HBaseFilterBuilder
Github user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/949
+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
Github user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/949
The issue was a bunch of missing function prototypes?
---
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 kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/951#discussion_r81869294
--- Diff: src/backend/access/external/libchurl.c ---
@@ -312,16 +313,29 @@ CHURL_HANDLE churl_init(const char* url,
CHURL_HEADERS headers
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/951#discussion_r81863084
--- Diff: src/backend/access/external/pxfutils.c ---
@@ -133,3 +137,60 @@ static void process_request(ClientContext*
client_context, char *uri
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/906#discussion_r81858427
--- Diff:
pxf/pxf-api/src/main/java/org/apache/hawq/pxf/api/FilterParser.java ---
@@ -81,6 +88,8 @@
* @throws Exception if building
Github user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/951
+1 aside from comments
---
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
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/951#discussion_r81802588
--- Diff: src/backend/access/external/libchurl.c ---
@@ -312,16 +313,29 @@ CHURL_HANDLE churl_init(const char* url,
CHURL_HEADERS headers
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/951#discussion_r81798661
--- Diff: src/backend/access/external/libchurl.c ---
@@ -312,16 +313,29 @@ CHURL_HANDLE churl_init(const char* url,
CHURL_HEADERS headers
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/951#discussion_r81801521
--- Diff: src/backend/access/external/pxfutils.c ---
@@ -133,3 +137,60 @@ static void process_request(ClientContext*
client_context, char *uri
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/951#discussion_r81800773
--- Diff: src/backend/access/external/pxfutils.c ---
@@ -133,3 +137,60 @@ static void process_request(ClientContext*
client_context, char *uri
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/935#discussion_r80351715
--- Diff:
pxf/pxf-hive/src/main/java/org/apache/hawq/pxf/plugins/hive/utilities/EnumHiveToHawqType.java
---
@@ -45,7 +45,7 @@
BinaryType
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/915#discussion_r79740844
--- Diff: src/backend/access/external/pxfheaders.c ---
@@ -66,8 +66,13 @@ void build_http_header(PxfInputData *input)
if (proj_info != NULL
Github user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/915
Aside from my confusion with my comment, +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
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/915#discussion_r79740167
--- Diff: src/backend/access/external/pxfheaders.c ---
@@ -66,8 +66,13 @@ void build_http_header(PxfInputData *input)
if (proj_info != NULL
GitHub user kavinderd opened a pull request:
https://github.com/apache/incubator-hawq/pull/906
HAWQ-964. Support for OR and NOT Logical Operators
Signed-off-by: Leslie Chang
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user kavinderd closed the pull request at:
https://github.com/apache/incubator-hawq/pull/905
---
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 user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/905#discussion_r79261601
--- Diff:
pxf/pxf-hbase/src/main/java/org/apache/hawq/pxf/plugins/hbase/HBaseFilterBuilder.java
---
@@ -71,14 +74,19 @@ public HBaseFilterBuilder
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/905#discussion_r79261531
--- Diff:
pxf/pxf-hive/src/test/java/org/apache/hawq/pxf/plugins/hive/HiveORCSearchArgumentExample.java
---
@@ -0,0 +1,87 @@
+package
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/905#discussion_r79261543
--- Diff:
pxf/pxf-hive/src/test/java/org/apache/hawq/pxf/plugins/hive/HiveFilterBuilderTest.java
---
@@ -20,11 +20,14
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/905#discussion_r79261518
--- Diff:
pxf/pxf-hive/src/test/java/org/apache/hawq/pxf/plugins/hive/HiveORCSearchArgumentExample.java
---
@@ -0,0 +1,87 @@
+package
GitHub user kavinderd opened a pull request:
https://github.com/apache/incubator-hawq/pull/905
HAWQ-964. Support for OR and NOT Logical Operators
Signed-off-by: Leslie Chang
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/902
+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
Github user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/895
Can you remove all references to `Vectorized` Hive classes since they are
not yet being used
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/895#discussion_r78207962
--- Diff:
pxf/pxf-hive/src/test/java/org/apache/hawq/pxf/plugins/hive/HiveDataFragmenterTest.java
---
@@ -52,55 +52,55
Github user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/894
LGTM
---
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
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/821#discussion_r77913853
--- Diff:
pxf/pxf-hive/src/main/java/org/apache/hawq/pxf/plugins/hive/HiveColumnarSerdeResolver.java
---
@@ -76,6 +77,8 @@ void parseUserData
Github user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/873
Overall LGTM, but I'd like to spend more time reviewing. This is a lot of
code
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/873#discussion_r77096574
--- Diff:
pxf/pxf-hive/src/main/java/org/apache/hawq/pxf/plugins/hive/utilities/EnumHiveToHawqType.java
---
@@ -110,4 +122,68 @@ public static
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/873#discussion_r77082531
--- Diff:
pxf/pxf-api/src/main/java/org/apache/hawq/pxf/api/utilities/EnumHawqType.java
---
@@ -43,37 +45,40 @@ public void serialize(EnumHawqType
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/873#discussion_r77082166
--- Diff:
pxf/pxf-api/src/main/java/org/apache/hawq/pxf/api/utilities/EnumHawqType.java
---
@@ -43,37 +45,40 @@ public void serialize(EnumHawqType
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/873#discussion_r77047856
--- Diff:
pxf/pxf-api/src/main/java/org/apache/hawq/pxf/api/utilities/ColumnDescriptor.java
---
@@ -26,10 +26,11 @@
*/
public class
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/837#discussion_r77047594
--- Diff:
pxf/pxf-hbase/src/main/java/org/apache/hawq/pxf/plugins/hbase/HBaseFilterBuilder.java
---
@@ -165,6 +165,14 @@ private Filter
Github user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/854
LGTM
---
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
Github user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/853
Nice work! LGTM
---
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
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/853#discussion_r76100388
--- Diff: src/backend/access/external/pxffilters.c ---
@@ -451,6 +452,51 @@ opexpr_to_pxffilter(OpExpr *expr, PxfFilterDesc
*filter)
return
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/837#discussion_r76098154
--- Diff: src/backend/optimizer/plan/createplan.c ---
@@ -1146,7 +1146,7 @@ static char** create_pxf_plan(char **segdb_file_map,
RelOptInfo *rel
Github user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/848
LGTM
---
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
Github user kavinderd commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/837#discussion_r74286229
--- Diff: src/backend/access/external/test/pxffilters_test.c ---
@@ -61,7 +62,7 @@ test__supported_filter_type(void **state)
/* go
Github user kavinderd closed the pull request at:
https://github.com/apache/incubator-hawq/pull/830
---
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 user kavinderd opened a pull request:
https://github.com/apache/incubator-hawq/pull/830
HAWQ-973. Check pi_varList before adding proj headers
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/kavinderd/incubator-hawq HAWQ
GitHub user kavinderd opened a pull request:
https://github.com/apache/incubator-hawq/pull/829
HAWQ-972.Setup multi handle after setting all conf
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/kavinderd/incubator-hawq HAWQ-972
Github user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/812
Already reverted
---
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
Github user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/817
Merged
---
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
Github user kavinderd closed the pull request at:
https://github.com/apache/incubator-hawq/pull/812
---
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 user kavinderd closed the pull request at:
https://github.com/apache/incubator-hawq/pull/817
---
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 user kavinderd commented on the issue:
https://github.com/apache/incubator-hawq/pull/808
Added `INT32_CHAR_SIZE` to `configure.in`
---
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
1 - 100 of 129 matches
Mail list logo