Hello Zoltan Borok-Nagy, Todd Lipcon, Csaba Ringhofer, Impala Public Jenkins,

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

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

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

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
---
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, 42 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/36/13736/3
--
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: newpatchset
Gerrit-Change-Id: Ifcc31c7ddcfc471b0e5308f7e4aaadfa8189a905
Gerrit-Change-Number: 13736
Gerrit-PatchSet: 3
Gerrit-Owner: Yongzhi Chen <yc...@cloudera.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Yongzhi Chen <yc...@cloudera.com>
Gerrit-Reviewer: Zoltan Borok-Nagy <borokna...@cloudera.com>

Reply via email to