[Impala-ASF-CR] IMPALA-2550 Introduce query-wide execution context.

2016-08-22 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-2550 Introduce query-wide execution context. .. Patch Set 8: (9 comments) http://gerrit.cloudera.org:8080/#/c/3817/7//COMMIT_MSG Commit Message: PS7,

[Impala-ASF-CR] IMPALA-2550 Introduce query-wide execution context.

2016-08-22 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has uploaded a new patch set (#8). Change subject: IMPALA-2550 Introduce query-wide execution context. .. IMPALA-2550 Introduce query-wide execution context. This patch is a header preview of a query wide

[Impala-ASF-CR] CDH-43354: Bump CDH components' versions.

2016-08-22 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: CDH-43354: Bump CDH components' versions. .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4084 To unsubscribe, visit

[Impala-ASF-CR] CDH-43354: Bump CDH components' versions.

2016-08-22 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: CDH-43354: Bump CDH components' versions. .. CDH-43354: Bump CDH components' versions. Change-Id: I217f2467628d0213afee1b8a531e48d035237212

[Impala-CR](cdh5-trunk) IMPALA-3820: Handle linkage errors while loading Java UDFs in Catalog

2016-08-22 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3820: Handle linkage errors while loading Java UDFs in Catalog .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3567 To

[Impala-ASF-CR] IMPALA-3893, IMPALA-3901: impala-shell prints incorrect coordinator address, overly verbose

2016-08-22 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-3893, IMPALA-3901: impala-shell prints incorrect coordinator address, overly verbose .. Patch Set 5: Code-Review+2 (1 comment) Thanks for the review

[Impala-ASF-CR] IMPALA-3893, IMPALA-3901: impala-shell prints incorrect coordinator address, overly verbose

2016-08-22 Thread Sailesh Mukil (Code Review)
Hello Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3994 to look at the new patch set (#6). Change subject: IMPALA-3893, IMPALA-3901: impala-shell prints incorrect coordinator address, overly verbose

[Impala-ASF-CR] IMPALA-(3895,3859): Don't log file data on parse errors

2016-08-22 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-(3895,3859): Don't log file data on parse errors .. Patch Set 4: This passes a core test run. Running exhaustive, and local (to test FS changes) now.

Re: Error linking impalad with -build_shared_libs option

2016-08-22 Thread Laszlo Gaal
link.txt is generated by cmake based on the contents of CMakeLists.txt and the project/environment settings. Most of the link-time dependencies seem to be set up in $(IMPALA_HOME)/be/CMakeLists.txt, based on settings calculated one level up in $(IMPALA_HOME)/CMakeLists.txt.

[Impala-ASF-CR] IMPALA-(3895,3859): Don't log file data on parse errors

2016-08-22 Thread Henry Robinson (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4020 to look at the new patch set (#4). Change subject: IMPALA-(3895,3859): Don't log file data on parse errors ..

[Impala-ASF-CR] IMPALA-3893, IMPALA-3901: impala-shell prints incorrect coordinator address, overly verbose

2016-08-22 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-3893, IMPALA-3901: impala-shell prints incorrect coordinator address, overly verbose .. Patch Set 5: Code-Review+2 (1 comment)

[Impala-ASF-CR] PREVIEW: IMPALA-3725 Support Kudu UPSERT in Impala

2016-08-22 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: PREVIEW: IMPALA-3725 Support Kudu UPSERT in Impala .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/4047/1//COMMIT_MSG Commit Message:

[Impala-ASF-CR] IMPALA-3201: reservation implementation for new buffer pool

2016-08-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3201: reservation implementation for new buffer pool .. Patch Set 6: Updated to use the UsedReservation/UnusedReservation terminology I discussed

[Impala-ASF-CR] IMPALA-3201: reservation implementation for new buffer pool

2016-08-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#6). Change subject: IMPALA-3201: reservation implementation for new buffer pool .. IMPALA-3201: reservation implementation for new buffer pool This patch implements the reservation

[Impala-ASF-CR] IMPALA-3986: Python hive-client may fail silently while dropping partitions

2016-08-22 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3986: Python hive-client may fail silently while dropping partitions .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-2550 Introduce query-wide execution context.

2016-08-22 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-2550 Introduce query-wide execution context. .. Patch Set 7: (9 comments) http://gerrit.cloudera.org:8080/#/c/3817/7//COMMIT_MSG Commit Message: PS7,

Re: Error linking impalad with -build_shared_libs option

2016-08-22 Thread Yonghyun Hwang
I observed the same linking error and digged this into further. The below is what I've found. *1. how to reproduce* # assuming that impala-config.sh is sourced $ cd ${IMPALA_HOME} *$ make -j8 impalad* ... [100%] Built target CodeGen Linking CXX executable ../../build/debug/service/impalad ...

[Impala-ASF-CR] IMPALA-3986: Python hive-client may fail silently while dropping partitions

2016-08-22 Thread Sailesh Mukil (Code Review)
Hello Matthew Jacobs, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4083 to look at the new patch set (#2). Change subject: IMPALA-3986: Python hive-client may fail silently while dropping partitions

[Impala-ASF-CR] IMPALA-3986: Python hive-client may fail silently while dropping partitions

2016-08-22 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-3986: Python hive-client may fail silently while dropping partitions .. Patch Set 2: (1 comment)

[Impala-ASF-CR] CDH-43354: Bump CDH components' versions.

2016-08-22 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: CDH-43354: Bump CDH components' versions. .. Patch Set 1: Code-Review+2 Thanks, Michael! -- To view, visit http://gerrit.cloudera.org:8080/4084 To unsubscribe,

[Impala-ASF-CR] IMPALA-3567 Part 2, IMPALA-3899: factor out PHJ builder

2016-08-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#9). Change subject: IMPALA-3567 Part 2, IMPALA-3899: factor out PHJ builder .. IMPALA-3567 Part 2, IMPALA-3899: factor out PHJ builder The main outcome of this patch is to split out

[Impala-ASF-CR] IMPALA-3567 Part 2, IMPALA-3899: factor out PHJ builder

2016-08-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3567 Part 2, IMPALA-3899: factor out PHJ builder .. Patch Set 8: Rebased this onto master MemLimitExceeded() changes. Had to make a few straightforward

[Impala-ASF-CR] IMPALA-3567 Part 2, IMPALA-3899: factor out PHJ builder

2016-08-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#8). Change subject: IMPALA-3567 Part 2, IMPALA-3899: factor out PHJ builder .. IMPALA-3567 Part 2, IMPALA-3899: factor out PHJ builder The main outcome of this patch is to split out

[Impala-ASF-CR] IMPALA-4006: impala-config.sh contains dangerous rm -rf statements

2016-08-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4006: impala-config.sh contains dangerous rm -rf statements .. Patch Set 2: (2 comments) Mostly looks good (aside from LOG_DIR that needs to be added

[Impala-ASF-CR] CDH-43354: Bump CDH components' versions.

2016-08-22 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: CDH-43354: Bump CDH components' versions. .. Patch Set 1: A core run passed: http://sandbox.jenkins.cloudera.com/job/impala-private-build-and-test/4002/ -- To

[Impala-ASF-CR] CDH-43354: Bump CDH components' versions.

2016-08-22 Thread Michael Ho (Code Review)
Michael Ho has uploaded a new change for review. http://gerrit.cloudera.org:8080/4084 Change subject: CDH-43354: Bump CDH components' versions. .. CDH-43354: Bump CDH components' versions. Change-Id:

[Impala-CR](cdh5-trunk) IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner

2016-08-22 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner .. Patch Set 11: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3774 To

[Impala-ASF-CR] IMPALA-3986: Python hive-client may fail silently while dropping partitions

2016-08-22 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3986: Python hive-client may fail silently while dropping partitions .. Patch Set 1: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-3986: Python hive-client may fail silently while dropping partitions

2016-08-22 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has uploaded a new change for review. http://gerrit.cloudera.org:8080/4083 Change subject: IMPALA-3986: Python hive-client may fail silently while dropping partitions .. IMPALA-3986: Python hive-client may fail

[Impala-CR](cdh5-trunk) IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner

2016-08-22 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner .. Patch Set 10: (3 comments)

[Impala-CR](cdh5-trunk) IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner

2016-08-22 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Matthew Jacobs, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3774 to look at the new patch set (#11). Change subject: IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner

[Impala-CR](cdh5-trunk) IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner

2016-08-22 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner .. Patch Set 10: (1 comment)

[Impala-CR](cdh5-trunk) IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner

2016-08-22 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner .. Patch Set 10: (7 comments)

[Impala-CR](cdh5-trunk) IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner

2016-08-22 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner .. Patch Set 10: (2 comments)

[Impala-CR](cdh5-trunk) IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner

2016-08-22 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Matthew Jacobs, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3774 to look at the new patch set (#10). Change subject: IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner

[Impala-ASF-CR] IMPALA-2540: Add regression test.

2016-08-22 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-2540: Add regression test. .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4071 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3671: Add query option to limit scratch space usage

2016-08-22 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3671: Add query option to limit scratch space usage .. Patch Set 4: (16 comments)

[Impala-CR](cdh5-trunk) IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner

2016-08-22 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner .. Patch Set 9: (1 comment)

[Impala-ASF-CR] IMPALA-2550 Introduce query-wide execution context.

2016-08-22 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has uploaded a new patch set (#7). Change subject: IMPALA-2550 Introduce query-wide execution context. .. IMPALA-2550 Introduce query-wide execution context. This patch is a header preview of a query wide

[Impala-ASF-CR] Infomation schema (preview for frontend)

2016-08-22 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#6). Change subject: Infomation_schema (preview for frontend) .. Infomation_schema (preview for frontend) Since this is just a preview and not the final patch, request reviewers to please

[Impala-ASF-CR] PREVIEW: IMPALA-2550 Introduce query-wide execution context.

2016-08-22 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: PREVIEW: IMPALA-2550 Introduce query-wide execution context. .. Patch Set 6: (1 comment)

[Impala-ASF-CR] IMPALA-1659: Netezza compatibility functions: metadata

2016-08-22 Thread Zoltan Ivanfi (Code Review)
Zoltan Ivanfi has posted comments on this change. Change subject: IMPALA-1659: Netezza compatibility functions: metadata .. Patch Set 2: > My recommendation would be to skip non-obvious > implementations (like CURRENT_SCHEMA)

[Impala-ASF-CR] IMPALA-1659: Netezza compatibility functions: metadata

2016-08-22 Thread Zoltan Ivanfi (Code Review)
Zoltan Ivanfi has uploaded a new patch set (#2). Change subject: IMPALA-1659: Netezza compatibility functions: metadata .. IMPALA-1659: Netezza compatibility functions: metadata Added the SQL functions current_catalog(),

[Impala-ASF-CR] IMPALA-4001: qgen: add proof of concept tests for Query() objects

2016-08-22 Thread Michael Brown (Code Review)
Michael Brown has uploaded a new change for review. http://gerrit.cloudera.org:8080/4081 Change subject: IMPALA-4001: qgen: add proof of concept tests for Query() objects .. IMPALA-4001: qgen: add proof of concept tests for

[Impala-ASF-CR] IMPALA-4006: impala-config.sh contains dangerous rm -rf statements

2016-08-22 Thread Zoltan Ivanfi (Code Review)
Zoltan Ivanfi has posted comments on this change. Change subject: IMPALA-4006: impala-config.sh contains dangerous rm -rf statements .. Patch Set 1: > (13 comments) > > This seems like a good practice, but I'm not clear on

[Impala-ASF-CR] IMPALA-4006: impala-config.sh contains dangerous rm -rf statements

2016-08-22 Thread Zoltan Ivanfi (Code Review)
Zoltan Ivanfi has posted comments on this change. Change subject: IMPALA-4006: impala-config.sh contains dangerous rm -rf statements .. Patch Set 1: (20 comments) > (12 comments) > > Thanks for cleaning things up. Not

[Impala-ASF-CR] IMPALA-4006: impala-config.sh contains dangerous rm -rf statements

2016-08-22 Thread Zoltan Ivanfi (Code Review)
Zoltan Ivanfi has uploaded a new patch set (#2). Change subject: IMPALA-4006: impala-config.sh contains dangerous rm -rf statements .. IMPALA-4006: impala-config.sh contains dangerous rm -rf statements Quoted variable

[Impala-ASF-CR] IMPALA-3912: test random rpc timeout is flaky.

2016-08-22 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3912: test_random_rpc_timeout is flaky. .. Patch Set 1: Couldn't find why the metric "num-fragments-in-flight" not back to 0. One suspect is a query might

[Impala-ASF-CR] IMPALA-3912: test random rpc timeout is flaky.

2016-08-22 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new change for review. http://gerrit.cloudera.org:8080/4080 Change subject: IMPALA-3912: test_random_rpc_timeout is flaky. .. IMPALA-3912: test_random_rpc_timeout is flaky. Change-Id:

[Impala-ASF-CR] IMPALA-3938: Prevent illegal implicit collection references being used.

2016-08-22 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3938: Prevent illegal implicit collection references being used. .. Patch Set 1: (3 comments)

[Impala-ASF-CR] IMPALA-3610: Account for memory used by filters in the coordinator

2016-08-22 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-3610: Account for memory used by filters in the coordinator .. Patch Set 1: Code-Review+1 (1 comment)

[Impala-CR](cdh5-trunk) IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner

2016-08-22 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner .. Patch Set 9: (3 comments)

[Impala-ASF-CR] IMPALA-3938: Prevent illegal implicit collection references being used.

2016-08-22 Thread Christopher Channing (Code Review)
Christopher Channing has uploaded a new change for review. http://gerrit.cloudera.org:8080/4079 Change subject: IMPALA-3938: Prevent illegal implicit collection references being used. .. IMPALA-3938: Prevent illegal implicit

[Impala-ASF-CR] IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend

2016-08-22 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend .. Patch Set 3: (2 comments)

[Impala-ASF-CR] IMPALA-4006 impala-config.sh contains dangerous rm -rf statements

2016-08-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4006 impala-config.sh contains dangerous rm -rf statements .. Patch Set 1: (13 comments) This seems like a good practice, but I'm not clear on how you

[Impala-ASF-CR] IMPALA-4006 impala-config.sh contains dangerous rm -rf statements

2016-08-22 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4006 impala-config.sh contains dangerous rm -rf statements .. Patch Set 1: (12 comments) Thanks for cleaning things up. Not sure if there is a easier way

[Impala-ASF-CR] IMPALA-3988: Only use first 96 bits of query id

2016-08-22 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-3988: Only use first 96 bits of query id .. Patch Set 9: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4065 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3988: Only use first 96 bits of query id

2016-08-22 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has submitted this change and it was merged. Change subject: IMPALA-3988: Only use first 96 bits of query id .. IMPALA-3988: Only use first 96 bits of query id This adds utility functions in uid-util.h to

[Impala-ASF-CR] IMPALA-3988: Only use first 96 bits of query id

2016-08-22 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-3988: Only use first 96 bits of query id .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4065 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend

2016-08-22 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend .. Patch Set 3: (22 comments) Partial review. I'm posting

[Impala-ASF-CR] IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend

2016-08-22 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend .. Patch Set 3: (9 comments)

[Impala-CR](cdh5-trunk) IMPALA-3820: Handle linkage errors while loading Java UDFs in Catalog

2016-08-22 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-3820: Handle linkage errors while loading Java UDFs in Catalog .. Patch Set 4: (1 comment)

[Impala-ASF-CR] IMPALA-4006 impala-config.sh contains dangerous rm -rf statements

2016-08-22 Thread Zoltan Ivanfi (Code Review)
Zoltan Ivanfi has uploaded a new change for review. http://gerrit.cloudera.org:8080/4078 Change subject: IMPALA-4006 impala-config.sh contains dangerous rm -rf statements .. IMPALA-4006 impala-config.sh contains dangerous rm

[Impala-CR](cdh5-trunk) IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2.

2016-08-22 Thread Youwei Wang (Code Review)
Hello Marcel Kornacker, Jim Apple, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3081 to look at the new patch set (#47). Change subject: IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2.

[Impala-CR](cdh5-trunk) IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2.

2016-08-22 Thread Youwei Wang (Code Review)
Youwei Wang has posted comments on this change. Change subject: IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2. .. Patch Set 45: (1 comment)

[Impala-CR](cdh5-trunk) IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2.

2016-08-22 Thread Youwei Wang (Code Review)
Hello Marcel Kornacker, Jim Apple, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3081 to look at the new patch set (#46). Change subject: IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2.

Re: Contributions to Cloudera Impala

2016-08-22 Thread Nishidha Panpaliya
Hi Jim, We tried the steps mentioned at https://cwiki.apache.org/confluence/display/IMPALA/How+to+switch+to +Apache-hosted+git. On Cloudera's Impala source tree -> $git remote add apache https://git-wip-us.apache.org/repos/asf/incubator-impala.git $git remote add asf-gerrit

[Impala-ASF-CR] IMPALA-1659: Netezza compatibility functions: metadata

2016-08-22 Thread Zoltan Ivanfi (Code Review)
Zoltan Ivanfi has posted comments on this change. Change subject: IMPALA-1659: Netezza compatibility functions: metadata .. Patch Set 1: And what happens if schemas are disabled? I think I found the relevant docs:

[Impala-ASF-CR] PREVIEW: IMPALA-2550 Introduce query-wide execution context.

2016-08-22 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: PREVIEW: IMPALA-2550 Introduce query-wide execution context. .. Patch Set 5: (33 comments) http://gerrit.cloudera.org:8080/#/c/3817/5//COMMIT_MSG Commit