kosiew commented on code in PR #24055:
URL: https://github.com/apache/datafusion/pull/24055#discussion_r3782783552
##########
docs/source/user-guide/metrics.md:
##########
@@ -42,6 +42,25 @@ DataFusion operators expose runtime metrics so you can
understand where time is
| ----------- |
----------------------------------------------------------------- |
| selectivity | Selectivity of the filter, calculated as output_rows /
input_rows |
+### HashJoinExec
+
+`HashJoinExec` also exposes the common `BaselineMetrics`. Its
+`elapsed_compute` metric is the sum of the build-side collection time and the
+subsequent join processing time.
+
+| Metric | Description
|
+| ----------------------- |
---------------------------------------------------------------------------------------------------------
|
+| build_time | Total time spent collecting and building the build
side of the join. |
+| build_input_batches | Number of input batches consumed from the build
side. |
+| build_input_rows | Number of input rows consumed from the build side.
|
+| build_mem_used | Peak memory used by the build side, in bytes.
|
Review Comment:
Could we say "Peak tracked memory used by the build side" instead of "Peak
memory used" here? `build_mem_used` is based on accounting from RecordBatch
sizes, ArrayMap and hash-table estimates, and the visited bitmap. That makes it
useful for the operator's reservation and accounting, but it is not an
allocator-measured process-memory peak.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]