[ https://issues.apache.org/jira/browse/HIVE-20801?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17021517#comment-17021517 ]
Hive QA commented on HIVE-20801: -------------------------------- Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/12976705/HIVE-20801.3.patch {color:red}ERROR:{color} -1 due to build exiting with an error Test results: https://builds.apache.org/job/PreCommit-HIVE-Build/20280/testReport Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/20280/console Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-20280/ Messages: {noformat} Executing org.apache.hive.ptest.execution.TestCheckPhase Executing org.apache.hive.ptest.execution.PrepPhase Tests exited with: NonZeroExitCodeException Command 'bash /data/hiveptest/working/scratch/source-prep.sh' failed with exit status 1 and output '+ date '+%Y-%m-%d %T.%3N' 2020-01-22 21:28:52.830 + [[ -n /usr/lib/jvm/java-8-openjdk-amd64 ]] + export JAVA_HOME=/usr/lib/jvm/java-8-openjdk-amd64 + JAVA_HOME=/usr/lib/jvm/java-8-openjdk-amd64 + export PATH=/usr/lib/jvm/java-8-openjdk-amd64/bin/:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games + PATH=/usr/lib/jvm/java-8-openjdk-amd64/bin/:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games + export 'ANT_OPTS=-Xmx1g -XX:MaxPermSize=256m ' + ANT_OPTS='-Xmx1g -XX:MaxPermSize=256m ' + export 'MAVEN_OPTS=-Xmx1g ' + MAVEN_OPTS='-Xmx1g ' + cd /data/hiveptest/working/ + tee /data/hiveptest/logs/PreCommit-HIVE-Build-20280/source-prep.txt + [[ false == \t\r\u\e ]] + mkdir -p maven ivy + [[ git = \s\v\n ]] + [[ git = \g\i\t ]] + [[ -z master ]] + [[ -d apache-github-source-source ]] + [[ ! -d apache-github-source-source/.git ]] + [[ ! -d apache-github-source-source ]] + date '+%Y-%m-%d %T.%3N' 2020-01-22 21:28:52.833 + cd apache-github-source-source + git fetch origin + git reset --hard HEAD HEAD is now at 05cabc8 HIVE-22666: Introduce TopNKey operator for PTF Reduce Sink (Krisztian Kasa, reviewed by Jesus Camacho Rodriguez) + git clean -f -d + git checkout master Already on 'master' Your branch is up-to-date with 'origin/master'. + git reset --hard origin/master HEAD is now at 05cabc8 HIVE-22666: Introduce TopNKey operator for PTF Reduce Sink (Krisztian Kasa, reviewed by Jesus Camacho Rodriguez) + git merge --ff-only origin/master Already up-to-date. + date '+%Y-%m-%d %T.%3N' 2020-01-22 21:28:53.513 + rm -rf ../yetus_PreCommit-HIVE-Build-20280 + mkdir ../yetus_PreCommit-HIVE-Build-20280 + git gc + cp -R . ../yetus_PreCommit-HIVE-Build-20280 + mkdir /data/hiveptest/logs/PreCommit-HIVE-Build-20280/yetus + patchCommandPath=/data/hiveptest/working/scratch/smart-apply-patch.sh + patchFilePath=/data/hiveptest/working/scratch/build.patch + [[ -f /data/hiveptest/working/scratch/build.patch ]] + chmod +x /data/hiveptest/working/scratch/smart-apply-patch.sh + /data/hiveptest/working/scratch/smart-apply-patch.sh /data/hiveptest/working/scratch/build.patch Trying to apply the patch with -p0 error: patch failed: common/src/java/org/apache/hadoop/hive/conf/HiveConf.java:2561 Falling back to three-way merge... Applied patch to 'common/src/java/org/apache/hadoop/hive/conf/HiveConf.java' with conflicts. error: patch failed: ql/src/java/org/apache/hadoop/hive/ql/io/AcidUtils.java:2338 Falling back to three-way merge... Applied patch to 'ql/src/java/org/apache/hadoop/hive/ql/io/AcidUtils.java' cleanly. Going to apply patch with: git apply -p0 /data/hiveptest/working/scratch/build.patch:11: trailing whitespace. "unlocked partitition by partition, instead of being transactionally scoped.\n" + /data/hiveptest/working/scratch/build.patch:70: trailing whitespace. // Test non-strict read-locking mode - the read locks are only obtained for the ACID side error: patch failed: common/src/java/org/apache/hadoop/hive/conf/HiveConf.java:2561 Falling back to three-way merge... Applied patch to 'common/src/java/org/apache/hadoop/hive/conf/HiveConf.java' with conflicts. error: patch failed: ql/src/java/org/apache/hadoop/hive/ql/io/AcidUtils.java:2338 Falling back to three-way merge... Applied patch to 'ql/src/java/org/apache/hadoop/hive/ql/io/AcidUtils.java' cleanly. U common/src/java/org/apache/hadoop/hive/conf/HiveConf.java warning: 2 lines add whitespace errors. + result=1 + '[' 1 -ne 0 ']' + rm -rf yetus_PreCommit-HIVE-Build-20280 + exit 1 ' {noformat} This message is automatically generated. ATTACHMENT ID: 12976705 - PreCommit-HIVE-Build > ACID: Allow DbTxnManager to ignore non-ACID table locking > --------------------------------------------------------- > > Key: HIVE-20801 > URL: https://issues.apache.org/jira/browse/HIVE-20801 > Project: Hive > Issue Type: Bug > Components: Locking, Transactions > Affects Versions: 4.0.0 > Reporter: Gopal Vijayaraghavan > Assignee: Gopal Vijayaraghavan > Priority: Major > Labels: Branch3Candidate, TODOC > Attachments: HIVE-20801.1.patch, HIVE-20801.2.patch, > HIVE-20801.2.patch, HIVE-20801.3.patch, HIVE-20801.3.patch > > > Enabling ACIDv1 on a cluster produces a central locking bottleneck for all > table types, which is not always the intention. > The Hive locking for non-acid tables are advisory (i.e a client can > write/read without locking), which means that the implementation does not > offer strong consistency despite the lock manager consuming resources > centrally. > Disabling this lock acquisition would improve the performance of non-ACID > tables co-existing with a globally configured DbTxnManager implementation. -- This message was sent by Atlassian Jira (v8.3.4#803005)