[Impala-ASF-CR] (DO NOT MERGE) IMPALA-12737: List key columns in query history

2024-03-21 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/21142 )

Change subject: (DO NOT MERGE) IMPALA-12737: List key columns in query history
..


Patch Set 16:

Build Successful

https://jenkins.impala.io/job/gerrit-code-review-checks/15616/ : Initial code 
review checks passed. Use gerrit-verify-dryrun-external or gerrit-verify-dryrun 
to run full precommit tests.


--
To view, visit http://gerrit.cloudera.org:8080/21142
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I78f3670b067c0c192ee8a212fba95466fbcb51d7
Gerrit-Change-Number: 21142
Gerrit-PatchSet: 16
Gerrit-Owner: Michael Smith 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Smith 
Gerrit-Comment-Date: Thu, 21 Mar 2024 19:39:23 +
Gerrit-HasComments: No


[Impala-ASF-CR] (DO NOT MERGE) IMPALA-12737: List key columns in query history

2024-03-21 Thread Michael Smith (Code Review)
Hello Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/21142

to look at the new patch set (#16).

Change subject: (DO NOT MERGE) IMPALA-12737: List key columns in query history
..

(DO NOT MERGE) IMPALA-12737: List key columns in query history

Adds "Key Columns" to the query profile, enumerating a
comma-separated list of the canonical path of columns used in the query:

  Key Columns: functional.alltypes.month,functional.alltypes.year

Also adds "key_columns" to impala_query_log and impala_query_live with
the same content.

Requires 'drop table sys.impala_query_log' to recreate it with the new
column.

Change-Id: I78f3670b067c0c192ee8a212fba95466fbcb51d7
---
M be/src/exec/system-table-scanner.cc
M be/src/service/client-request-state.cc
M be/src/service/client-request-state.h
M be/src/service/query-state-record.cc
M be/src/service/query-state-record.h
M be/src/service/workload-management-fields.cc
M common/thrift/Frontend.thrift
M common/thrift/SystemTables.thrift
M fe/src/main/java/org/apache/impala/analysis/Analyzer.java
M fe/src/main/java/org/apache/impala/analysis/ModifyStmt.java
M fe/src/main/java/org/apache/impala/analysis/Path.java
M fe/src/main/java/org/apache/impala/analysis/SelectStmt.java
M fe/src/main/java/org/apache/impala/service/Frontend.java
M tests/util/workload_management.py
14 files changed, 105 insertions(+), 16 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/42/21142/16
--
To view, visit http://gerrit.cloudera.org:8080/21142
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I78f3670b067c0c192ee8a212fba95466fbcb51d7
Gerrit-Change-Number: 21142
Gerrit-PatchSet: 16
Gerrit-Owner: Michael Smith 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Smith 


[Impala-ASF-CR] (DO NOT MERGE) IMPALA-12737: List key columns in query history

2024-03-19 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/21142 )

Change subject: (DO NOT MERGE) IMPALA-12737: List key columns in query history
..


Patch Set 13:

Build Successful

https://jenkins.impala.io/job/gerrit-code-review-checks/15573/ : Initial code 
review checks passed. Use gerrit-verify-dryrun-external or gerrit-verify-dryrun 
to run full precommit tests.


--
To view, visit http://gerrit.cloudera.org:8080/21142
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I78f3670b067c0c192ee8a212fba95466fbcb51d7
Gerrit-Change-Number: 21142
Gerrit-PatchSet: 13
Gerrit-Owner: Michael Smith 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Smith 
Gerrit-Comment-Date: Tue, 19 Mar 2024 23:23:03 +
Gerrit-HasComments: No


[Impala-ASF-CR] (DO NOT MERGE) IMPALA-12737: List key columns in query history

2024-03-19 Thread Michael Smith (Code Review)
Hello Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/21142

to look at the new patch set (#13).

Change subject: (DO NOT MERGE) IMPALA-12737: List key columns in query history
..

(DO NOT MERGE) IMPALA-12737: List key columns in query history

Adds "Key Columns" to the query profile, enumerating a
comma-separated list of the canonical path of columns used in the query:

  Key Columns: functional.alltypes.month,functional.alltypes.year

Also adds "key_columns" to impala_query_log and impala_query_live with
the same content.

Requires 'drop table sys.impala_query_log' to recreate it with the new
column.

Change-Id: I78f3670b067c0c192ee8a212fba95466fbcb51d7
---
M be/src/exec/system-table-scanner.cc
M be/src/service/client-request-state.cc
M be/src/service/client-request-state.h
M be/src/service/query-state-record.cc
M be/src/service/query-state-record.h
M be/src/service/workload-management-fields.cc
M common/thrift/Frontend.thrift
M common/thrift/SystemTables.thrift
M fe/src/main/java/org/apache/impala/analysis/Analyzer.java
M fe/src/main/java/org/apache/impala/analysis/ModifyStmt.java
M fe/src/main/java/org/apache/impala/analysis/Path.java
M fe/src/main/java/org/apache/impala/analysis/SelectStmt.java
M fe/src/main/java/org/apache/impala/service/Frontend.java
M tests/util/workload_management.py
14 files changed, 105 insertions(+), 16 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/42/21142/13
--
To view, visit http://gerrit.cloudera.org:8080/21142
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I78f3670b067c0c192ee8a212fba95466fbcb51d7
Gerrit-Change-Number: 21142
Gerrit-PatchSet: 13
Gerrit-Owner: Michael Smith 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Smith 


[Impala-ASF-CR] (DO NOT MERGE) IMPALA-12737: List key columns in query history

2024-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/21142 )

Change subject: (DO NOT MERGE) IMPALA-12737: List key columns in query history
..


Patch Set 11:

Build Successful

https://jenkins.impala.io/job/gerrit-code-review-checks/15539/ : Initial code 
review checks passed. Use gerrit-verify-dryrun-external or gerrit-verify-dryrun 
to run full precommit tests.


--
To view, visit http://gerrit.cloudera.org:8080/21142
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I78f3670b067c0c192ee8a212fba95466fbcb51d7
Gerrit-Change-Number: 21142
Gerrit-PatchSet: 11
Gerrit-Owner: Michael Smith 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Smith 
Gerrit-Comment-Date: Fri, 15 Mar 2024 19:21:55 +
Gerrit-HasComments: No


[Impala-ASF-CR] (DO NOT MERGE) IMPALA-12737: List key columns in query history

2024-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/21142 )

Change subject: (DO NOT MERGE) IMPALA-12737: List key columns in query history
..


Patch Set 10:

Build Successful

https://jenkins.impala.io/job/gerrit-code-review-checks/15537/ : Initial code 
review checks passed. Use gerrit-verify-dryrun-external or gerrit-verify-dryrun 
to run full precommit tests.


--
To view, visit http://gerrit.cloudera.org:8080/21142
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I78f3670b067c0c192ee8a212fba95466fbcb51d7
Gerrit-Change-Number: 21142
Gerrit-PatchSet: 10
Gerrit-Owner: Michael Smith 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Smith 
Gerrit-Comment-Date: Fri, 15 Mar 2024 19:19:25 +
Gerrit-HasComments: No


[Impala-ASF-CR] (DO NOT MERGE) IMPALA-12737: List key columns in query history

2024-03-15 Thread Michael Smith (Code Review)
Hello Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/21142

to look at the new patch set (#11).

Change subject: (DO NOT MERGE) IMPALA-12737: List key columns in query history
..

(DO NOT MERGE) IMPALA-12737: List key columns in query history

Adds "Key Columns" to the query profile, enumerating a
comma-separated list of the canonical path of columns used in the query:

  Key Columns: functional.alltypes.month,functional.alltypes.year

Also adds "key_columns" to impala_query_log and impala_query_live with
the same content.

Requires 'drop table sys.impala_query_log' to recreate it with the new
column.

Change-Id: I78f3670b067c0c192ee8a212fba95466fbcb51d7
---
M be/src/exec/system-table-scanner.cc
M be/src/service/client-request-state.cc
M be/src/service/client-request-state.h
M be/src/service/query-state-record.cc
M be/src/service/query-state-record.h
M be/src/service/workload-management-fields.cc
M common/thrift/Frontend.thrift
M common/thrift/SystemTables.thrift
M fe/src/main/java/org/apache/impala/analysis/Analyzer.java
M fe/src/main/java/org/apache/impala/analysis/ModifyStmt.java
M fe/src/main/java/org/apache/impala/analysis/Path.java
M fe/src/main/java/org/apache/impala/analysis/SelectStmt.java
M fe/src/main/java/org/apache/impala/service/Frontend.java
M tests/util/workload_management.py
14 files changed, 103 insertions(+), 16 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/42/21142/11
--
To view, visit http://gerrit.cloudera.org:8080/21142
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I78f3670b067c0c192ee8a212fba95466fbcb51d7
Gerrit-Change-Number: 21142
Gerrit-PatchSet: 11
Gerrit-Owner: Michael Smith 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Smith 


[Impala-ASF-CR] (DO NOT MERGE) IMPALA-12737: List key columns in query history

2024-03-15 Thread Michael Smith (Code Review)
Hello Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/21142

to look at the new patch set (#10).

Change subject: (DO NOT MERGE) IMPALA-12737: List key columns in query history
..

(DO NOT MERGE) IMPALA-12737: List key columns in query history

Adds "Key Columns" to the query profile, enumerating a
comma-separated list of the canonical path of columns used in the query:

  Key Columns: functional.alltypes.month,functional.alltypes.year

Also adds "key_columns" to impala_query_log and impala_query_live with
the same content.

Requires 'drop table sys.impala_query_log' to recreate it with the new
column.

Change-Id: I78f3670b067c0c192ee8a212fba95466fbcb51d7
---
M be/src/exec/system-table-scanner.cc
M be/src/service/client-request-state.cc
M be/src/service/client-request-state.h
M be/src/service/query-state-record.cc
M be/src/service/query-state-record.h
M be/src/service/workload-management-fields.cc
M common/thrift/Frontend.thrift
M common/thrift/SystemTables.thrift
M fe/src/main/java/org/apache/impala/analysis/Analyzer.java
M fe/src/main/java/org/apache/impala/analysis/ModifyStmt.java
M fe/src/main/java/org/apache/impala/analysis/Path.java
M fe/src/main/java/org/apache/impala/analysis/SelectStmt.java
M fe/src/main/java/org/apache/impala/service/Frontend.java
M tests/util/workload_management.py
14 files changed, 103 insertions(+), 16 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/42/21142/10
--
To view, visit http://gerrit.cloudera.org:8080/21142
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I78f3670b067c0c192ee8a212fba95466fbcb51d7
Gerrit-Change-Number: 21142
Gerrit-PatchSet: 10
Gerrit-Owner: Michael Smith 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Smith 


[Impala-ASF-CR] (DO NOT MERGE) IMPALA-12737: List key columns in query history

2024-03-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/21142 )

Change subject: (DO NOT MERGE) IMPALA-12737: List key columns in query history
..


Patch Set 9:

Build Successful

https://jenkins.impala.io/job/gerrit-code-review-checks/15535/ : Initial code 
review checks passed. Use gerrit-verify-dryrun-external or gerrit-verify-dryrun 
to run full precommit tests.


--
To view, visit http://gerrit.cloudera.org:8080/21142
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I78f3670b067c0c192ee8a212fba95466fbcb51d7
Gerrit-Change-Number: 21142
Gerrit-PatchSet: 9
Gerrit-Owner: Michael Smith 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Smith 
Gerrit-Comment-Date: Fri, 15 Mar 2024 18:41:45 +
Gerrit-HasComments: No


[Impala-ASF-CR] (DO NOT MERGE) IMPALA-12737: List key columns in query history

2024-03-15 Thread Michael Smith (Code Review)
Michael Smith has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/21142 )

Change subject: (DO NOT MERGE) IMPALA-12737: List key columns in query history
..


Patch Set 9:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/21142/9/common/thrift/Frontend.thrift
File common/thrift/Frontend.thrift:

http://gerrit.cloudera.org:8080/#/c/21142/9/common/thrift/Frontend.thrift@684
PS9, Line 684:   21: optional list key_columns
Needs comment.



--
To view, visit http://gerrit.cloudera.org:8080/21142
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I78f3670b067c0c192ee8a212fba95466fbcb51d7
Gerrit-Change-Number: 21142
Gerrit-PatchSet: 9
Gerrit-Owner: Michael Smith 
Gerrit-Reviewer: Impala Public Jenkins 
Gerrit-Reviewer: Michael Smith 
Gerrit-Comment-Date: Fri, 15 Mar 2024 18:16:06 +
Gerrit-HasComments: Yes


[Impala-ASF-CR] (DO NOT MERGE) IMPALA-12737: List key columns in query history

2024-03-15 Thread Michael Smith (Code Review)
Hello Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/21142

to look at the new patch set (#8).

Change subject: (DO NOT MERGE) IMPALA-12737: List key columns in query history
..

(DO NOT MERGE) IMPALA-12737: List key columns in query history

Adds "Key Columns" to the query profile, enumerating a
comma-separated list of the canonical path of columns used in the query:

  Key Columns: functional.alltypes.month,functional.alltypes.year

Also adds "key_columns" to impala_query_log and impala_query_live with
the same content.

Requires 'drop table sys.impala_query_log' to recreate it with the new
column.

Change-Id: I78f3670b067c0c192ee8a212fba95466fbcb51d7
---
M be/src/exec/system-table-scanner.cc
M be/src/service/client-request-state.cc
M be/src/service/client-request-state.h
M be/src/service/query-state-record.cc
M be/src/service/query-state-record.h
M be/src/service/workload-management-fields.cc
M common/thrift/Frontend.thrift
M common/thrift/SystemTables.thrift
M fe/src/main/java/org/apache/impala/analysis/Analyzer.java
M fe/src/main/java/org/apache/impala/analysis/ModifyStmt.java
M fe/src/main/java/org/apache/impala/analysis/Path.java
M fe/src/main/java/org/apache/impala/analysis/SelectStmt.java
M fe/src/main/java/org/apache/impala/service/Frontend.java
M tests/util/workload_management.py
14 files changed, 98 insertions(+), 16 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/42/21142/8
--
To view, visit http://gerrit.cloudera.org:8080/21142
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I78f3670b067c0c192ee8a212fba95466fbcb51d7
Gerrit-Change-Number: 21142
Gerrit-PatchSet: 8
Gerrit-Owner: Michael Smith 
Gerrit-Reviewer: Impala Public Jenkins