[Impala-ASF-CR] draft version -- not completed yet

2016-08-09 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#2). Change subject: draft version -- not completed yet .. draft version -- not completed yet update, modified descriptor part, toTriftDescriptor

[Impala-ASF-CR] draft version -- not completed yet

2016-08-09 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: draft version -- not completed yet .. Patch Set 2: (19 comments) http://gerrit.cloudera.org:8080/#/c/3863/1/be/src/exec/info-schema-scan-node.cc File

[Impala-ASF-CR] draft version -- not completed yet

2016-08-15 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#5). Change subject: draft version -- not completed yet .. draft version -- not completed yet patch 5 1. solve NullPointerExecption and other bugs frontend works 2. solve back end compile

[Impala-ASF-CR] draft version -- not completed yet

2016-08-11 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#4). Change subject: draft version -- not completed yet .. draft version -- not completed yet patch4: pass compile, solve priveledge problem, but throw NullPointerException when

[Impala-CR](cdh5-trunk) IMPALA-3715: Include total usage of JVM memory

2016-07-12 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-3715: Include total usage of JVM memory .. Patch Set 2: Thanks. I have fixed those. -- To view, visit http://gerrit.cloudera.org:8080/3625 To unsubscribe,

[Impala-CR](cdh5-trunk) IMPALA-3716: Add Memory Tab in query's Details page

2016-07-20 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-3716: Add Memory Tab in query's Details page .. Patch Set 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/3664/4/be/src/service/impala-http-handler.cc

[Impala-CR](cdh5-trunk) IMPALA-3716: Add Memory Tab in query's Details page

2016-07-20 Thread Kathy Sun (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3664 to look at the new patch set (#5). Change subject: IMPALA-3716: Add Memory Tab in query's Details page ..

[Impala-CR](cdh5-trunk) IMPALA-3716: Add Memory Tab in query's Details page

2016-07-18 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-3716: Add Memory Tab in query's Details page .. Patch Set 3: > (1 comment) > > Gerrit shows the trailing whitespace in red for me. > > I also have a

[Impala-CR](cdh5-trunk) IMPALA-3716: Add Memory Tab in query's Details page

2016-07-18 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new change for review. http://gerrit.cloudera.org:8080/3664 Change subject: IMPALA-3716: Add Memory Tab in query's Details page .. IMPALA-3716: Add Memory Tab in query's Details page Add a

[Impala-CR](cdh5-trunk) IMPALA-3716: Add Memory Tab in query's Details page

2016-07-18 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#2). Change subject: IMPALA-3716: Add Memory Tab in query's Details page .. IMPALA-3716: Add Memory Tab in query's Details page Add a QueryMemoryHandler and register it to memory tab's

[Impala-CR](cdh5-trunk) IMPALA-3716: Add Memory Tab in query's Details page

2016-07-18 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-3716: Add Memory Tab in query's Details page .. Patch Set 1: > Is query_memory.tmpl missing from this patch? Don't forget to do > this once: > > git add

[Impala-CR](cdh5-trunk) IMPALA-3716: Add Memory Tab in query's Details page

2016-07-18 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-3716: Add Memory Tab in query's Details page .. Patch Set 2: (4 comments) > (4 comments) Oops.. fixed them. Tim, how could you even notice there is a

[Impala-CR](cdh5-trunk) IMPALA-3716: Add Memory Tab in query's Details page

2016-07-18 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#3). Change subject: IMPALA-3716: Add Memory Tab in query's Details page .. IMPALA-3716: Add Memory Tab in query's Details page Add a QueryMemoryHandler and register it to memory tab's

[Impala-CR](cdh5-trunk) IMPALA-3716: Add Memory Tab in query's Details page

2016-07-19 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-3716: Add Memory Tab in query's Details page .. Patch Set 3: (7 comments) http://gerrit.cloudera.org:8080/#/c/3664/3/be/src/service/impala-http-handler.cc

[Impala-CR](cdh5-trunk) IMPALA-3716: Add Memory Tab in query's Details page

2016-07-19 Thread Kathy Sun (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3664 to look at the new patch set (#4). Change subject: IMPALA-3716: Add Memory Tab in query's Details page ..

[Impala-CR](cdh5-trunk) IMPALA-3716: Add Memory Tab in query's Details page

2016-07-19 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-3716: Add Memory Tab in query's Details page .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3664/3/be/src/service/impala-http-handler.cc

[Impala-CR](cdh5-trunk) IMPALA-3715: Include more info by default in Impala debug memz webpage

2016-07-12 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-3715: Include more info by default in Impala debug memz webpage .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3540 To

[Impala-CR](cdh5-trunk) IMPALA-3715: Include more info by default in Impala debug memz webpage

2016-07-12 Thread Kathy Sun (Code Review)
Kathy Sun has submitted this change and it was merged. Change subject: IMPALA-3715: Include more info by default in Impala debug memz webpage .. IMPALA-3715: Include more info by default in Impala debug memz webpage Testing:

[Impala-ASF-CR] draft version -- not completed yet

2016-08-08 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new change for review. http://gerrit.cloudera.org:8080/3863 Change subject: draft version -- not completed yet .. draft version -- not completed yet To Tim, This is the version before we talked

[Impala-CR](cdh5-trunk) IMPALA-3715: Include more info by default in Impala debug memz webpage

2016-07-01 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new change for review. http://gerrit.cloudera.org:8080/3540 Change subject: IMPALA-3715: Include more info by default in Impala debug memz webpage .. IMPALA-3715: Include more info by default in

[Impala-CR](cdh5-trunk) IMPALA-3715: Include more info by default in Impala debug memz webpage

2016-07-02 Thread Kathy Sun (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3540 to look at the new patch set (#4). Change subject: IMPALA-3715: Include more info by default in Impala debug memz webpage

[Impala-CR](cdh5-trunk) IMPALA-3715: Include more info by default in Impala debug memz webpage

2016-07-02 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-3715: Include more info by default in Impala debug memz webpage .. Patch Set 4: remove the conditional on 'detailed' in memz.tmpl -- To view, visit

[Impala-ASF-CR] IMPALA-3981: Fixed the bug of "Crash when access statestore/catalog memz webpage"

2016-08-16 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#2). Change subject: IMPALA-3981: Fixed the bug of "Crash when access statestore/catalog memz webpage" .. IMPALA-3981: Fixed the bug of "Crash when access statestore/catalog memz webpage"

[Impala-ASF-CR] IMPALA-3981: Fixed the bug of "Crash when access statestore/catalog memz webpage"

2016-08-16 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-3981: Fixed the bug of "Crash when access statestore/catalog memz webpage" .. Patch Set 2: I will add test of webpage in next patch -- To view, visit

[Impala-ASF-CR] IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page

2016-08-17 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#3). Change subject: IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page .. IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page The /memz page

[Impala-ASF-CR] IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page

2016-08-17 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page .. Patch Set 3: (3 comments)

[Impala-ASF-CR] IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page

2016-08-17 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page .. Patch Set 4: (7 comments)

[Impala-ASF-CR] IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page

2016-08-17 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#5). Change subject: IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page .. IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page The /memz page

[Impala-ASF-CR] IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page

2016-08-18 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#6). Change subject: IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page .. IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page The /memz page

[Impala-ASF-CR] IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page

2016-08-18 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page .. Patch Set 6: (4 comments)

[Impala-ASF-CR] IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page

2016-08-19 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#7). Change subject: IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page .. IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page The /memz page

[Impala-ASF-CR] IMPALA-3981: Fixed the bug of "Crash when access statestore/catalog memz webpage"

2016-08-15 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new change for review. http://gerrit.cloudera.org:8080/3998 Change subject: IMPALA-3981: Fixed the bug of "Crash when access statestore/catalog memz webpage" .. IMPALA-3981: Fixed the bug of "Crash

[Impala-ASF-CR] IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page

2016-08-19 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#8). Change subject: IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page .. IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page The /memz page

[Impala-ASF-CR] IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page

2016-08-19 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page .. Patch Set 7: (13 comments)

[Impala-ASF-CR] IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page

2016-08-19 Thread Kathy Sun (Code Review)
Hello Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3998 to look at the new patch set (#9). Change subject: IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page

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

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

[Impala-ASF-CR] save as draft, checkout to see. text file

2016-08-24 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new change for review. http://gerrit.cloudera.org:8080/4109 Change subject: save as draft, checkout to see. text file .. save as draft, checkout to see. text file Change-Id:

[Impala-ASF-CR] change name, next work on loop

2016-08-24 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new change for review. http://gerrit.cloudera.org:8080/4110 Change subject: change name, next work on loop .. change name, next work on loop Change-Id: I0bbd707a845460ff979fc1b515edc6d4a3808636 --- M

[Impala-ASF-CR] change name, next work on loop

2016-08-24 Thread Kathy Sun (Code Review)
Kathy Sun has abandoned this change. Change subject: change name, next work on loop .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4110 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

[Impala-ASF-CR] save as draft, checkout to see. text file

2016-08-24 Thread Kathy Sun (Code Review)
Kathy Sun has abandoned this change. Change subject: save as draft, checkout to see. text file .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4109 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

[Impala-ASF-CR] System Database (preview for frontend)

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

[Impala-ASF-CR] System Database (preview for frontend)

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

[Impala-ASF-CR] System Database (preview for frontend)

2016-08-24 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: System Database (preview for frontend) .. Patch Set 9: (6 comments) I change the names and re-upload the test file.

[Impala-ASF-CR] System Database (preview for frontend)

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

[Impala-ASF-CR] System Database (preview for frontend)

2016-08-24 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: System Database (preview for frontend) .. Patch Set 12: system-table-scan-node.cc and system-table-scan-node.h are not new files, it comes from the rename of

[Impala-ASF-CR] System Database (preview for frontend)

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

[Impala-ASF-CR] System Database (preview for frontend)

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

[Impala-ASF-CR] System Database (preview for frontend)

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

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-08-31 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#16). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-08-31 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#16). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-08-31 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#17). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-08-31 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#16). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-08-31 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#16). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-08-31 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. Patch Set 16: This patch add a SystemTableScanner and its subclass MetricScanner to

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-09-06 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#20). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-09-06 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. Patch Set 19: (4 comments) 1.Refactor code in system-table-scan-node remove functions

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-09-06 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#19). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-09-06 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#19). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-09-06 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. Patch Set 18: (28 comments)

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-09-01 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#18). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-09-02 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#18). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-09-02 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. Patch Set 18: (15 comments) made change based on review. 1. assign table id to metrics

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-09-02 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#18). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-08-30 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#15). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-08-30 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#15). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-08-30 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#15). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-08-30 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#15). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-08-30 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. Patch Set 15: This two patch 1. add authz test and conjunct test. 2. make the scan

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-09-08 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#21). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-09-08 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. Patch Set 20: (25 comments) > We also need to fix some of the describe commands: > >

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-09-08 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#22). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-09-08 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. Patch Set 22: patch 22: "fixed some of the describe commands" show create table metrics

[Impala-ASF-CR] IMPALA-3981: Fixed the bug of "Crash when access statestore/catalog memz webpage"

2016-08-17 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: IMPALA-3981: Fixed the bug of "Crash when access statestore/catalog memz webpage" .. Patch Set 2: (7 comments)

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

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

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

2016-08-23 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: Infomation_schema (preview for frontend) .. Patch Set 8: (13 comments) Please skip patch 7, it's a wrong reformat. Patch 8 is a new reformat using .clang_format

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

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

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

2016-08-23 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: Infomation_schema (preview for frontend) .. Patch Set 7: (1 comment) Patch 7 is only for reformatting. you can skip reviewing this

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-08-29 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#14). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-08-29 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#14). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose

[Impala-ASF-CR] System Database (preview for frontend)

2016-08-29 Thread Kathy Sun (Code Review)
Kathy Sun has posted comments on this change. Change subject: System Database (preview for frontend) .. Patch Set 13: (15 comments) http://gerrit.cloudera.org:8080/#/c/3863/13/be/src/exec/system-table-scan-node.cc File

[Impala-ASF-CR] IMPALA-4024: Add "system" database and expose Impala metrics as a table

2016-08-29 Thread Kathy Sun (Code Review)
Kathy Sun has uploaded a new patch set (#14). Change subject: IMPALA-4024: Add "system" database and expose Impala metrics as a table .. IMPALA-4024: Add "system" database and expose Impala metrics as a table This is to expose