[Impala-ASF-CR] IMPALA-5842: Write page index in Parquet files

2018-04-16 Thread Zoltan Borok-Nagy (Code Review)
Zoltan Borok-Nagy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9693 ) Change subject: IMPALA-5842: Write page index in Parquet files .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/96

[Impala-ASF-CR] IMPALA-5842: Write page index in Parquet files

2018-04-16 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/9693 ) Change subject: IMPALA-5842: Write page index in Parquet files .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/9693/10/

[Impala-ASF-CR] IMPALA-6723: [DOCS] Hints for CTAS

2018-04-16 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/9993 ) Change subject: IMPALA-6723: [DOCS] Hints for CTAS .. Patch Set 6: I don't see a new Patch Set. Did you forget to push it? -- To view,

[Impala-ASF-CR] IMPALA-6917: Limit impalad mem-limit to 12GB.

2018-04-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10051 ) Change subject: IMPALA-6917: Limit impalad mem-limit to 12GB. .. Patch Set 1: Build started: https://jenkins.impala.io/job/ger

[Impala-ASF-CR] IMPALA-6723: [DOCS] Hints for CTAS

2018-04-16 Thread Alex Rodoni (Code Review)
Hello Thomas Tauber-Marshall, Lars Volker, Csaba Ringhofer, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9993 to look at the new patch set (#7). Change subject: IMPALA-6723: [DOCS] Hints for CTAS ...

[Impala-ASF-CR] IMPALA-6793: Fix empty metadata after statestore restarts

2018-04-16 Thread Tianyi Wang (Code Review)
Tianyi Wang has uploaded a new patch set (#9). ( http://gerrit.cloudera.org:8080/9921 ) Change subject: IMPALA-6793: Fix empty metadata after statestore restarts .. IMPALA-6793: Fix empty metadata after statestore restarts IMPA

[Impala-ASF-CR] IMPALA-6464: [DOCS] COMPUTE STATS supports a list of columns

2018-04-16 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/10070 ) Change subject: IMPALA-6464: [DOCS] COMPUTE STATS supports a list of columns .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8

[Impala-ASF-CR] IMPALA-6732: [DOCS] 2.12 Release Notes

2018-04-16 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10071 ) Change subject: IMPALA-6732: [DOCS] 2.12 Release Notes .. Patch Set 1: Could you review and approve this so that I can start 3.0 release

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Hello Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#5). Change subject: IMPALA-6847: work around high memory estimates for AC ..

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Hello Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#6). Change subject: IMPALA-6847: work around high memory estimates for AC ..

[Impala-ASF-CR] Allow bootstrap system.sh to work on existing impala/ repo

2018-04-16 Thread Sailesh Mukil (Code Review)
Hello Jim Apple, Philip Zeyliger, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10068 to look at the new patch set (#2). Change subject: Allow bootstrap_system.sh to work on existing impala/ repo

[Impala-ASF-CR] Allow bootstrap system.sh to work on existing impala/ repo

2018-04-16 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10068 ) Change subject: Allow bootstrap_system.sh to work on existing impala/ repo .. Patch Set 2: (1 comment) > Patch Set 1: > > (1 comment)

[Impala-ASF-CR] IMPALA-6464: [DOCS] COMPUTE STATS supports a list of columns

2018-04-16 Thread Alex Rodoni (Code Review)
Hello Alex Behm, Impala Public Jenkins, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10070 to look at the new patch set (#4). Change subject: IMPALA-6464: [DOCS] COMPUTE STATS supports a list of columns .

[Impala-ASF-CR] IMPALA-6464: [DOCS] COMPUTE STATS supports a list of columns

2018-04-16 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10070 ) Change subject: IMPALA-6464: [DOCS] COMPUTE STATS supports a list of columns .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:808

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/100

[Impala-ASF-CR] IMPALA-6522: [DOCS] Document Decimal V2

2018-04-16 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10066 ) Change subject: IMPALA-6522: [DOCS] Document Decimal V2 .. Patch Set 5: Let me know if you want me to distribute the latest draft in HTM

[Impala-ASF-CR] Allow bootstrap system.sh to work on existing impala/ repo

2018-04-16 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/10068 ) Change subject: Allow bootstrap_system.sh to work on existing impala/ repo .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:808

[Impala-ASF-CR] Allow bootstrap system.sh to work on existing impala/ repo

2018-04-16 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/10068 ) Change subject: Allow bootstrap_system.sh to work on existing impala/ repo .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:

[Impala-ASF-CR] IMPALA-6723: [DOCS] Hints for CTAS

2018-04-16 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/9993 ) Change subject: IMPALA-6723: [DOCS] Hints for CTAS .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/9993/7/docs/topics/im

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/

[Impala-ASF-CR] IMPALA-6723: [DOCS] Hints for CTAS

2018-04-16 Thread Alex Rodoni (Code Review)
Hello Thomas Tauber-Marshall, Lars Volker, Csaba Ringhofer, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9993 to look at the new patch set (#8). Change subject: IMPALA-6723: [DOCS] Hints for CTAS ...

[Impala-ASF-CR] IMPALA-6723: [DOCS] Hints for CTAS

2018-04-16 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/9993 ) Change subject: IMPALA-6723: [DOCS] Hints for CTAS .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/9993/7/docs/topics/im

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/

[Impala-ASF-CR] IMPALA-5893: Remove old kinit code for Impala 3

2018-04-16 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/9941 ) Change subject: IMPALA-5893: Remove old kinit code for Impala 3 .. Patch Set 3: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Hello Philip Zeyliger, Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#7). Change subject: IMPALA-6847: work around high memory estimates for AC .

[Impala-ASF-CR] IMPALA-6723: [DOCS] Hints for CTAS

2018-04-16 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/9993 ) Change subject: IMPALA-6723: [DOCS] Hints for CTAS .. Patch Set 8: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/9993

[Impala-ASF-CR] IMPALA-6723: [DOCS] Hints for CTAS

2018-04-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9993 ) Change subject: IMPALA-6723: [DOCS] Hints for CTAS .. Patch Set 8: Build started: https://jenkins.impala.io/job/gerrit-docs-sub

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/c/10

[Impala-ASF-CR] IMPALA-6723: [DOCS] Hints for CTAS

2018-04-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9993 ) Change subject: IMPALA-6723: [DOCS] Hints for CTAS .. Patch Set 8: Verified+1 -- To view, visit http://gerrit.cloudera.org:808

[Impala-ASF-CR] Allow bootstrap system.sh to work on existing impala/ repo

2018-04-16 Thread Sailesh Mukil (Code Review)
Hello Fredy Wijaya, Jim Apple, Philip Zeyliger, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10068 to look at the new patch set (#3). Change subject: Allow bootstrap_system.sh to work on existing impala/ repo ..

[Impala-ASF-CR] Allow bootstrap system.sh to work on existing impala/ repo

2018-04-16 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10068 ) Change subject: Allow bootstrap_system.sh to work on existing impala/ repo .. Patch Set 2: (6 comments) Thanks for the review. http:

[Impala-ASF-CR] IMPALA-6464: [DOCS] COMPUTE STATS supports a list of columns

2018-04-16 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/10070 ) Change subject: IMPALA-6464: [DOCS] COMPUTE STATS supports a list of columns .. Patch Set 4: Code-Review+2 -- To view, visit http://g

[Impala-ASF-CR] IMPALA-6732: [DOCS] 2.12 Release Notes

2018-04-16 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10071 ) Change subject: IMPALA-6732: [DOCS] 2.12 Release Notes .. Patch Set 1: (1 comment) LGTM. http://gerrit.cloudera.org:8080/#/c/10071/1

[Impala-ASF-CR] IMPALA-6732: [DOCS] 2.12 Release Notes

2018-04-16 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10071 ) Change subject: IMPALA-6732: [DOCS] 2.12 Release Notes .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/10071/1/docs/imp

[Impala-ASF-CR] IMPALA-5690: Part 2: Upgrade thrift to 0.9.3-p3

2018-04-16 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/9300 ) Change subject: IMPALA-5690: Part 2: Upgrade thrift to 0.9.3-p3 .. Patch Set 11: (5 comments) http://gerrit.cloudera.org:8080/#/c/9300/11

[Impala-ASF-CR] IMPALA-6732: [DOCS] 2.12 Release Notes

2018-04-16 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10071 ) Change subject: IMPALA-6732: [DOCS] 2.12 Release Notes .. Patch Set 1: Code-Review+2 LGTM -- To view, visit http://gerrit.cloudera.o

[Impala-ASF-CR] IMPALA-5893: Remove old kinit code for Impala 3

2018-04-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9941 ) Change subject: IMPALA-5893: Remove old kinit code for Impala 3 .. Patch Set 4: Build started: https://jenkins.impala.io/job/ge

[Impala-ASF-CR] IMPALA-6732: [DOCS] 2.12 Release Notes

2018-04-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10071 ) Change subject: IMPALA-6732: [DOCS] 2.12 Release Notes .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-doc

[Impala-ASF-CR] IMPALA-5893: Remove old kinit code for Impala 3

2018-04-16 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/9941 ) Change subject: IMPALA-5893: Remove old kinit code for Impala 3 .. Patch Set 4: Code-Review+2 (1 comment) Rebase, carry +2. http://ge

[Impala-ASF-CR] IMPALA-6651: [DOCS] Fine grained privileges

2018-04-16 Thread Alex Rodoni (Code Review)
Alex Rodoni has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10079 Change subject: IMPALA-6651: [DOCS] Fine grained privileges .. IMPALA-6651: [DOCS] Fine grained privileges Change-Id: I7b018bf847537ed461

[Impala-ASF-CR] IMPALA-6464: [DOCS] COMPUTE STATS supports a list of columns

2018-04-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10070 ) Change subject: IMPALA-6464: [DOCS] COMPUTE STATS supports a list of columns .. Patch Set 4: Build started: https://jenkins.im

[Impala-ASF-CR] IMPALA-6732: [DOCS] 2.12 Release Notes

2018-04-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10071 ) Change subject: IMPALA-6732: [DOCS] 2.12 Release Notes .. IMPALA-6732: [DOCS] 2.12 Release Notes Change-Id: Ia0eee2a5b

[Impala-ASF-CR] IMPALA-6464: [DOCS] COMPUTE STATS supports a list of columns

2018-04-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10070 ) Change subject: IMPALA-6464: [DOCS] COMPUTE STATS supports a list of columns .. Patch Set 4: Verified+1 -- To view, visit htt

[Impala-ASF-CR] IMPALA-6464: [DOCS] COMPUTE STATS supports a list of columns

2018-04-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10070 ) Change subject: IMPALA-6464: [DOCS] COMPUTE STATS supports a list of columns .. IMPALA-6464: [DOCS] COMPUTE STATS suppo

[Impala-ASF-CR] IMPALA-6748: [DOCS] Separators when casting STRING to TIMESTAMP

2018-04-16 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10052 ) Change subject: IMPALA-6748: [DOCS] Separators when casting STRING to TIMESTAMP .. Patch Set 2: Could you please review the latest patch

[Impala-ASF-CR] IMPALA-6790: Upgrade sqlparse to 0.1.19

2018-04-16 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10044 ) Change subject: IMPALA-6790: Upgrade sqlparse to 0.1.19 .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:80

[Impala-ASF-CR] IMPALA-6748: [DOCS] Separators when casting STRING to TIMESTAMP

2018-04-16 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10052 ) Change subject: IMPALA-6748: [DOCS] Separators when casting STRING to TIMESTAMP .. Patch Set 1: (1 comment) http://gerrit.cloudera.or

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Hello Philip Zeyliger, Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#8). Change subject: IMPALA-6847: work around high memory estimates for AC .

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/

[Impala-ASF-CR] IMPALA-6860: [DOCS] Upgrade considerations for Impala 3.0

2018-04-16 Thread Alex Rodoni (Code Review)
Alex Rodoni has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10080 Change subject: IMPALA-6860: [DOCS] Upgrade considerations for Impala 3.0 .. IMPALA-6860: [DOCS] Upgrade considerations for Impala 3.0 Ch

[Impala-ASF-CR] IMPALA-6748: [DOCS] Separators when casting STRING to TIMESTAMP

2018-04-16 Thread Alex Rodoni (Code Review)
Hello Vincent Tran, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10052 to look at the new patch set (#3). Change subject: IMPALA-6748: [DOCS] Separators when casting STRING to TIMESTAMP ...

[Impala-ASF-CR] IMPALA-6748: [DOCS] Separators when casting STRING to TIMESTAMP

2018-04-16 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10052 ) Change subject: IMPALA-6748: [DOCS] Separators when casting STRING to TIMESTAMP .. Patch Set 1: > (1 comment) Done. -- To view, visit

[Impala-ASF-CR] IMPALA-5310: [DOCS] Document TABLESAMPLE clause for COMPUTE STATS

2018-04-16 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/8975 ) Change subject: IMPALA-5310: [DOCS] Document TABLESAMPLE clause for COMPUTE STATS .. Patch Set 3: Alex B: Could you review this patch fo

[Impala-ASF-CR] IMPALA-6860: [DOCS] Upgrade considerations for Impala 3.0

2018-04-16 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/10080 ) Change subject: IMPALA-6860: [DOCS] Upgrade considerations for Impala 3.0 .. Patch Set 1: (12 comments) http://gerrit.cloudera.org:80

[Impala-ASF-CR] IMPALA-6679,IMPALA-6678: reduce scan reservation

2018-04-16 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/9757 ) Change subject: IMPALA-6679,IMPALA-6678: reduce scan reservation .. Patch Set 6: (20 comments) > This isn't ready for review yet, but

[Impala-ASF-CR] IMPALA-6860: [DOCS] Upgrade considerations for Impala 3.0

2018-04-16 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10080 ) Change subject: IMPALA-6860: [DOCS] Upgrade considerations for Impala 3.0 .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 8: (6 comments) As we discussed offline, I think we'd

[Impala-ASF-CR] Use redirection-aware downloader for pip download.py.

2018-04-16 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10083 Change subject: Use redirection-aware downloader for pip_download.py. .. Use redirection-aware downloader for pip_download.py. As par

[Impala-ASF-CR] Use redirection-aware downloader for pip download.py.

2018-04-16 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/10083 ) Change subject: Use redirection-aware downloader for pip_download.py. .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cl

[Impala-ASF-CR] IMPALA-6860: [DOCS] Upgrade considerations for Impala 3.0

2018-04-16 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10080 ) Change subject: IMPALA-6860: [DOCS] Upgrade considerations for Impala 3.0 .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/

[Impala-ASF-CR] IMPALA-6860: [DOCS] Upgrade considerations for Impala 3.0

2018-04-16 Thread Alex Rodoni (Code Review)
Hello Philip Zeyliger, Sailesh Mukil, Impala Public Jenkins, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10080 to look at the new patch set (#2). Change subject: IMPALA-6860: [DOCS] Upgrade considerations for Impala 3.0 ...

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Hello Philip Zeyliger, Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#9). Change subject: IMPALA-6847: work around high memory estimates for AC .

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Hello Philip Zeyliger, Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#10). Change subject: IMPALA-6847: work around high memory estimates for AC

[Impala-ASF-CR] Use redirection-aware downloader for pip download.py.

2018-04-16 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/10083 ) Change subject: Use redirection-aware downloader for pip_download.py. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/1

[Impala-ASF-CR] IMPALA-6723: [DOCS] Hints for CTAS

2018-04-16 Thread Alex Rodoni (Code Review)
Hello Thomas Tauber-Marshall, Lars Volker, Csaba Ringhofer, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9993 to look at the new patch set (#9). Change subject: IMPALA-6723: [DOCS] Hints for CTAS ...

[Impala-ASF-CR] IMPALA-6723: [DOCS] Hints for CTAS

2018-04-16 Thread Alex Rodoni (Code Review)
Hello Thomas Tauber-Marshall, Lars Volker, Csaba Ringhofer, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9993 to look at the new patch set (#10). Change subject: IMPALA-6723: [DOCS] Hints for CTAS ..

[Impala-ASF-CR] IMPALA-6723: [DOCS] Hints for CTAS

2018-04-16 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/9993 ) Change subject: IMPALA-6723: [DOCS] Hints for CTAS .. Patch Set 10: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/9993

[Impala-ASF-CR] IMPALA-6723: [DOCS] Hints for CTAS

2018-04-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9993 ) Change subject: IMPALA-6723: [DOCS] Hints for CTAS .. Patch Set 10: Build started: https://jenkins.impala.io/job/gerrit-docs-su

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Hello Philip Zeyliger, Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#11). Change subject: IMPALA-6847: work around high memory estimates for AC

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 11: (6 comments) Here's the new simplified version

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Tim Armstrong (Code Review)
Hello Philip Zeyliger, Bikramjeet Vig, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10058 to look at the new patch set (#12). Change subject: IMPALA-6847: work around high memory estimates for AC

[Impala-ASF-CR] IMPALA-6723: [DOCS] Hints for CTAS

2018-04-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9993 ) Change subject: IMPALA-6723: [DOCS] Hints for CTAS .. Patch Set 10: Verified+1 -- To view, visit http://gerrit.cloudera.org:80

[Impala-ASF-CR] IMPALA-6723: [DOCS] Hints for CTAS

2018-04-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9993 ) Change subject: IMPALA-6723: [DOCS] Hints for CTAS .. IMPALA-6723: [DOCS] Hints for CTAS Change-Id: I91d9f4f039a603382f

[Impala-ASF-CR] IMPALA-6917: Limit impalad mem-limit to 12GB.

2018-04-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10051 ) Change subject: IMPALA-6917: Limit impalad mem-limit to 12GB. .. Patch Set 1: Verified-1 Build failed: https://jenkins.impala.

[Impala-ASF-CR] [DOCS] Release version updated to 3.0

2018-04-16 Thread Alex Rodoni (Code Review)
Alex Rodoni has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10084 Change subject: [DOCS] Release version updated to 3.0 .. [DOCS] Release version updated to 3.0 Change-Id: I511235e543018b322c12132d3470ce

[Impala-ASF-CR] IMPALA-686: Make pip download.py honor redirects.

2018-04-16 Thread Philip Zeyliger (Code Review)
Hello Lars Volker, Fredy Wijaya, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10083 to look at the new patch set (#2). Change subject: IMPALA-686: Make pip_download.py honor redirects. ..

[Impala-ASF-CR] IMPALA-686: Make pip download.py honor redirects.

2018-04-16 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/10083 ) Change subject: IMPALA-686: Make pip_download.py honor redirects. .. Patch Set 2: I started https://jenkins.impala.io/job/gerrit-ver

[Impala-ASF-CR] IMPALA-686: Make pip download.py honor redirects.

2018-04-16 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/10083 ) Change subject: IMPALA-686: Make pip_download.py honor redirects. .. Patch Set 2: Code-Review+2 Thanks for fixing this! -- To view, vi

[Impala-ASF-CR] IMPALA-5893: Remove old kinit code for Impala 3

2018-04-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9941 ) Change subject: IMPALA-5893: Remove old kinit code for Impala 3 .. Patch Set 4: Verified-1 Build failed: https://jenkins.impala

[Impala-ASF-CR] IMPALA-6862: Privilege.java needs to support Sentry 1.5.1 and 2.0.0

2018-04-16 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10082 Change subject: IMPALA-6862: Privilege.java needs to support Sentry 1.5.1 and 2.0.0 .. IMPALA-6862: Privilege.java needs to support Sent

[Impala-ASF-CR] IMPALA-6749: Implement ALTER DATABASE SET DBPROPERTIES

2018-04-16 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10085 Change subject: IMPALA-6749: Implement ALTER DATABASE SET DBPROPERTIES .. IMPALA-6749: Implement ALTER DATABASE SET DBPROPERTIES ALTER D

[Impala-ASF-CR] IMPALA-6749: Implement ALTER DATABASE SET DBPROPERTIES

2018-04-16 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/10085 ) Change subject: IMPALA-6749: Implement ALTER DATABASE SET DBPROPERTIES .. Patch Set 4: Do I need to put Cherry-picks: not for 2.x? It's

[Impala-ASF-CR] IMPALA-6847: work around high memory estimates for AC

2018-04-16 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/10058 ) Change subject: IMPALA-6847: work around high memory estimates for AC .. Patch Set 12: (3 comments) +1 for the simplified version (p