Skip to content

[MINOR][DOC] Add class document for PruneFileSourcePartitions and PruneHiveTablePartitions - #27535

Closed
fuwhu wants to merge 2 commits into
apache:masterfrom
fuwhu:SPARK-15616-FOLLOW-UP
Closed

[MINOR][DOC] Add class document for PruneFileSourcePartitions and PruneHiveTablePartitions#27535
fuwhu wants to merge 2 commits into
apache:masterfrom
fuwhu:SPARK-15616-FOLLOW-UP

Conversation

@fuwhu

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Add class document for PruneFileSourcePartitions and PruneHiveTablePartitions.

Why are the changes needed?

To describe these two classes.

Does this PR introduce any user-facing change?

no

How was this patch tested?

no


/**
* Prune the partitions of file source based table using partition filters. Currently, this rule
* is applied to [[HadoopFsRelation]] and [[DataSourceV2ScanRelation]].

@cloud-fancloud-fanFeb 11, 2020

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DataSourceV2ScanRelation -> DataSourceV2ScanRelation with FileScan

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cloud-fan updated as :

DataSourceV2ScanRelation -> DataSourceV2ScanRelation with FileScan
HadoopFsRelation -> HadoopFsRelation with CatalogFileIndex

@cloud-fancloud-fan changed the title Add class document for PruneFileSourcePartitions and PruneHiveTablePartitions[MINOR][DOC] Add class document for PruneFileSourcePartitions and PruneHiveTablePartitionsFeb 11, 2020
@SparkQA

Copy link
Copy Markdown

Test build #118194 has finished for PR 27535 at commit 23813ec.

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

@SparkQA

Copy link
Copy Markdown

Test build #118208 has finished for PR 27535 at commit 5274b1a.

  • This patch fails due to an unknown error code, -9.
  • This patch merges cleanly.
  • This patch adds no public classes.

@cloud-fan

Copy link
Copy Markdown
Contributor

retest this please

@SparkQA

Copy link
Copy Markdown

Test build #118225 has finished for PR 27535 at commit 5274b1a.

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

@cloud-fan

Copy link
Copy Markdown
Contributor

thanks, merging to master/3.0!

cloud-fan pushed a commit that referenced this pull request Feb 11, 2020
…neHiveTablePartitions
### What changes were proposed in this pull request?
Add class document for PruneFileSourcePartitions and PruneHiveTablePartitions.
### Why are the changes needed?
To describe these two classes.
### Does this PR introduce any user-facing change?
no
### How was this patch tested?
no
Closes#27535 from fuwhu/SPARK-15616-FOLLOW-UP.
Authored-by: fuwhu <bestwwg@163.com>
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
(cherry picked from commit f1d0dce)
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
@fuwhu
fuwhu deleted the SPARK-15616-FOLLOW-UP branch February 11, 2020 22:32
@fuwhu

Copy link
Copy Markdown
ContributorAuthor

thank you all for help.

sjincho pushed a commit to sjincho/spark that referenced this pull request Apr 15, 2020
…neHiveTablePartitions
### What changes were proposed in this pull request?
Add class document for PruneFileSourcePartitions and PruneHiveTablePartitions.
### Why are the changes needed?
To describe these two classes.
### Does this PR introduce any user-facing change?
no
### How was this patch tested?
no
Closesapache#27535 from fuwhu/SPARK-15616-FOLLOW-UP.
Authored-by: fuwhu <bestwwg@163.com>
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
Sign up for freeto 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

@fuwhu@SparkQA@cloud-fan