You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Deepak Jaiswal <dj...@hortonworks.com> on 2018/06/15 03:33:37 UTC
Review Request 67612: HIVE-19904 : Load data rewrite into Tez job
fails for ACID
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67612/
-----------------------------------------------------------
Review request for hive, Eugene Koifman and Sankar Hariappan.
Bugs: HIVE-19904
https://issues.apache.org/jira/browse/HIVE-19904
Repository: hive-git
Description
-------
Load data rewrite into IAS fails for ACID as there is some code which does not take into account the table name could be in upper case, specifically ValidTxnWriteIdList
Diffs
-----
ql/src/test/org/apache/hadoop/hive/ql/TestTxnLoadData.java fb88f25285
storage-api/src/java/org/apache/hadoop/hive/common/ValidTxnWriteIdList.java 9867a81a7b
Diff: https://reviews.apache.org/r/67612/diff/1/
Testing
-------
Thanks,
Deepak Jaiswal
Re: Review Request 67612: HIVE-19904 : Load data rewrite into Tez job
fails for ACID
Posted by Deepak Jaiswal <dj...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67612/
-----------------------------------------------------------
(Updated June 15, 2018, 9:46 p.m.)
Review request for hive, Eugene Koifman and Sankar Hariappan.
Changes
-------
Use lower case for temp table name prefix.
Bugs: HIVE-19904
https://issues.apache.org/jira/browse/HIVE-19904
Repository: hive-git
Description
-------
Load data rewrite into IAS fails for ACID as there is some code which does not take into account the table name could be in upper case, specifically ValidTxnWriteIdList
Diffs (updated)
-----
ql/src/java/org/apache/hadoop/hive/ql/parse/LoadSemanticAnalyzer.java 189975ed2e
ql/src/test/org/apache/hadoop/hive/ql/TestTxnLoadData.java fb88f25285
ql/src/test/results/clientpositive/llap/load_data_using_job.q.out 7a62be23dd
Diff: https://reviews.apache.org/r/67612/diff/2/
Changes: https://reviews.apache.org/r/67612/diff/1-2/
Testing
-------
Thanks,
Deepak Jaiswal