Skip to content

[SPARK-27288][SQL] Pruning nested field in complex map key from object serializers#24220

Closed
viirya wants to merge 1 commit intoapache:masterfrom
viirya:SPARK-26847-followup
Closed

[SPARK-27288][SQL] Pruning nested field in complex map key from object serializers#24220
viirya wants to merge 1 commit intoapache:masterfrom
viirya:SPARK-26847-followup

Conversation

@viirya
Copy link
Member

@viirya viirya commented Mar 26, 2019

What changes were proposed in this pull request?

In the original PR #24158, pruning nested field in complex map key was not supported, because some methods in schema pruning did't support it at that moment. This is a followup to add it.

How was this patch tested?

Added tests.

@viirya
Copy link
Member Author

viirya commented Mar 26, 2019

cc @dongjoon-hyun @maropu

@SparkQA
Copy link

SparkQA commented Mar 26, 2019

Test build #103970 has finished for PR 24220 at commit 688fced.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@dongjoon-hyun
Copy link
Member

In this case, I'd like to recommend to have a new JIRA issue.

@dongjoon-hyun
Copy link
Member

cc @dbtsai

@viirya viirya changed the title [SPARK-26847][SQL][FOLLOWUP] Pruning nested field in complex map key [SPARK-27288][SQL] Pruning nested field in complex map key from object serializers Mar 27, 2019
@viirya
Copy link
Member Author

viirya commented Mar 27, 2019

@dongjoon-hyun Thanks! Created a new JIRA for this.

@dbtsai
Copy link
Member

dbtsai commented Mar 27, 2019

LGTM.

@maropu
Copy link
Member

maropu commented Mar 27, 2019

Thanks! Merged to master.

@maropu maropu closed this Mar 27, 2019
maropu pushed a commit that referenced this pull request Mar 27, 2019
…t serializers

## What changes were proposed in this pull request?

In the original PR #24158, pruning nested field in complex map key was not supported, because some methods in schema pruning did't support it at that moment. This is a followup to add it.

## How was this patch tested?

Added tests.

Closes #24220 from viirya/SPARK-26847-followup.

Authored-by: Liang-Chi Hsieh <viirya@gmail.com>
Signed-off-by: Takeshi Yamamuro <yamamuro@apache.org>
@viirya viirya deleted the SPARK-26847-followup branch December 27, 2023 18:22
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.

5 participants