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 <[email protected]>
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 <[email protected]>
Signed-off-by: Huaxin Gao <[email protected]>
(cherry picked from commit 0076eba8d066936c32790ebc4058c52e2d21a207)
Signed-off-by: Huaxin Gao <[email protected]>
---
.../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: [email protected]
For additional commands, e-mail: [email protected]
