Impala Public Jenkins has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/13736 )
Change subject: IMPALA-8681: Only show ValidWriteIdLists for Acid tables
......................................................................
IMPALA-8681: Only show ValidWriteIdLists for Acid tables
Lists ValidWriteIds for transactional tables in profile.
If a query does not trigger any transactional table loading,
the query profile will not have the "Loaded ValidWriteIdLists"
timeline.
Tests:
Manual tests.
Fixed StmtMetadataLoaderTest.
Added acid_profile test
Sample output:
Query Compilation: 3s525ms
- Metadata load started: 37.369ms (37.369ms)
- Metadata load finished. loaded-tables=1/1 ...
- Loaded ValidWriteIdLists for transactional tables:
functional.insert_only_transactional_table:0:9223372036854775807::
: 3s312ms (551.463us)
- Analysis finished: 3s370ms (58.110ms)
...
Change-Id: Ifcc31c7ddcfc471b0e5308f7e4aaadfa8189a905
Reviewed-on: http://gerrit.cloudera.org:8080/13736
Reviewed-by: Csaba Ringhofer <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
M fe/src/main/java/org/apache/impala/analysis/StmtMetadataLoader.java
M fe/src/test/java/org/apache/impala/analysis/StmtMetadataLoaderTest.java
A testdata/workloads/functional-query/queries/QueryTest/acid-profile.test
M tests/common/skip.py
M tests/query_test/test_acid.py
5 files changed, 46 insertions(+), 10 deletions(-)
Approvals:
Csaba Ringhofer: Looks good to me, approved
Impala Public Jenkins: Verified
--
To view, visit http://gerrit.cloudera.org:8080/13736
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ifcc31c7ddcfc471b0e5308f7e4aaadfa8189a905
Gerrit-Change-Number: 13736
Gerrit-PatchSet: 5
Gerrit-Owner: Yongzhi Chen <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Yongzhi Chen <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
