diff options
author | Reynold Xin <rxin@databricks.com> | 2016-07-12 10:07:23 -0700 |
---|---|---|
committer | Reynold Xin <rxin@databricks.com> | 2016-07-12 10:07:23 -0700 |
commit | c377e49e38a290e5c4fbc178278069788674dfb7 (patch) | |
tree | ef043d59d8ab9eb0b778fe6a703cf73bfee6cfaf /yarn | |
parent | 5ad68ba5ce625c7005b540ca50ed001ca18de967 (diff) | |
download | spark-c377e49e38a290e5c4fbc178278069788674dfb7.tar.gz spark-c377e49e38a290e5c4fbc178278069788674dfb7.tar.bz2 spark-c377e49e38a290e5c4fbc178278069788674dfb7.zip |
[SPARK-16489][SQL] Guard against variable reuse mistakes in expression code generation
## What changes were proposed in this pull request?
In code generation, it is incorrect for expressions to reuse variable names across different instances of itself. As an example, SPARK-16488 reports a bug in which pmod expression reuses variable name "r".
This patch updates ExpressionEvalHelper test harness to always project two instances of the same expression, which will help us catch variable reuse problems in expression unit tests. This patch also fixes the bug in crc32 expression.
## How was this patch tested?
This is a test harness change, but I also created a new test suite for testing the test harness.
Author: Reynold Xin <rxin@databricks.com>
Closes #14146 from rxin/SPARK-16489.
Diffstat (limited to 'yarn')
0 files changed, 0 insertions, 0 deletions