[Impala-ASF-CR] IMPALA-8100: Add initial support for Ranger

2019-03-05 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#8). ( http://gerrit.cloudera.org:8080/12632 ) Change subject: IMPALA-8100: Add initial support for Ranger .. IMPALA-8100: Add initial support for Ranger This patch adds an initial sup

[Impala-ASF-CR] IMPALA-8100: Add initial support for Ranger

2019-03-05 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12632 ) Change subject: IMPALA-8100: Add initial support for Ranger .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/12632/7/fe

[Impala-ASF-CR] IMPALA-8100: Add initial support for Ranger

2019-03-05 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#7). ( http://gerrit.cloudera.org:8080/12632 ) Change subject: IMPALA-8100: Add initial support for Ranger .. IMPALA-8100: Add initial support for Ranger This patch adds an initial sup

[Impala-ASF-CR] IMPALA-8255: [DOCS] Document the DEFAULT FILE FORMAT query option

2019-03-05 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12670 ) Change subject: IMPALA-8255: [DOCS] Document the DEFAULT_FILE_FORMAT query option .. Patch Set 2: (1 comment) http://gerrit.cloudera.

[Impala-ASF-CR] IMPALA-7916: Remove support for authorization policy file

2019-03-05 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12637 ) Change subject: IMPALA-7916: Remove support for authorization policy file .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/

[Impala-ASF-CR] IMPALA-7916: Remove support for authorization policy file

2019-03-05 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12637 ) Change subject: IMPALA-7916: Remove support for authorization policy file .. Patch Set 7: Code-Review+1 LGTM, thanks! Will be bumping i

[Impala-ASF-CR] IMPALA-7917 (Part 2): Decouple Sentry from Impala

2019-03-04 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12542 ) Change subject: IMPALA-7917 (Part 2): Decouple Sentry from Impala .. Patch Set 11: Code-Review+1 (2 comments) Carry Bharath's +1. htt

[Impala-ASF-CR] IMPALA-7917 (Part 2): Decouple Sentry from Impala

2019-03-04 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#11). ( http://gerrit.cloudera.org:8080/12542 ) Change subject: IMPALA-7917 (Part 2): Decouple Sentry from Impala .. IMPALA-7917 (Part 2): Decouple Sentry from Impala This patch adds a

[Impala-ASF-CR] IMPALA-7916: Remove support for authorization policy file

2019-03-04 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12637 ) Change subject: IMPALA-7916: Remove support for authorization policy file .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/

[Impala-ASF-CR] IMPALA-8254: Fix error when running compute stats with compression codec set

2019-03-04 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#11). ( http://gerrit.cloudera.org:8080/12635 ) Change subject: IMPALA-8254: Fix error when running compute stats with compression_codec set .. IMPALA-8254: Fix error when running comp

[Impala-ASF-CR] IMPALA-8254: Fix error when running compute stats with compression codec set

2019-03-04 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12635 ) Change subject: IMPALA-8254: Fix error when running compute stats with compression_codec set .. Patch Set 11: Code-Review+2 (2 comment

[Impala-ASF-CR] IMPALA-7917 (Part 2): Decouple Sentry from Impala

2019-03-04 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#8). ( http://gerrit.cloudera.org:8080/12542 ) Change subject: IMPALA-7917 (Part 2): Decouple Sentry from Impala .. IMPALA-7917 (Part 2): Decouple Sentry from Impala This patch adds a

[Impala-ASF-CR] IMPALA-7917 (Part 2): Decouple Sentry from Impala

2019-03-04 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12542 ) Change subject: IMPALA-7917 (Part 2): Decouple Sentry from Impala .. Patch Set 8: (8 comments) http://gerrit.cloudera.org:8080/#/c/125

[Impala-ASF-CR] IMPALA-8261: Enhance create-test-configuration.sh to not fail when FE has not been built

2019-03-04 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12630 ) Change subject: IMPALA-8261: Enhance create-test-configuration.sh to not fail when FE has not been built .. Patch Set 4: Build looks g

[Impala-ASF-CR] IMPALA-8100: Add initial support for Ranger

2019-03-04 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12632 ) Change subject: IMPALA-8100: Add initial support for Ranger .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/12632/6/be

[Impala-ASF-CR] IMPALA-8100: Add initial support for Ranger

2019-03-04 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12632 ) Change subject: IMPALA-8100: Add initial support for Ranger .. Patch Set 6: (4 comments) http://gerrit.cloudera.org:8080/#/c/12632/5/f

[Impala-ASF-CR] IMPALA-8100: Add initial support for Ranger

2019-03-04 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#6). ( http://gerrit.cloudera.org:8080/12632 ) Change subject: IMPALA-8100: Add initial support for Ranger .. IMPALA-8100: Add initial support for Ranger This patch adds an initial sup

[Impala-ASF-CR] IMPALA-8100: Add initial support for Ranger

2019-03-04 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12632 ) Change subject: IMPALA-8100: Add initial support for Ranger .. Patch Set 5: This is ready for review now. It depends on this CR: https

[Impala-ASF-CR] IMPALA-8100: Add initial support for Ranger

2019-03-04 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/12632 ) Change subject: IMPALA-8100: Add initial support for Ranger .. IMPALA-8100: Add initial support for Ranger This patch adds an initial sup

[Impala-ASF-CR] IMPALA-8254: Fix error when running compute stats with compression codec set

2019-03-01 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12635 ) Change subject: IMPALA-8254: Fix error when running compute stats with compression_codec set .. Patch Set 10: I created a follow-up JI

[Impala-ASF-CR] IMPALA-8261: Enhance create-test-configuration.sh to not fail when FE has not been built

2019-03-01 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12630 ) Change subject: IMPALA-8261: Enhance create-test-configuration.sh to not fail when FE has not been built .. Patch Set 3: > Patch Set 3

[Impala-ASF-CR] IMPALA-8254: Fix error when running compute stats with compression codec set

2019-03-01 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#10). ( http://gerrit.cloudera.org:8080/12635 ) Change subject: IMPALA-8254: Fix error when running compute stats with compression_codec set .. IMPALA-8254: Fix error when running comp

[Impala-ASF-CR] IMPALA-7916: Remove support for authorization policy file flag

2019-03-01 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12637 ) Change subject: IMPALA-7916: Remove support for authorization policy file flag .. Patch Set 5: (4 comments) http://gerrit.cloudera.org

[Impala-ASF-CR] IMPALA-8254: Fix error when running compute stats with compression codec set

2019-03-01 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12635 ) Change subject: IMPALA-8254: Fix error when running compute stats with compression_codec set .. Patch Set 9: (4 comments) http://gerr

[Impala-ASF-CR] IMPALA-8254: Fix error when running compute stats with compression codec set

2019-03-01 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#9). ( http://gerrit.cloudera.org:8080/12635 ) Change subject: IMPALA-8254: Fix error when running compute stats with compression_codec set .. IMPALA-8254: Fix error when running compu

[Impala-ASF-CR] IMPALA-7916: Remove support for authorization policy file flag

2019-03-01 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12637 ) Change subject: IMPALA-7916: Remove support for authorization policy file flag .. Patch Set 4: (5 comments) Thanks! Overall it looks g

[Impala-ASF-CR] IMPALA-8254: Fix error when running compute stats with compression codec set

2019-03-01 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12635 ) Change subject: IMPALA-8254: Fix error when running compute stats with compression_codec set .. Patch Set 8: (3 comments) http://gerr

[Impala-ASF-CR] IMPALA-8254: Fix error when running compute stats with compression codec set

2019-03-01 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#8). ( http://gerrit.cloudera.org:8080/12635 ) Change subject: IMPALA-8254: Fix error when running compute stats with compression_codec set .. IMPALA-8254: Fix error when running compu

[Impala-ASF-CR] IMPALA-7916: Remove support for authorization policy file flag

2019-02-28 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12637 ) Change subject: IMPALA-7916: Remove support for authorization policy file flag .. Patch Set 3: (12 comments) http://gerrit.cloudera.or

[Impala-ASF-CR] IMPALA-8254: Fix error when running compute stats with compression codec set

2019-02-28 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#7). ( http://gerrit.cloudera.org:8080/12635 ) Change subject: IMPALA-8254: Fix error when running compute stats with compression_codec set .. IMPALA-8254: Fix error when running compu

[Impala-ASF-CR] IMPALA-8254: Fix error when running compute stats with compression codec set

2019-02-28 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12635 ) Change subject: IMPALA-8254: Fix error when running compute stats with compression_codec set .. Patch Set 7: (1 comment) http://gerri

[Impala-ASF-CR] IMPALA-8261: Enhance create-test-configuration.sh to not fail when FE has not been built

2019-02-28 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12630 ) Change subject: IMPALA-8261: Enhance create-test-configuration.sh to not fail when FE has not been built .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-8261: Enhance create-test-configuration.sh to not fail when FE has not been built

2019-02-28 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/12630 ) Change subject: IMPALA-8261: Enhance create-test-configuration.sh to not fail when FE has not been built .. IMPALA-8261: Enhance create-t

[Impala-ASF-CR] IMPALA-7916: Remove support for authorization policy file flag

2019-02-28 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12637 ) Change subject: IMPALA-7916: Remove support for authorization policy file flag .. Patch Set 2: (6 comments) I did a quick look at this

[Impala-ASF-CR] IMPALA-8254: Fix error when running compute stats with compression codec set

2019-02-28 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12635 ) Change subject: IMPALA-8254: Fix error when running compute stats with compression_codec set .. Patch Set 5: (1 comment) http://gerri

[Impala-ASF-CR] IMPALA-8254: Fix error when running compute stats with compression codec set

2019-02-28 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/12635 ) Change subject: IMPALA-8254: Fix error when running compute stats with compression_codec set .. IMPALA-8254: Fix error when running compu

[Impala-ASF-CR] IMPALA-8254: Fix error when running compute stats with compression codec set

2019-02-28 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/12635 ) Change subject: IMPALA-8254: Fix error when running compute stats with compression_codec set .. IMPALA-8254: Fix error when running compu

[Impala-ASF-CR] IMPALA-8254: Fix error when running compute stats with compression codec set

2019-02-28 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12635 ) Change subject: IMPALA-8254: Fix error when running compute stats with compression_codec set .. Patch Set 4: (2 comments) http://gerr

[Impala-ASF-CR] IMPALA-8254: Fix error when running compute stats with compression codec set

2019-02-27 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12635 Change subject: IMPALA-8254: Fix error when running compute stats with compression_codec set .. IMPALA-8254: Fix error when running comp

[Impala-ASF-CR] IMPALA-8100 (WIP): Add initial support for Ranger

2019-02-27 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12632 ) Change subject: IMPALA-8100 (WIP): Add initial support for Ranger .. Patch Set 3: Make the rat happy. -- To view, visit http://gerrit

[Impala-ASF-CR] IMPALA-8100 (WIP): Add initial support for Ranger

2019-02-27 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/12632 ) Change subject: IMPALA-8100 (WIP): Add initial support for Ranger .. IMPALA-8100 (WIP): Add initial support for Ranger This patch adds an

[Impala-ASF-CR] IMPALA-8100 (WIP): Add initial support for Ranger

2019-02-27 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12632 Change subject: IMPALA-8100 (WIP): Add initial support for Ranger .. IMPALA-8100 (WIP): Add initial support for Ranger This patch adds a

[Impala-ASF-CR] IMPALA-8100 (WIP): Add initial support for Ranger

2019-02-27 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12632 ) Change subject: IMPALA-8100 (WIP): Add initial support for Ranger .. Patch Set 2: This is a sneak-peak of initial support for Ranger th

[Impala-ASF-CR] IMPALA-8261: Enhance create-test-configuration.sh to not fail when FE has not been built

2019-02-27 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12630 Change subject: IMPALA-8261: Enhance create-test-configuration.sh to not fail when FE has not been built .. IMPALA-8261: Enhance create-

[Impala-ASF-CR] IMPALA-8240: Event processor should keep trying when metastore is unavailable.

2019-02-26 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12601 ) Change subject: IMPALA-8240: Event processor should keep trying when metastore is unavailable. .. Patch Set 4: (10 comments) http://g

[Impala-ASF-CR] IMPALA-7645: Add a query option to set the default table file format

2019-02-25 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#7). ( http://gerrit.cloudera.org:8080/12568 ) Change subject: IMPALA-7645: Add a query option to set the default table file format .. IMPALA-7645: Add a query option to set the defaul

[Impala-ASF-CR] IMPALA-7645: Add a query option to set the default table file format

2019-02-25 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12568 ) Change subject: IMPALA-7645: Add a query option to set the default table file format .. Patch Set 7: (5 comments) http://gerrit.cloud

[Impala-ASF-CR] IMPALA-7645: Add a query option to set the default table file format

2019-02-25 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12568 ) Change subject: IMPALA-7645: Add a query option to set the default table file format .. Patch Set 6: > Patch Set 6: > > Can the file f

[Impala-ASF-CR] IMPALA-7645: Add a query option to set the default table file format

2019-02-25 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12568 ) Change subject: IMPALA-7645: Add a query option to set the default table file format .. Patch Set 6: Code-Review+1 (1 comment) Carry

[Impala-ASF-CR] IMPALA-7645: Add a query option to set the default table file format

2019-02-25 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#6). ( http://gerrit.cloudera.org:8080/12568 ) Change subject: IMPALA-7645: Add a query option to set the default table file format .. IMPALA-7645: Add a query option to set the defaul

[Impala-ASF-CR] IMPALA-7917 (Part 2): Decouple Sentry from Impala

2019-02-25 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#7). ( http://gerrit.cloudera.org:8080/12542 ) Change subject: IMPALA-7917 (Part 2): Decouple Sentry from Impala .. IMPALA-7917 (Part 2): Decouple Sentry from Impala This patch adds a

[Impala-ASF-CR] IMPALA-7917 (Part 2): Decouple Sentry from Impala

2019-02-25 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12542 ) Change subject: IMPALA-7917 (Part 2): Decouple Sentry from Impala .. Patch Set 6: (7 comments) I redesigned the CR to be reduce the le

[Impala-ASF-CR] IMPALA-7917 (Part 2): Decouple Sentry from Impala

2019-02-25 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#6). ( http://gerrit.cloudera.org:8080/12542 ) Change subject: IMPALA-7917 (Part 2): Decouple Sentry from Impala .. IMPALA-7917 (Part 2): Decouple Sentry from Impala This patch adds a

[Impala-ASF-CR] IMPALA-7645: Add a query option to set default table file format

2019-02-23 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12568 Change subject: IMPALA-7645: Add a query option to set default table file format .. IMPALA-7645: Add a query option to set default table

[Impala-ASF-CR](2.x) IMPALA-7140 (part 5): support fetching file info for FS tables

2019-02-23 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12569 ) Change subject: IMPALA-7140 (part 5): support fetching file info for FS tables .. Patch Set 1: Code-Review+2 -- To view, visit http://

[Impala-ASF-CR] IMPALA-7645: Add a query option to set the default table file format

2019-02-23 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/12568 ) Change subject: IMPALA-7645: Add a query option to set the default table file format .. IMPALA-7645: Add a query option to set the defaul

[Impala-ASF-CR] IMPALA-7917 (Part 2): Decouple Sentry from Impala

2019-02-21 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/12542 ) Change subject: IMPALA-7917 (Part 2): Decouple Sentry from Impala .. IMPALA-7917 (Part 2): Decouple Sentry from Impala This patch adds a

[Impala-ASF-CR] IMPALA-7917 (Part 2): Decouple Sentry from Impala

2019-02-21 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12542 ) Change subject: IMPALA-7917 (Part 2): Decouple Sentry from Impala .. Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/125

[Impala-ASF-CR] IMPALA-8233: Do not re-download Ranger if it is already downloaded

2019-02-21 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12541 ) Change subject: IMPALA-8233: Do not re-download Ranger if it is already downloaded .. Patch Set 3: Code-Review+2 (2 comments) Carry P

[Impala-ASF-CR] IMPALA-8233: Do not re-download Ranger if it is already downloaded

2019-02-21 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/12541 ) Change subject: IMPALA-8233: Do not re-download Ranger if it is already downloaded .. IMPALA-8233: Do not re-download Ranger if it is alr

[Impala-ASF-CR] IMPALA-8168: Disable Sentry HDFS sync on S3

2019-02-20 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12482 Change subject: IMPALA-8168: Disable Sentry HDFS sync on S3 .. IMPALA-8168: Disable Sentry HDFS sync on S3 This patch updates the build

[Impala-ASF-CR] IMPALA-7917 (Part 2): Decouple Sentry from Impala

2019-02-20 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12542 Change subject: IMPALA-7917 (Part 2): Decouple Sentry from Impala .. IMPALA-7917 (Part 2): Decouple Sentry from Impala This patch adds a

[Impala-ASF-CR] IMPALA-8233: Do not re-download Ranger if it is already downloaded

2019-02-20 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12541 Change subject: IMPALA-8233: Do not re-download Ranger if it is already downloaded .. IMPALA-8233: Do not re-download Ranger if it is al

[Impala-ASF-CR] IMPALA-7917 (Part 1): Decouple Sentry from Impala

2019-02-19 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#12). ( http://gerrit.cloudera.org:8080/12020 ) Change subject: IMPALA-7917 (Part 1): Decouple Sentry from Impala .. IMPALA-7917 (Part 1): Decouple Sentry from Impala The first part of

[Impala-ASF-CR] IMPALA-7917 (Part 1): Decouple Sentry from Impala

2019-02-19 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12020 ) Change subject: IMPALA-7917 (Part 1): Decouple Sentry from Impala .. Patch Set 12: Rebased to fix merge conflicts. Paul/Bharath, can yo

[Impala-ASF-CR] IMPALA-8099: Update the build scripts to support Apache Ranger

2019-02-13 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12469 ) Change subject: IMPALA-8099: Update the build scripts to support Apache Ranger .. Patch Set 3: I ran all tests with test-with-docker an

[Impala-ASF-CR] IMPALA-8099: Update the build scripts to support Apache Ranger

2019-02-13 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/12469 ) Change subject: IMPALA-8099: Update the build scripts to support Apache Ranger .. IMPALA-8099: Update the build scripts to support Apache

[Impala-ASF-CR] IMPALA-7917 (Part 1): Decouple Sentry from Impala

2019-02-13 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#11). ( http://gerrit.cloudera.org:8080/12020 ) Change subject: IMPALA-7917 (Part 1): Decouple Sentry from Impala .. IMPALA-7917 (Part 1): Decouple Sentry from Impala The first part of

[Impala-ASF-CR] IMPALA-7917 (Part 1): Decouple Sentry from Impala

2019-02-13 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12020 ) Change subject: IMPALA-7917 (Part 1): Decouple Sentry from Impala .. Patch Set 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/120

[Impala-ASF-CR] IMPALA-7917 (Part 1): Decouple Sentry from Impala

2019-02-13 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12020 ) Change subject: IMPALA-7917 (Part 1): Decouple Sentry from Impala .. Patch Set 10: This change is mostly on moving some files, some bas

[Impala-ASF-CR] IMPALA-7917 (Part 1): Decouple Sentry from Impala

2019-02-13 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#10). ( http://gerrit.cloudera.org:8080/12020 ) Change subject: IMPALA-7917 (Part 1): Decouple Sentry from Impala .. IMPALA-7917 (Part 1): Decouple Sentry from Impala The first part of

[Impala-ASF-CR] IMPALA-7917 (Part 1): Decouple Sentry from Impala

2019-02-13 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12020 ) Change subject: IMPALA-7917 (Part 1): Decouple Sentry from Impala .. Patch Set 10: (10 comments) http://gerrit.cloudera.org:8080/#/c/1

[Impala-ASF-CR] IMPALA-7917 (Part 1): Decouple Sentry from Impala

2019-02-13 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12020 ) Change subject: IMPALA-7917 (Part 1): Decouple Sentry from Impala .. Patch Set 9: Rebased and fixed merge conflicts. -- To view, visi

[Impala-ASF-CR] IMPALA-7917 (Part 1): Decouple Sentry from Impala

2019-02-13 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#9). ( http://gerrit.cloudera.org:8080/12020 ) Change subject: IMPALA-7917 (Part 1): Decouple Sentry from Impala .. IMPALA-7917 (Part 1): Decouple Sentry from Impala The first part of

[Impala-ASF-CR] IMPALA-8099: Update the build scripts to support Apache Ranger

2019-02-13 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12469 Change subject: IMPALA-8099: Update the build scripts to support Apache Ranger .. IMPALA-8099: Update the build scripts to support Apache

[Impala-ASF-CR](2.x) IMPALA-8155: Modify bootstrap system.sh to bind Impala-lzo/2.x

2019-02-11 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12339 ) Change subject: IMPALA-8155: Modify bootstrap_system.sh to bind Impala-lzo/2.x .. Patch Set 4: Code-Review+2 -- To view, visit http://

[Impala-ASF-CR](2.x) IMPALA-7128 (part 1) Refactor interfaces for Db, View, Table, Partition

2019-02-09 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12345 ) Change subject: IMPALA-7128 (part 1) Refactor interfaces for Db, View, Table, Partition .. Patch Set 2: Code-Review+2 -- To view, vis

[Impala-ASF-CR] IMPALA-8044: [DOCS] Document REFRESH AUTHORIZATION statement

2019-02-07 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12378 ) Change subject: IMPALA-8044: [DOCS] Document REFRESH AUTHORIZATION statement .. Patch Set 4: Code-Review+2 -- To view, visit http://ge

[Impala-ASF-CR] IMPALA-7917 (Part 1): Decouple Sentry from Impala

2019-02-07 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has removed Philip Zeyliger from this change. ( http://gerrit.cloudera.org:8080/12020 ) Change subject: IMPALA-7917 (Part 1): Decouple Sentry from Impala .. Removed reviewer Philip Zeyliger. -- To view, visit http

[Impala-ASF-CR] IMPALA-7917 (Part 1): Decouple Sentry from Impala

2019-02-07 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12020 ) Change subject: IMPALA-7917 (Part 1): Decouple Sentry from Impala .. Patch Set 8: Paul, this is ready for another review. Can you take

[Impala-ASF-CR] IMPALA-8044: [DOCS] Document REFRESH AUTHORIZATION statement

2019-02-07 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12378 ) Change subject: IMPALA-8044: [DOCS] Document REFRESH AUTHORIZATION statement .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8

[Impala-ASF-CR] IMPALA-7917 (Part 1): Decouple Sentry from Impala

2019-02-07 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#8). ( http://gerrit.cloudera.org:8080/12020 ) Change subject: IMPALA-7917 (Part 1): Decouple Sentry from Impala .. IMPALA-7917 (Part 1): Decouple Sentry from Impala The first part of

[Impala-ASF-CR] IMPALA-7917 (Part 1): Decouple Sentry from Impala

2019-02-07 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12020 ) Change subject: IMPALA-7917 (Part 1): Decouple Sentry from Impala .. Patch Set 8: (7 comments) http://gerrit.cloudera.org:8080/#/c/120

[Impala-ASF-CR] IMPALA-8148: Misc. FE code cleanup

2019-02-05 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12317 ) Change subject: IMPALA-8148: Misc. FE code cleanup .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/123

[Impala-ASF-CR] IMPALA-8104: IF NOT EXISTS to ALTER TABLE ADD COLUMN(S)

2019-02-05 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12361 ) Change subject: IMPALA-8104: IF NOT EXISTS to ALTER TABLE ADD COLUMN(S) .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.

[Impala-ASF-CR] IMPALA-8104: IF NOT EXISTS to ALTER TABLE ADD COLUMN(S)

2019-02-05 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12361 ) Change subject: IMPALA-8104: IF NOT EXISTS to ALTER TABLE ADD COLUMN(S) .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#

[Impala-ASF-CR] Enable full stacktrace logging with surefire

2019-02-05 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12370 ) Change subject: Enable full stacktrace logging with surefire .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.or

[Impala-ASF-CR] IMPALA-8150: Fix buggy AuditingTest::TestAccessEventsOnAuthFailure

2019-02-04 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12334 ) Change subject: IMPALA-8150: Fix buggy AuditingTest::TestAccessEventsOnAuthFailure .. Patch Set 6: Code-Review+2 So apparently, to ena

[Impala-ASF-CR] IMPALA-8150: Fix buggy AuditingTest::TestAccessEventsOnAuthFailure

2019-02-04 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#6). ( http://gerrit.cloudera.org:8080/12334 ) Change subject: IMPALA-8150: Fix buggy AuditingTest::TestAccessEventsOnAuthFailure .. IMPALA-8150: Fix buggy AuditingTest::TestAccessEven

[Impala-ASF-CR] IMPALA-8150: Fix buggy AuditingTest::TestAccessEventsOnAuthFailure

2019-02-04 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12334 ) Change subject: IMPALA-8150: Fix buggy AuditingTest::TestAccessEventsOnAuthFailure .. Patch Set 5: (4 comments) http://gerrit.clouder

[Impala-ASF-CR] IMPALA-8150: Fix buggy AuditingTest::TestAccessEventsOnAuthFailure

2019-02-04 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/12334 ) Change subject: IMPALA-8150: Fix buggy AuditingTest::TestAccessEventsOnAuthFailure .. IMPALA-8150: Fix buggy AuditingTest::TestAccessEven

[Impala-ASF-CR] IMPALA-8150: Fix buggy AuditingTest::TestAccessEventsOnAuthFailure

2019-02-01 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12334 ) Change subject: IMPALA-8150: Fix buggy AuditingTest::TestAccessEventsOnAuthFailure .. Patch Set 4: Rebased to fix gerrit-code-review-c

[Impala-ASF-CR] IMPALA-8150: Fix buggy AuditingTest::TestAccessEventsOnAuthFailure

2019-02-01 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/12334 ) Change subject: IMPALA-8150: Fix buggy AuditingTest::TestAccessEventsOnAuthFailure .. IMPALA-8150: Fix buggy AuditingTest::TestAccessEven

[Impala-ASF-CR](2.x) IMPALA-7144: Re-enable TestDescribeTableResults

2019-02-01 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12331 ) Change subject: IMPALA-7144: Re-enable TestDescribeTableResults .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera

[Impala-ASF-CR] IMPALA-8148: Misc. FE code cleanup

2019-02-01 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12317 ) Change subject: IMPALA-8148: Misc. FE code cleanup .. Patch Set 3: Code-Review+1 (2 comments) http://gerrit.cloudera.org:8080/#/c/1231

[Impala-ASF-CR] IMPALA-8150: Fix buggy AuditingTest::TestAccessEventsOnAuthFailure

2019-02-01 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12334 Change subject: IMPALA-8150: Fix buggy AuditingTest::TestAccessEventsOnAuthFailure .. IMPALA-8150: Fix buggy AuditingTest::TestAccessEve

[Impala-ASF-CR](2.x) IMPALA-6479: Update DESCRIBE to respect column privileges

2019-01-31 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12292 ) Change subject: IMPALA-6479: Update DESCRIBE to respect column privileges .. Patch Set 2: Code-Review+2 I spoke to Bharath offline and

[Impala-ASF-CR] IMPALA-7934: Switch to java.util.Base64 implementation

2019-01-31 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12250 ) Change subject: IMPALA-7934: Switch to java.util.Base64 implementation .. Patch Set 4: (2 comments) Added some benchmark numbers. htt

[Impala-ASF-CR] IMPALA-7934: Switch to java.util.Base64 implementation

2019-01-31 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/12250 ) Change subject: IMPALA-7934: Switch to java.util.Base64 implementation .. IMPALA-7934: Switch to java.util.Base64 implementation It is sh

[Impala-ASF-CR] IMPALA-5872: Testcase builder for query planner

2019-01-30 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/12221 ) Change subject: IMPALA-5872: Testcase builder for query planner .. Patch Set 3: (7 comments) The authorization looks correct, but we n

<    1   2   3   4   5   6   7   8   9   10   >