[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-11-09 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8034 ) Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder .. Patch Set 16: (10 comments)

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-11-09 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, Tim Armstrong, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8034 to look at the new patch set (#16). Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

[Impala-ASF-CR] IMPALA-5142 EventSequence displays negative elapsed time.

2017-11-01 Thread Pranay Singh (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8215 to look at the new patch set (#6). Change subject: IMPALA-5142 EventSequence displays negative elapsed time. ..

[Impala-ASF-CR] IMPALA-5142 EventSequence displays negative elapsed time.

2017-11-01 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8215 ) Change subject: IMPALA-5142 EventSequence displays negative elapsed time. .. Patch Set 5: (3 comments)

[Impala-ASF-CR] IMPALA-5142 EventSequence displays negative elapsed time.

2017-11-01 Thread Pranay Singh (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8215 to look at the new patch set (#5). Change subject: IMPALA-5142 EventSequence displays negative elapsed time. ..

[Impala-ASF-CR] IMPALA-5142 EventSequence displays negative elapsed time.

2017-11-01 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8215 ) Change subject: IMPALA-5142 EventSequence displays negative elapsed time. .. Patch Set 4: (2 comments)

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-11-01 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, Tim Armstrong, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8034 to look at the new patch set (#15). Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-11-01 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8034 ) Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder .. Patch Set 14: (1 comment)

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-10-31 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, Tim Armstrong, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8034 to look at the new patch set (#14). Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-10-31 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, Tim Armstrong, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8034 to look at the new patch set (#13). Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-10-31 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8034 ) Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder .. Patch Set 12: (5 comments)

[Impala-ASF-CR] IMPALA-5142 EventSequence displays negative elapsed time.

2017-10-25 Thread Pranay Singh (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8215 to look at the new patch set (#4). Change subject: IMPALA-5142 EventSequence displays negative elapsed time. ..

[Impala-ASF-CR] IMPALA-5142 EventSequence displays negative elapsed time.

2017-10-25 Thread Pranay Singh (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8215 to look at the new patch set (#3). Change subject: IMPALA-5142 EventSequence displays negative elapsed time. ..

[Impala-ASF-CR] IMPALA-5142 EventSequence displays negative elapsed time.

2017-10-25 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8215 ) Change subject: IMPALA-5142 EventSequence displays negative elapsed time. .. Patch Set 2: (2 comments)

[Impala-ASF-CR] IMPALA-5142 EventSequence displays negative elapsed time.

2017-10-24 Thread Pranay Singh (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8215 to look at the new patch set (#2). Change subject: IMPALA-5142 EventSequence displays negative elapsed time. ..

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-10-23 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, Tim Armstrong, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8034 to look at the new patch set (#12). Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-10-20 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, Tim Armstrong, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8034 to look at the new patch set (#11). Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-10-20 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8034 ) Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder .. Patch Set 10: (5 comments)

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-10-17 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, Tim Armstrong, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8034 to look at the new patch set (#10). Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-10-17 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8034 ) Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder .. Patch Set 9: (5 comments)

[Impala-ASF-CR] IMPALA-5142 EventSequence displays negative elapsed time.

2017-10-17 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8215 ) Change subject: IMPALA-5142 EventSequence displays negative elapsed time. .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-5142 EventSequence displays negative elapsed time.

2017-10-16 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8215 ) Change subject: IMPALA-5142 EventSequence displays negative elapsed time. .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-10-13 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8034 to look at the new patch set (#9). Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-10-13 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8034 ) Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder .. Patch Set 8: (8 comments)

[Impala-ASF-CR] Addressing comments.

2017-10-12 Thread Pranay Singh (Code Review)
Pranay Singh has abandoned this change. ( http://gerrit.cloudera.org:8080/8263 ) Change subject: Addressing comments. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/8263 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-10-12 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8034 to look at the new patch set (#8). Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

[Impala-ASF-CR] Addressing comments.

2017-10-12 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8263 Change subject: Addressing comments. .. Addressing comments. Change-Id: Iac2047ad6a548f4c9b1edb51189fe0970507d6dd --- M

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-10-12 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8034 to look at the new patch set (#7). Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-10-05 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8056 ) Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. Patch Set 8: (1 comment)

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-10-04 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8034 ) Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder .. Patch Set 6: (5 comments)

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-10-04 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8056 ) Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. Patch Set 8: (1 comment)

[Impala-ASF-CR] IMPALA-5142 EventSequence displays negative elapsed time.

2017-10-04 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8215 Change subject: IMPALA-5142 EventSequence displays negative elapsed time. .. IMPALA-5142 EventSequence displays negative elapsed time.

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-10-03 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8056 to look at the new patch set (#8). Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-10-03 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8056 to look at the new patch set (#7). Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-10-03 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8056 ) Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. Patch Set 6: (4 comments)

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-10-03 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8034 to look at the new patch set (#6). Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-10-03 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8034 to look at the new patch set (#5). Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-10-03 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8034 ) Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder .. Patch Set 3: (6 comments)

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-10-03 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8034 to look at the new patch set (#4). Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-10-02 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8056 to look at the new patch set (#6). Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-10-02 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8056 ) Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. Patch Set 5: (2 comments)

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-09-28 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8034 to look at the new patch set (#3). Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-26 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8056 to look at the new patch set (#5). Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-26 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/8056 ) Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. Patch Set 4: (5 comments)

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-22 Thread Pranay Singh (Code Review)
Hello Joe McDonnell, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8056 to look at the new patch set (#4). Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

[Impala-ASF-CR] IMPALA-1649 Pass precision to Print TCounterType::DOUBLE VALUE

2017-09-21 Thread Pranay Singh (Code Review)
Pranay Singh has abandoned this change. ( http://gerrit.cloudera.org:8080/7725 ) Change subject: IMPALA-1649 Pass precision to Print TCounterType::DOUBLE_VALUE .. Abandoned After talking to Tim and Joe, it appears that

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-17 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#3). Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet test_scanners_fuzz.py

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-17 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#5). Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet test_scanners_fuzz.py

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-17 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#4). Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet test_scanners_fuzz.py

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-15 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#3). Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet test_scanners_fuzz.py

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-15 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#3). Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet test_scanners_fuzz.py

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-15 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#3). Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet test_scanners_fuzz.py

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-15 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. Patch Set 2: (2 comments)

[Impala-ASF-CR] IMPALA-1649 Pass precision to Print TCounterType::DOUBLE VALUE

2017-09-14 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. Change subject: IMPALA-1649 Pass precision to Print TCounterType::DOUBLE_VALUE .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-13 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#2). Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet test_scanners_fuzz.py

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-13 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#2). Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet test_scanners_fuzz.py

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-13 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. Patch Set 1: (3 comments)

[Impala-ASF-CR] IMPALA-1649 Pass precision to Print TCounterType::DOUBLE VALUE

2017-09-13 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. Change subject: IMPALA-1649 Pass precision to Print TCounterType::DOUBLE_VALUE .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-1649 Pass precision to Print TCounterType::DOUBLE VALUE

2017-09-13 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#2). Change subject: IMPALA-1649 Pass precision to Print TCounterType::DOUBLE_VALUE .. IMPALA-1649 Pass precision to Print TCounterType::DOUBLE_VALUE Fix: Current code uses

[Impala-ASF-CR] IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet

2017-09-13 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new change for review. http://gerrit.cloudera.org:8080/8056 Change subject: IMPALA-5525 Extend TestScannersFuzzing to test uncompressed parquet .. IMPALA-5525 Extend TestScannersFuzzing to test

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-09-12 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#2). Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder .. IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder Currently DictDecoder class and

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-09-12 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/8034/1/be/src/exec/exec-node.h

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-09-12 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#2). Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder .. IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder Currently DictDecoder class and

[Impala-ASF-CR] IMPALA-5108: Followup change to do the cleanup.

2017-08-28 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#3). Change subject: IMPALA-5108: Followup change to do the cleanup. .. IMPALA-5108: Followup change to do the cleanup. This change involves giving names to constant values to improve

[Impala-ASF-CR] IMPALA-5108: Followup change to do the cleanup.

2017-08-28 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#3). Change subject: IMPALA-5108: Followup change to do the cleanup. .. IMPALA-5108: Followup change to do the cleanup. This change involves giving names to constant values to improve

[Impala-ASF-CR] IMPALA-5108: Followup change to do the cleanup.

2017-08-28 Thread Pranay Singh (Code Review)
Pranay Singh has restored this change. Change subject: IMPALA-5108: Followup change to do the cleanup. .. Restored -- To view, visit http://gerrit.cloudera.org:8080/7801 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5108: Followup change to do the cleanup.

2017-08-28 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#2). Change subject: IMPALA-5108: Followup change to do the cleanup. .. IMPALA-5108: Followup change to do the cleanup. This change involves giving names to constant values to improve

[Impala-ASF-CR] IMPALA-5108: Followup change to do the cleanup.

2017-08-28 Thread Pranay Singh (Code Review)
Pranay Singh has abandoned this change. Change subject: IMPALA-5108: Followup change to do the cleanup. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/7801 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5108: Followup change to do the cleanup.

2017-08-28 Thread Pranay Singh (Code Review)
Pranay Singh has abandoned this change. Change subject: IMPALA-5108: Followup change to do the cleanup. .. Abandoned Need to use the old commit id -- To view, visit http://gerrit.cloudera.org:8080/7807 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5108: Followup change to do the cleanup.

2017-08-24 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new change for review. http://gerrit.cloudera.org:8080/7807 Change subject: IMPALA-5108: Followup change to do the cleanup. .. IMPALA-5108: Followup change to do the cleanup. This change involves

[Impala-ASF-CR] IMPALA-5108: Followup change to do the cleanup.

2017-08-23 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new change for review. http://gerrit.cloudera.org:8080/7801 Change subject: IMPALA-5108: Followup change to do the cleanup. .. IMPALA-5108: Followup change to do the cleanup. This change involves

[Impala-ASF-CR] IMPALA-5108: idle session timeout kicks in later than expected

2017-08-23 Thread Pranay Singh (Code Review)
Pranay Singh has abandoned this change. Change subject: IMPALA-5108: idle_session_timeout kicks in later than expected .. Abandoned This change was merged so I need to abandon the change I'll address the review comments as a

[Impala-ASF-CR] IMPALA-5108: idle session timeout kicks in later than expected

2017-08-23 Thread Pranay Singh (Code Review)
Pranay Singh has abandoned this change. Change subject: IMPALA-5108: idle_session_timeout kicks in later than expected .. Abandoned The previous commit was merged so I have to abandon this change -- To view, visit

[Impala-ASF-CR] IMPALA-5108: idle session timeout kicks in later than expected

2017-08-23 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. Change subject: IMPALA-5108: idle_session_timeout kicks in later than expected .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7791/1//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-5108: idle session timeout kicks in later than expected

2017-08-23 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new change for review. http://gerrit.cloudera.org:8080/7791 Change subject: IMPALA-5108: idle_session_timeout kicks in later than expected .. IMPALA-5108: idle_session_timeout kicks in later than

[Impala-ASF-CR] IMPALA-5108: idle session timeout kicks in later than expected

2017-08-23 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. Change subject: IMPALA-5108: idle_session_timeout kicks in later than expected .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/7729/1//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-5108: idle session timeout kicks in later than expected

2017-08-18 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#3). Change subject: IMPALA-5108: idle_session_timeout kicks in later than expected .. IMPALA-5108: idle_session_timeout kicks in later than expected Fix: The issue was caused because

[Impala-ASF-CR] IMPALA-5108: idle session timeout kicks in later than expected

2017-08-18 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new change for review. http://gerrit.cloudera.org:8080/7729 Change subject: IMPALA-5108: idle_session_timeout kicks in later than expected .. IMPALA-5108: idle_session_timeout kicks in later than

[Impala-ASF-CR] IMPALA-5108: idle session timeout kicks in later than expected

2017-08-18 Thread Pranay Singh (Code Review)
Hello Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7709 to look at the new patch set (#3). Change subject: IMPALA-5108: idle_session_timeout kicks in later than expected

[Impala-ASF-CR] IMPALA-1649 Pass precision to Print TCounterType::DOUBLE VALUE

2017-08-18 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new change for review. http://gerrit.cloudera.org:8080/7725 Change subject: IMPALA-1649 Pass precision to Print TCounterType::DOUBLE_VALUE .. IMPALA-1649 Pass precision to Print

[Impala-ASF-CR] IMPALA-5108:idle session timeout kicks in later than expected

2017-08-17 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. Change subject: IMPALA-5108:idle_session_timeout kicks in later than expected .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7712/1//COMMIT_MSG Commit Message:

[Impala-ASF-CR] IMPALA-5108:idle session timeout kicks in later than expected

2017-08-17 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#2). Change subject: IMPALA-5108:idle_session_timeout kicks in later than expected .. IMPALA-5108:idle_session_timeout kicks in later than expected Fix: The issue was caused because

[Impala-ASF-CR] IMPALA-5108:idle session timeout kicks in later than expected

2017-08-17 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#2). Change subject: IMPALA-5108:idle_session_timeout kicks in later than expected .. IMPALA-5108:idle_session_timeout kicks in later than expected Fix: The issue was caused because

[Impala-ASF-CR] IMPALA-5108:idle session timeout kicks in later than expected

2017-08-17 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new change for review. http://gerrit.cloudera.org:8080/7712 Change subject: IMPALA-5108:idle_session_timeout kicks in later than expected .. IMPALA-5108:idle_session_timeout kicks in later than

[Impala-ASF-CR] IMPALA-5108:idle session timeout kicks in later than expected

2017-08-17 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. Change subject: IMPALA-5108:idle_session_timeout kicks in later than expected .. Patch Set 1: (5 comments)

[Impala-ASF-CR] IMPALA-5108:idle session timeout kicks in later than expected

2017-08-17 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new change for review. http://gerrit.cloudera.org:8080/7709 Change subject: IMPALA-5108:idle_session_timeout kicks in later than expected .. IMPALA-5108:idle_session_timeout kicks in later than

[Impala-ASF-CR] IMPALA-4861: READ WRITE warning on CREATE TABLE LIKE PARQUET.

2017-08-14 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new change for review. http://gerrit.cloudera.org:8080/7671 Change subject: IMPALA-4861: READ_WRITE warning on CREATE TABLE LIKE PARQUET. .. IMPALA-4861: READ_WRITE warning on CREATE TABLE LIKE

[Impala-ASF-CR] IMPALA-1478: Improve error message when subquery is used in the ON clause

2017-08-10 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#4). Change subject: IMPALA-1478: Improve error message when subquery is used in the ON clause .. IMPALA-1478: Improve error message when subquery is used in the

[Impala-ASF-CR] IMPALA-1478: Improve error message when subquery is used in the ON clause

2017-08-08 Thread Pranay Singh (Code Review)
Hello Bharath Vissapragada, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7588 to look at the new patch set (#4). Change subject: IMPALA-1478: Improve error message when subquery is used in the ON clause

[Impala-ASF-CR] IMPALA-1478: Improve error message when subquery is used in the ON clause

2017-08-08 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. Change subject: IMPALA-1478: Improve error message when subquery is used in the ON clause .. Patch Set 3: (4 comments) Addressed the comments

[Impala-ASF-CR] IMPALA-1478: Improve error message when subquery is used in the ON clause

2017-08-07 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#3). Change subject: IMPALA-1478: Improve error message when subquery is used in the ON clause .. IMPALA-1478: Improve error message when subquery is used in the ON clause Fix: Print

[Impala-ASF-CR] IMPALA-1478: Improve error message when subquery is used in the ON clause

2017-08-07 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new patch set (#2). Change subject: IMPALA-1478: Improve error message when subquery is used in the ON clause .. IMPALA-1478: Improve error message when subquery is used in the ON clause Fix: Print

[Impala-ASF-CR] IMPALA-1478: Improve error message when subquery is used in the ON clause

2017-08-07 Thread Pranay Singh (Code Review)
Pranay Singh has uploaded a new change for review. http://gerrit.cloudera.org:8080/7603 Change subject: IMPALA-1478: Improve error message when subquery is used in the ON clause .. IMPALA-1478: Improve error message when