Skip to content

Conversation

@marmbrus
Copy link
Contributor

We need to use the analyzed attributes otherwise we end up with a tree that will never resolve.

@SparkQA
Copy link

SparkQA commented Jul 17, 2014

QA tests have started for PR 1470. This patch merges cleanly.
View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16790/consoleFull

@SparkQA
Copy link

SparkQA commented Jul 17, 2014

QA results for PR 1470:
- This patch PASSES unit tests.
- This patch merges cleanly
- This patch adds no public classes

For more information see test ouptut:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16790/consoleFull

@concretevitamin
Copy link
Contributor

LGTM.

@marmbrus
Copy link
Contributor Author

Thanks for reviewing. Merged to master and 1.0

asfgit pushed a commit that referenced this pull request Jul 22, 2014
We need to use the analyzed attributes otherwise we end up with a tree that will never resolve.

Author: Michael Armbrust <[email protected]>

Closes #1470 from marmbrus/fixApplySchema and squashes the following commits:

f968195 [Michael Armbrust] Use analyzed attributes when applying the schema.
4969015 [Michael Armbrust] Add test case.

(cherry picked from commit 511a731)
Signed-off-by: Michael Armbrust <[email protected]>
@asfgit asfgit closed this in 511a731 Jul 22, 2014
@marmbrus marmbrus deleted the fixApplySchema branch August 27, 2014 20:44
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
We need to use the analyzed attributes otherwise we end up with a tree that will never resolve.

Author: Michael Armbrust <[email protected]>

Closes apache#1470 from marmbrus/fixApplySchema and squashes the following commits:

f968195 [Michael Armbrust] Use analyzed attributes when applying the schema.
4969015 [Michael Armbrust] Add test case.
sunchao pushed a commit to sunchao/spark that referenced this pull request Jun 2, 2023
…#1470)

Updates parquet to include fix for rdar://95861938
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants