[ 
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)

Reply via email to