Hello Csaba Ringhofer, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16472
to look at the new patch set (#2).
Change subject: IMPALA-9652: CTAS doesn't respect transactional properties
......................................................................
IMPALA-9652: CTAS doesn't respect transactional properties
Because of a bug the INSERT part of a CTAS statement didn't
run in a transaction and it just put the new files under the
root directory of the table. This didn't cause too much problems
because there couldn't be any concurrent operations as the table was
under construction. However, this behavior is not working particularly
well in the context of replication, as the notification event needs
a transaction id.
With this fix the INSERT operation runs in a transaction and the new
files are created under a delta directory.
Testing:
* Added CTAS statements and SHOW FILES <tbl> queries to acid-insert.test
Check if the files are created in a delta directory, if so, then
the INSERT must have been running in a transaction.
Change-Id: I6ed96aeadbcead9fdc548da5922a066460ff9f77
---
M fe/src/main/java/org/apache/impala/service/Frontend.java
M testdata/workloads/functional-query/queries/QueryTest/acid-insert.test
2 files changed, 56 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/72/16472/2
--
To view, visit http://gerrit.cloudera.org:8080/16472
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I6ed96aeadbcead9fdc548da5922a066460ff9f77
Gerrit-Change-Number: 16472
Gerrit-PatchSet: 2
Gerrit-Owner: Zoltan Borok-Nagy <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
