[ https://issues.apache.org/jira/browse/HIVE-20113?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16564452#comment-16564452 ]
Hive QA commented on HIVE-20113: -------------------------------- Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/12933821/HIVE-20113.4.patch {color:red}ERROR:{color} -1 due to build exiting with an error Test results: https://builds.apache.org/job/PreCommit-HIVE-Build/12973/testReport Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/12973/console Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-12973/ Messages: {noformat} **** This message was trimmed, see log for full details **** + cd apache-github-source-source + git fetch origin + git reset --hard HEAD HEAD is now at 5b2124b HIVE-20244 : forward port HIVE-19704 to master (Sergey Shelukhin, reviewed by Prasanth Jayachandran) + git clean -f -d Removing standalone-metastore/metastore-server/src/gen/ + git checkout master Already on 'master' Your branch is up-to-date with 'origin/master'. + git reset --hard origin/master HEAD is now at 5b2124b HIVE-20244 : forward port HIVE-19704 to master (Sergey Shelukhin, reviewed by Prasanth Jayachandran) + git merge --ff-only origin/master Already up-to-date. + date '+%Y-%m-%d %T.%3N' 2018-07-31 22:46:15.399 + rm -rf ../yetus_PreCommit-HIVE-Build-12973 + mkdir ../yetus_PreCommit-HIVE-Build-12973 + git gc + cp -R . ../yetus_PreCommit-HIVE-Build-12973 + mkdir /data/hiveptest/logs/PreCommit-HIVE-Build-12973/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 error: a/ql/src/java/org/apache/hadoop/hive/ql/parse/GenTezUtils.java: does not exist in index error: a/ql/src/test/results/clientpositive/llap/acid_no_buckets.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/auto_smb_mapjoin_14.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/auto_sortmerge_join_9.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/bucket_map_join_tez2.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/cluster.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/column_access_stats.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/correlationoptimizer2.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/correlationoptimizer3.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/correlationoptimizer6.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/explainuser_1.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/llap_smb_ptf.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/materialized_view_create_rewrite_4.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/parallel.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/parallel_colstats.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/partition_shared_scan.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/subquery_in.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/subquery_multi.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/subquery_notin.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/subquery_scalar.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/subquery_select.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/subquery_views.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/tez_smb_reduce_side.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/tez_union_group_by.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/llap/vector_auto_smb_mapjoin_14.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query10.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query14.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query16.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query2.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query31.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query32.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query33.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query35.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query45.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query56.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query58.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query6.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query60.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query64.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query69.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query77.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query78.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query83.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query92.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query94.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query95.q.out: does not exist in index error: a/ql/src/test/results/clientpositive/perf/tez/query97.q.out: does not exist in index error: patch failed: ql/src/test/results/clientpositive/llap/subquery_notin.q.out:3223 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/llap/subquery_notin.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query10.q.out:248 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query10.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query14.q.out:596 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query14.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query16.q.out:108 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query16.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query2.q.out:162 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query2.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query31.q.out:154 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query31.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query32.q.out:60 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query32.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query35.q.out:244 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query35.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query45.q.out:117 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query45.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query60.q.out:227 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query60.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query64.q.out:248 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query64.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query69.q.out:139 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query69.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query77.q.out:342 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query77.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query78.q.out:218 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query78.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query92.q.out:61 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query92.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query94.q.out:104 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query94.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query95.q.out:73 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query95.q.out' with conflicts. Going to apply patch with: git apply -p1 /data/hiveptest/working/scratch/build.patch:71: trailing whitespace. if (reduceSinkOperator.getConf().isForwarding() && error: patch failed: ql/src/test/results/clientpositive/llap/subquery_notin.q.out:3223 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/llap/subquery_notin.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query10.q.out:248 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query10.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query14.q.out:596 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query14.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query16.q.out:108 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query16.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query2.q.out:162 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query2.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query31.q.out:154 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query31.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query32.q.out:60 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query32.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query35.q.out:244 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query35.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query45.q.out:117 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query45.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query60.q.out:227 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query60.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query64.q.out:248 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query64.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query69.q.out:139 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query69.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query77.q.out:342 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query77.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query78.q.out:218 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query78.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query92.q.out:61 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query92.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query94.q.out:104 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query94.q.out' with conflicts. error: patch failed: ql/src/test/results/clientpositive/perf/tez/query95.q.out:73 Falling back to three-way merge... Applied patch to 'ql/src/test/results/clientpositive/perf/tez/query95.q.out' with conflicts. U ql/src/test/results/clientpositive/llap/subquery_notin.q.out U ql/src/test/results/clientpositive/perf/tez/query10.q.out U ql/src/test/results/clientpositive/perf/tez/query14.q.out U ql/src/test/results/clientpositive/perf/tez/query16.q.out U ql/src/test/results/clientpositive/perf/tez/query2.q.out U ql/src/test/results/clientpositive/perf/tez/query31.q.out U ql/src/test/results/clientpositive/perf/tez/query32.q.out U ql/src/test/results/clientpositive/perf/tez/query35.q.out U ql/src/test/results/clientpositive/perf/tez/query45.q.out U ql/src/test/results/clientpositive/perf/tez/query60.q.out U ql/src/test/results/clientpositive/perf/tez/query64.q.out U ql/src/test/results/clientpositive/perf/tez/query69.q.out U ql/src/test/results/clientpositive/perf/tez/query77.q.out U ql/src/test/results/clientpositive/perf/tez/query78.q.out U ql/src/test/results/clientpositive/perf/tez/query92.q.out U ql/src/test/results/clientpositive/perf/tez/query94.q.out U ql/src/test/results/clientpositive/perf/tez/query95.q.out warning: 1 line adds whitespace errors. + result=1 + '[' 1 -ne 0 ']' + rm -rf yetus_PreCommit-HIVE-Build-12973 + exit 1 ' {noformat} This message is automatically generated. ATTACHMENT ID: 12933821 - PreCommit-HIVE-Build > Shuffle avoidance: Disable 1-1 edges for sorted shuffle > -------------------------------------------------------- > > Key: HIVE-20113 > URL: https://issues.apache.org/jira/browse/HIVE-20113 > Project: Hive > Issue Type: Bug > Components: Tez > Reporter: Gopal V > Assignee: Gopal V > Priority: Major > Labels: Branch3Candidate > Attachments: HIVE-20113.1.patch, HIVE-20113.2.patch, > HIVE-20113.3.patch, HIVE-20113.4.patch, HIVE-20113.4.patch > > > The sorted shuffle avoidance can have some issues when the shuffle data gets > broken up into multiple chunks on disk. > The 1-1 edge cannot skip the tez final merge - there's no reason for 1-1 to > have a final merge at all, it should open a single compressed file and write > a single index entry. > Until the shuffle issue is resolved & a lot more testing, it is prudent to > disable the optimization for sorted shuffle edges and stop rewriting the > RS(sorted) = = = RS(sorted) into RS(sorted) = = = RS(FORWARD). -- This message was sent by Atlassian JIRA (v7.6.3#76005)