mirror of https://github.com/citusdata/citus.git
0259815d3a
In #3901 the "Data received from worker(s)" sections were added to EXPLAIN ANALYZE. After merging @pykello posted some review comments. This addresses those comments as well as fixing a other issues that I found while addressing them. The things this does: 1. Fix `EXPLAIN ANALYZE EXECUTE p1` to not increase received data on every execution 2. Fix `EXPLAIN ANALYZE EXECUTE p1(1)` to not return 0 bytes as received data allways. 3. Move `EXPLAIN ANALYZE` specific logic to `multi_explain.c` from `adaptive_executor.c` 4. Change naming of new explain sections to `Tuple data received from node(s)`. Firstly because a task can reference the coordinator too, so "worker(s)" was incorrect. Secondly to indicate that this is tuple data and not all network traffic that was performed. 5. Rename `totalReceivedData` in our codebase to `totalReceivedTupleData` to make it clearer that it's a tuple data counter, not all network traffic. 6. Actually add `binary_protocol` test to `multi_schedule` (woops) 7. Fix a randomly failing test in `local_shard_execution.sql`. |
||
|---|---|---|
| .. | ||
| adaptive_executor.c | ||
| citus_custom_scan.c | ||
| directed_acyclic_graph_execution.c | ||
| distributed_execution_locks.c | ||
| distributed_intermediate_results.c | ||
| insert_select_executor.c | ||
| intermediate_results.c | ||
| local_executor.c | ||
| multi_client_executor.c | ||
| multi_executor.c | ||
| multi_server_executor.c | ||
| multi_task_tracker_executor.c | ||
| partitioned_intermediate_results.c | ||
| placement_access.c | ||
| query_stats.c | ||
| repartition_join_execution.c | ||
| subplan_execution.c | ||
| tuple_destination.c | ||