Yongzhi Chen has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13334 )

Change subject: acid: Filter unwanted files based on ACID state.
......................................................................


Patch Set 16:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/13334/16/fe/src/main/java/org/apache/impala/util/AcidUtils.java
File fe/src/main/java/org/apache/impala/util/AcidUtils.java:

http://gerrit.cloudera.org:8080/#/c/13334/16/fe/src/main/java/org/apache/impala/util/AcidUtils.java@96
PS16, Line 96:   /**
Should we also have a predicate based on writeID stored in each partition?


http://gerrit.cloudera.org:8080/#/c/13334/16/tests/common/impala_test_suite.py
File tests/common/impala_test_suite.py:

http://gerrit.cloudera.org:8080/#/c/13334/16/tests/common/impala_test_suite.py@507
PS16, Line 507:           result = h.execute(query, user=user)
We only need return the last query result of the block?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Icf0aeb36e10c827ead59ed7f67e731199394fe8e
Gerrit-Change-Number: 13334
Gerrit-PatchSet: 16
Gerrit-Owner: Sudhanshu Arora <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Sudhanshu Arora <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Vihang Karajgaonkar <[email protected]>
Gerrit-Reviewer: Yongzhi Chen <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Comment-Date: Thu, 23 May 2019 14:51:12 +0000
Gerrit-HasComments: Yes

Reply via email to