Skip to content

Conversation

@emkornfield
Copy link

What changes were proposed in this pull request?

Update documentation to reflect changes in faf220a

I've changed the documentation to reflect updated statistics may be used to improve query plan.

Why are the changes needed?

I believe the documentation is stale and misleading.

Does this PR introduce any user-facing change?

Yes, this is a javadoc documentation fix.

How was this patch tested?

Doc fix.

@emkornfield
Copy link
Author

CC @dongjoon-hyun @cloud-fan

@cloud-fan
Copy link
Contributor

ok to test

@SparkQA
Copy link

SparkQA commented Jul 1, 2020

Test build #124659 has finished for PR 28925 at commit 148a89f.

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

@cloud-fan
Copy link
Contributor

thanks, merging to master/3.0!

@cloud-fan cloud-fan closed this in a4ba344 Jul 1, 2020
cloud-fan pushed a commit that referenced this pull request Jul 1, 2020
…tatistics

### What changes were proposed in this pull request?
Update documentation to reflect changes in faf220a

I've changed the documentation to reflect updated statistics may be used to improve query plan.

### Why are the changes needed?
I believe the documentation is stale and misleading.

### Does this PR introduce _any_ user-facing change?
Yes, this is a javadoc documentation fix.

### How was this patch tested?
Doc fix.

Closes #28925 from emkornfield/spark-32095.

Authored-by: Micah Kornfield <[email protected]>
Signed-off-by: Wenchen Fan <[email protected]>
(cherry picked from commit a4ba344)
Signed-off-by: Wenchen Fan <[email protected]>
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.

3 participants