Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16472 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/16472 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- 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(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: I6ed96aeadbcead9fdc548da5922a066460ff9f77 Gerrit-Change-Number: 16472 Gerrit-PatchSet: 5 Gerrit-Owner: Zoltan Borok-Nagy <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
