Dimitris Tsirogiannis has uploaded a new patch set (#4).
Change subject: IMPALA-5500: Reduce catalog update topic size
..
IMPALA-5500: Reduce catalog update topic size
Problem: IMPALA-4029 introduced the use of the flatbuffers
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5500: Reduce catalog update topic size
..
Patch Set 3:
(6 comments)
http://gerrit.cloudera.org:8080/#/c/7268/3/be/src/catalog/catalog-server.cc
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5500: Reduce catalog update topic size
..
Patch Set 2:
(7 comments)
http://gerrit.cloudera.org:8080/#/c/7268/2/be/src/catalog/catalog-server.cc
Dimitris Tsirogiannis has uploaded a new patch set (#3).
Change subject: IMPALA-5500: Reduce catalog update topic size
..
IMPALA-5500: Reduce catalog update topic size
Problem: IMPALA-4029 introduced the use of the flatbuffers
Hello Alex Behm,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7299
to look at the new patch set (#2).
Change subject: IMPALA-5571: Fix authorization tests
..
IMPALA-5571: Fix
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5571: Fix authorization tests
..
Patch Set 2: Code-Review+2
Rebase. Keep Alex's +2
--
To view, visit http://gerrit.cloudera.org:8080/7299
To
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5571: Fix authorization tests
..
Patch Set 1:
It's not clear to me yet. I am just posting this to unblock the builds. I need
to investigate this
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4029: [DOCS] Include known issue for 'flat buffers'
change
..
Patch Set 3: Code-Review+2
(3 comments)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5431: Remove redundant path exists checks during table
load
..
Patch Set 6: Code-Review+2
(3 comments)
Dimitris Tsirogiannis has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7299
Change subject: IMPALA-5571: Fix authorization tests
..
IMPALA-5571: Fix authorization tests
Issue:
With the fix for
Dimitris Tsirogiannis has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7296
Change subject: IMPALA-5579: Fix IndexOutOfBoundsException in GetTables
metadata request
..
IMPALA-5579: Fix
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4029: [DOCS] Include known issue for 'flat buffers'
change
..
Patch Set 2:
(1 comment)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 6:
Maybe the associated JIRA is a better place to have this discussion, but I am
trying to
Dimitris Tsirogiannis has uploaded a new patch set (#2).
Change subject: IMPALA-5500: Reduce catalog update topic size
..
IMPALA-5500: Reduce catalog update topic size
Problem: IMPALA-4029 introduced the use of the flatbuffers
Dimitris Tsirogiannis has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7268
Change subject: IMPALA-5500: Reduce catalog update topic size
..
IMPALA-5500: Reduce catalog update topic size
Problem:
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5503: [DOCS] Document how to specify
coordinator/executor nodes
..
Patch Set 2: Code-Review+2
(3 comments)
Much better, thanks.
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5549: Remove deprecated fields from CatalogService API
..
Patch Set 3: Code-Review+2
Rebased. Keep Alex's +2
--
To view, visit
Hello Bharath Vissapragada,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7248
to look at the new patch set (#2).
Change subject: IMPALA-5549: Remove deprecated fields from CatalogService API
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5549: Remove deprecated fields from CatalogService API
..
Patch Set 1:
(1 comment)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5549: Remove deprecated fields from CatalogService API
..
Patch Set 1:
> IIRC, this was introduced because the permanent UDF changes broke
>
Dimitris Tsirogiannis has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7248
Change subject: IMPALA-5549: Remove deprecated fields from CatalogService API
..
IMPALA-5549: Remove deprecated fields from
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5503: [DOCS] Document how to specify
coordinator/executor nodes
..
Patch Set 1:
(2 comments)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4029: [DOCS] Include known issue for 'flat buffers'
change
..
Patch Set 1:
(3 comments)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4029: [DOCS] Include known issue for 'flat buffers'
change
..
Patch Set 1:
> I included a link to IMPALA-4029 in the description of the known
>
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5495: Improve error message if no impalad role is
configured
..
Patch Set 2: Code-Review+2
Thanks for fixing this :)
--
To view, visit
Hello Impala Public Jenkins, Alex Behm, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7064
to look at the new patch set (#13).
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
Patch Set 12: Code-Review+2
Fix more clang-tidy issue. Keep Dan's +2
--
To view,
Hello Impala Public Jenkins, Alex Behm, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7064
to look at the new patch set (#12).
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
Hello Impala Public Jenkins, Alex Behm, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7064
to look at the new patch set (#11).
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
Patch Set 10:
(2 comments)
Thanks for pointing these out.
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5469: Fix exception when processing catalog update
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7121
To
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
Patch Set 10: Code-Review+2
Fixed minor test issue, keep Dan's +2
--
To view,
Hello Impala Public Jenkins, Alex Behm, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7064
to look at the new patch set (#10).
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
Patch Set 9: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7064
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
Patch Set 8: Code-Review+2
Rebase and keep Dan's +2
--
To view, visit
Hello Alex Behm, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7064
to look at the new patch set (#9).
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
Patch Set 8:
(2 comments)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
Patch Set 7:
(6 comments)
Hello Alex Behm,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7064
to look at the new patch set (#8).
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
Patch Set 7:
(2 comments)
I am just answering two questions that Dan had. Let me
Hello Alex Behm,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7064
to look at the new patch set (#7).
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
Patch Set 6:
(1 comment)
Found a bug. Plz wait until I post a new patch before
Dimitris Tsirogiannis has uploaded a new patch set (#6).
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
IMPALA-4965: Authorize access to runtime profile and exec summary
Bug:
When
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
Patch Set 4:
(6 comments)
Dimitris Tsirogiannis has uploaded a new patch set (#5).
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
IMPALA-4965: Authorize access to runtime profile and exec summary
Bug:
When
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
Patch Set 1:
(5 comments)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
Patch Set 4:
Missed the comments in the test file. Sending a new patch in a while.
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
Patch Set 3:
(5 comments)
Dimitris Tsirogiannis has uploaded a new patch set (#4).
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
IMPALA-4965: Authorize access to runtime profile and exec summary
Bug:
When
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 5:
Many of these changes are essentially needed just for generating consistent
tests
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5056: Ensure analysis uses 'fresh' catalog after
metadata loading
..
Patch Set 4: Code-Review+1
(2 comments)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5431: Remove redundant path exists check during table
load
..
Patch Set 1: Code-Review+2
(1 comment)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5355: Fix the order of Sentry roles and privileges
..
Patch Set 4: Code-Review+2
(2 comments)
Dimitris Tsirogiannis has uploaded a new patch set (#3).
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
IMPALA-4965: Authorize access to runtime profile and exec summary
Bug:
When
Dimitris Tsirogiannis has uploaded a new patch set (#2).
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
IMPALA-4965: Authorize access to runtime profile and exec summary
Bug:
When
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
Patch Set 1:
(12 comments)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5420: Skip ACL fetch if the acl bit is not set.
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7071
To
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
Patch Set 1:
The description in the JIRA asks for a redacted profile. I pinged the
Dimitris Tsirogiannis has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7064
Change subject: IMPALA-4965: Authorize access to runtime profile and exec
summary
..
IMPALA-4965: Authorize access to runtime
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5056: Ensure analysis uses 'fresh' catalog after
metadata loading
..
Patch Set 2: Code-Review+2
(1 comment)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5056: Ensure analysis uses 'fresh' catalog after
metadata loading
..
Patch Set 1: Code-Review+2
(1 comment)
Thanks. Not sure if I am
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5355: Fix the order of Sentry roles and privileges
..
Patch Set 1:
(8 comments)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-2373: Extrapolate row counts for HDFS tables.
..
Patch Set 8: Code-Review+2
(4 comments)
http://gerrit.cloudera.org:8080/#/c/6840/6//COMMIT_MSG
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5325: Do not update totalHdfsBytes_/numHdfsFiles_ on
Catalogd
..
Patch Set 3: Code-Review+2
--
To view, visit
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5333: Add support for Impala to work with ADLS
..
Patch Set 6: Code-Review+2
Thanks Sailesh for the explanation.
--
To view, visit
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5259: Add REFRESH FUNCTIONS statement
..
Patch Set 10: Code-Review+2
(1 comment)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5333: Add support for Impala to work with ADLS
..
Patch Set 5:
(3 comments)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5325: Do not update totalHdfsBytes_/numHdfsFiles_ on
Catalogd
..
Patch Set 3:
(1 comment)
I wouldn't worry about the supportability aspect. I
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5259: Add REFRESH FUNCTIONS statement
..
Patch Set 9:
(1 comment)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5259: Add REFRESH FUNCTIONS statement
..
Patch Set 9:
(4 comments)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5342: Add comments of loaded tables in the response of
GetTables
..
Patch Set 3: Code-Review+2
Rebase, keep Alex's +2
--
To view, visit
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-2373: Extrapolate row counts for HDFS tables.
..
Patch Set 4: Code-Review+1
(2 comments)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5259: Add REFRESH FUNCTIONS statement
..
Patch Set 8:
(1 comment)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-2373: Extrapolate row counts for HDFS tables.
..
Patch Set 3:
(8 comments)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5342: Add comments of loaded tables in the response of
GetTables
..
Patch Set 1:
(3 comments)
Dimitris Tsirogiannis has uploaded a new patch set (#2).
Change subject: IMPALA-5342: Add comments of loaded tables in the response of
GetTables
..
IMPALA-5342: Add comments of loaded tables in the response of GetTables
This
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5144: Remove sortby() hint
..
Patch Set 3:
Lars, technically I can't +2 your patch. Ping Alex, it should be trivial to
review.
--
To view,
Dimitris Tsirogiannis has uploaded a new change for review.
http://gerrit.cloudera.org:8080/6933
Change subject: IMPALA-5342: Add comments of loaded tables in the response of
GetTables
..
IMPALA-5342: Add comments of loaded
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5333: Add support for Impala to work with ADLS
..
Patch Set 3:
I have a few high level questions about this patch. This patch treats S3 and
ADL
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5339: Fix analysis with sort.columns and expr rewrites
..
Patch Set 3: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/6921
To
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5333: Add support for Impala to work with ADLS
..
Patch Set 3:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/6910/3/tests/metadata/test_ddl.py
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5339: Fix analysis with sort.columns and expr rewrites
..
Patch Set 2:
(1 comment)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5333: Add support for Impala to work with ADLS
..
Patch Set 2:
(15 comments)
Nice! Mostly formatting nits.
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-2373: Extrapolate row counts for HDFS tables.
..
Patch Set 2:
(7 comments)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4482, IMPALA-4838: RECOVER PARTITIONS with
tpcds.store_sales
..
Patch Set 8:
David, plz abandon or finish this.
--
To view, visit
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5144: Remove sortby() hint
..
Patch Set 3: Code-Review+1
(1 comment)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5144: Remove sortby() hint
..
Patch Set 2:
(3 comments)
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-2373: Extrapolate row counts for HDFS tables.
..
Patch Set 1:
(11 comments)
First pass on main classes. I haven't looked at the tests yet.
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-5318: Generate access events with fully qualified table
names
..
Patch Set 1:
(2 comments)
Dimitris Tsirogiannis has uploaded a new patch set (#2).
Change subject: IMPALA-5318: Generate access events with fully qualified table
names
..
IMPALA-5318: Generate access events with fully qualified table names
This commit
Dimitris Tsirogiannis has uploaded a new change for review.
http://gerrit.cloudera.org:8080/6879
Change subject: IMPALA-5318: Generate access events with fully qualified table
names
..
IMPALA-5318: Generate access events
Hello Marcel Kornacker, Impala Public Jenkins, Bharath Vissapragada, Alex Behm,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6406
to look at the new patch set (#10).
Change subject: IMPALA-4029: Reduce memory requirements for storing file
metadata
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4029: Reduce memory requirements for storing file
metadata
..
Patch Set 9: Code-Review+2
Forgot to push some change.
--
To view, visit
Hello Marcel Kornacker, Bharath Vissapragada, Alex Behm,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6406
to look at the new patch set (#9).
Change subject: IMPALA-4029: Reduce memory requirements for storing file
metadata
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4029: Reduce memory requirements for storing file
metadata
..
Patch Set 8: Code-Review+2
Rebase and fix minor test. Keep Marcel's +2.
--
To
Hello Marcel Kornacker, Bharath Vissapragada, Alex Behm,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6406
to look at the new patch set (#8).
Change subject: IMPALA-4029: Reduce memory requirements for storing file
metadata
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4166: Add SORT BY sql clause
..
Patch Set 20: Code-Review+1
--
To view, visit http://gerrit.cloudera.org:8080/6495
To unsubscribe, visit
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4029: Reduce memory requirements for storing file
metadata
..
Patch Set 6:
(9 comments)
Hello Bharath Vissapragada, Alex Behm,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6406
to look at the new patch set (#6).
Change subject: IMPALA-4029: Reduce memory requirements for storing file
metadata
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-4029: Reduce memory requirements for storing file
metadata
..
Patch Set 5:
(21 comments)
101 - 200 of 492 matches
Mail list logo