diff options
| author | Ala Luszczak <ala@databricks.com> | 2017-02-07 14:21:30 +0100 |
|---|---|---|
| committer | Reynold Xin <rxin@databricks.com> | 2017-02-07 14:21:30 +0100 |
| commit | 6ed285c68fee451c45db7b01ca8ec1dea2efd479 (patch) | |
| tree | 341cff8e7b21f695ccb90c825440bd82b15210ea /R | |
| parent | e99e34d0f370211a7c7b96d144cc932b2fc71d10 (diff) | |
| download | spark-6ed285c68fee451c45db7b01ca8ec1dea2efd479.tar.gz spark-6ed285c68fee451c45db7b01ca8ec1dea2efd479.tar.bz2 spark-6ed285c68fee451c45db7b01ca8ec1dea2efd479.zip | |
[SPARK-19447] Fixing input metrics for range operator.
## What changes were proposed in this pull request?
This change introduces a new metric "number of generated rows". It is used exclusively for Range, which is a leaf in the query tree, yet doesn't read any input data, and therefore cannot report "recordsRead".
Additionally the way in which the metrics are reported by the JIT-compiled version of Range was changed. Previously, it was immediately reported that all the records were produced. This could be confusing for a user monitoring execution progress in the UI. Now, the metric is updated gradually.
In order to avoid negative impact on Range performance, the code generation was reworked. The values are now produced in batches in the tighter inner loop, while the metrics are updated in the outer loop.
The change also contains a number of unit tests, which should help ensure the correctness of metrics for various input sources.
## How was this patch tested?
Unit tests.
Author: Ala Luszczak <ala@databricks.com>
Closes #16829 from ala/SPARK-19447.
Diffstat (limited to 'R')
0 files changed, 0 insertions, 0 deletions