Github user sansanichfb commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1289#discussion_r140363620
--- Diff: src/backend/access/external/pxfheaders.c ---
@@ -65,14 +65,18 @@ void build_http_header(PxfInputData *input)
if (proj_info != NULL && proj_info->pi_isVarList)
{
- List* qualsAttributes = extractPxfAttributes(input->quals);
+ bool qualsAreSupported = true;
+ List* qualsAttributes = extractPxfAttributes(input->quals,
&qualsAreSupported);
--- End diff --
qualsAttributes could be null when WHERE clause is empty, and in this case,
we still want to use projection info. But if we don't support some operation in
WHERE clause - we just report qualsAreSupported flag and projection info will
not be used.
---
