This is an automated email from the ASF dual-hosted git repository.

huaxingao pushed a commit to branch branch-3.2
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.2 by this push:
     new af569d1  [MINOR][SQL][DOCS] Correct the 'options' description on 
UnresolvedRelation
af569d1 is described below

commit af569d1b0ac6b25dbd500804a395964ef7f9e60f
Author: Hyukjin Kwon <gurwls...@apache.org>
AuthorDate: Wed Sep 22 23:00:15 2021 -0700

    [MINOR][SQL][DOCS] Correct the 'options' description on UnresolvedRelation
    
    ### What changes were proposed in this pull request?
    
    This PR fixes the 'options' description on `UnresolvedRelation`. This 
comment was added in https://github.com/apache/spark/pull/29535 but not valid 
anymore because V1 also uses this `options` (and merge the options with the 
table properties) per https://github.com/apache/spark/pull/29712.
    
    This PR can go through from `master` to `branch-3.1`.
    
    ### Why are the changes needed?
    
    To make `UnresolvedRelation.options`'s description clearer.
    
    ### Does this PR introduce _any_ user-facing change?
    
    No, dev-only.
    
    ### How was this patch tested?
    
    Scala linter by `dev/linter-scala`.
    
    Closes #34075 from HyukjinKwon/minor-comment-unresolved-releation.
    
    Authored-by: Hyukjin Kwon <gurwls...@apache.org>
    Signed-off-by: Huaxin Gao <huaxin_...@apple.com>
    (cherry picked from commit 0076eba8d066936c32790ebc4058c52e2d21a207)
    Signed-off-by: Huaxin Gao <huaxin_...@apple.com>
---
 .../main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala
 
b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala
index 9f05367..9db038d 100644
--- 
a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala
+++ 
b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala
@@ -41,7 +41,7 @@ class UnresolvedException(function: String)
  * Holds the name of a relation that has yet to be looked up in a catalog.
  *
  * @param multipartIdentifier table name
- * @param options options to scan this relation. Only applicable to v2 table 
scan.
+ * @param options options to scan this relation.
  */
 case class UnresolvedRelation(
     multipartIdentifier: Seq[String],

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to