diff options
| author | Cheng Lian <lian@databricks.com> | 2015-03-24 01:12:11 -0700 |
|---|---|---|
| committer | Michael Armbrust <michael@databricks.com> | 2015-03-24 01:12:11 -0700 |
| commit | 1afcf773d0cafdfd9bf106fdc5c429ed2ba3dd36 (patch) | |
| tree | 5acfd6feaed6d9b977de1d2042367e11fea84cc8 /dev/run-tests-jenkins | |
| parent | 4ce2782a61e23ed0326faac2ee97a9bd36ec8963 (diff) | |
| download | spark-1afcf773d0cafdfd9bf106fdc5c429ed2ba3dd36.tar.gz spark-1afcf773d0cafdfd9bf106fdc5c429ed2ba3dd36.tar.bz2 spark-1afcf773d0cafdfd9bf106fdc5c429ed2ba3dd36.zip | |
[SPARK-6452] [SQL] Checks for missing attributes and unresolved operator for all types of operator
In `CheckAnalysis`, `Filter` and `Aggregate` are checked in separate case clauses, thus never hit those clauses for unresolved operators and missing input attributes.
This PR also removes the `prettyString` call when generating error message for missing input attributes. Because result of `prettyString` doesn't contain expression ID, and may give confusing messages like
> resolved attributes a missing from a
cc rxin
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/apache/spark/5129)
<!-- Reviewable:end -->
Author: Cheng Lian <lian@databricks.com>
Closes #5129 from liancheng/spark-6452 and squashes the following commits:
52cdc69 [Cheng Lian] Addresses comments
029f9bd [Cheng Lian] Checks for missing attributes and unresolved operator for all types of operator
Diffstat (limited to 'dev/run-tests-jenkins')
0 files changed, 0 insertions, 0 deletions