Skip to content

Conversation

@viirya
Copy link
Member

@viirya viirya commented May 6, 2020

What changes were proposed in this pull request?

This is a followup to address the #28366 (comment) by refining the SQL config document.

Why are the changes needed?

Make developers less confusing.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Only doc change.

@viirya
Copy link
Member Author

viirya commented May 6, 2020

cc @HyukjinKwon

@maropu maropu changed the title [SPARK-31365][SQL][Followup] Refine config document [SPARK-31365][SQL][FOLLOWUP] Refine config document for nested predicate pushdown May 7, 2020
@HyukjinKwon
Copy link
Member

Thanks @viirya.

@maropu maropu closed this in 9bf7387 May 7, 2020
maropu pushed a commit that referenced this pull request May 7, 2020
…ate pushdown

### What changes were proposed in this pull request?

This is a followup to address the #28366 (comment) by refining the SQL config document.

### Why are the changes needed?

Make developers less confusing.

### Does this PR introduce _any_ user-facing change?

No

### How was this patch tested?

Only doc change.

Closes #28468 from viirya/SPARK-31365-followup.

Authored-by: Liang-Chi Hsieh <[email protected]>
Signed-off-by: Takeshi Yamamuro <[email protected]>
(cherry picked from commit 9bf7387)
Signed-off-by: Takeshi Yamamuro <[email protected]>
@maropu
Copy link
Member

maropu commented May 7, 2020

Merged to master/3.0.

@viirya
Copy link
Member Author

viirya commented May 7, 2020

Thanks @HyukjinKwon @maropu

@SparkQA
Copy link

SparkQA commented May 7, 2020

Test build #122380 has finished for PR 28468 at commit 196d4c9.

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

@viirya viirya deleted the SPARK-31365-followup branch December 27, 2023 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants