mirror of
https://github.com/apache/impala.git
synced 2026-01-05 12:01:11 -05:00
IMPALA-4862: make resource profile consistent with backend behaviour
This moves away from the PipelinedPlanNodeSet approach of enumerating sets of concurrently-executing nodes because unions would force creating many overlapping sets of nodes. The new approach computes the peak resources during Open() and the peak resources between Open() and Close() (i.e. while calling GetNext()) bottom-up for each plan node in a fragment. The fragment resources are then combined to produce the query resources. The basic assumptions for the new resource estimates are: * resources are acquired during or after the first call to Open() and released in Close(). * Blocking nodes call Open() on their child before acquiring their own resources (this required some backend changes). * Blocking nodes call Close() on their children before returning from Open(). * The peak resource consumption of the query is the sum of the independent fragments (except for the parallel join build plans where we can assume there will be synchronisation). This is conservative but we don't synchronise fragment Open() and Close() across exchanges so can't make stronger assumptions in general. Also compute the sum of minimum reservations. This will be useful in the backend to determine exactly when all of the initial reservations have been claimed from a shared pool of initial reservations. Testing: * Updated planner tests to reflect behavioural changes. * Added extra resource requirement planner tests for unions, subplans, pipelines of blocking operators, and bushy join plans. * Added single-node plans to resource-requirements tests. These have more complex plan trees inside a single fragment, which is useful for testing the peak resource requirement logic. Change-Id: I492cf5052bb27e4e335395e2a8f8a3b07248ec9d Reviewed-on: http://gerrit.cloudera.org:8080/7223 Reviewed-by: Tim Armstrong <tarmstrong@cloudera.com> Tested-by: Impala Public Jenkins
This commit is contained in:
committed by
Impala Public Jenkins
parent
b329e62087
commit
64fd0115e5
@@ -6,9 +6,10 @@ select *
|
||||
from tpch.lineitem join tpch.orders on l_orderkey = o_orderkey;
|
||||
---- RESULTS: VERIFY_IS_EQUAL
|
||||
'Per-Host Resource Reservation: Memory=136.00MB'
|
||||
'Per-Host Resource Estimates: Memory=388.41MB'
|
||||
'Per-Host Resource Estimates: Memory=476.41MB'
|
||||
''
|
||||
'F02:PLAN FRAGMENT [UNPARTITIONED] hosts=1 instances=1'
|
||||
'| Per-Host Resources: mem-estimate=0B mem-reservation=0B'
|
||||
'PLAN-ROOT SINK'
|
||||
'| mem-estimate=0B mem-reservation=0B'
|
||||
'|'
|
||||
@@ -17,6 +18,7 @@ from tpch.lineitem join tpch.orders on l_orderkey = o_orderkey;
|
||||
'| tuple-ids=0,1 row-size=454B cardinality=5757710'
|
||||
'|'
|
||||
'F00:PLAN FRAGMENT [RANDOM] hosts=3 instances=3'
|
||||
'Per-Host Resources: mem-estimate=388.41MB mem-reservation=136.00MB'
|
||||
'02:HASH JOIN [INNER JOIN, BROADCAST]'
|
||||
'| hash predicates: l_orderkey = o_orderkey'
|
||||
'| fk/pk conjuncts: l_orderkey = o_orderkey'
|
||||
@@ -29,6 +31,7 @@ from tpch.lineitem join tpch.orders on l_orderkey = o_orderkey;
|
||||
'| | tuple-ids=1 row-size=191B cardinality=1500000'
|
||||
'| |'
|
||||
'| F01:PLAN FRAGMENT [RANDOM] hosts=2 instances=2'
|
||||
'| Per-Host Resources: mem-estimate=88.00MB mem-reservation=0B'
|
||||
'| 01:SCAN HDFS [tpch.orders, RANDOM]'
|
||||
row_regex:.*partitions=1/1 files=1 size=.*
|
||||
'| stats-rows=1500000 extrapolated-rows=disabled'
|
||||
@@ -54,7 +57,7 @@ from functional_avro.alltypes t1
|
||||
left outer join functional_avro.alltypes t3 on (t2.id = t3.id)
|
||||
where t1.month = 1 and t2.year = 2009 and t3.bool_col = false
|
||||
---- RESULTS: VERIFY_IS_SUBSET
|
||||
'Per-Host Resource Estimates: Memory=4.03GB'
|
||||
'Per-Host Resource Estimates: Memory=4.07GB'
|
||||
'WARNING: The following tables are missing relevant table and/or column statistics.'
|
||||
'functional_avro.alltypes, functional_parquet.alltypessmall'
|
||||
====
|
||||
|
||||
Reference in New Issue
Block a user