This is an automated email from the ASF dual-hosted git repository.
LuciferYang pushed a commit to branch branch-4.x
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/branch-4.x by this push:
new ba21158a6852 [MINOR][SQL] Fix incorrect type wording in Row getter
scaladoc
ba21158a6852 is described below
commit ba21158a6852140a3814512b4b10d1912b30e997
Author: YangJie <[email protected]>
AuthorDate: Fri Jul 17 11:28:12 2026 +0800
[MINOR][SQL] Fix incorrect type wording in Row getter scaladoc
### What changes were proposed in this pull request?
Fixes copy-pasted type wording in the scaladoc of several `Row` getters in
the sql-api module. The phrase "value at position i of `<type>` type" had been
copy-pasted from the date getters onto getters that return other types:
- `getGeometry`: "of date type" -> "of geometry type"
- `getGeography`: "of date type" -> "of geography type"
- `getTimestamp` / `getInstant`: "of date type" -> "of timestamp type"
- `getJavaMap`: "of array type" -> "of map type"
### Why are the changes needed?
The type wording on these getters was incorrect (copy-pasted from sibling
getters), which is misleading in the generated API docs.
### Does this PR introduce _any_ user-facing change?
No. Documentation-only; no behavior change.
### How was this patch tested?
No tests needed (documentation only).
### Was this patch authored or co-authored using generative AI tooling?
Generated-by: Claude Code(Opus 4.8)
Closes #57240 from LuciferYang/SPARK-tier3-scaladoc-cleanup.
Authored-by: YangJie <[email protected]>
Signed-off-by: yangjie01 <[email protected]>
(cherry picked from commit 2abe31bf88946f21a2ef9b2170d2aa31a99de617)
Signed-off-by: yangjie01 <[email protected]>
---
sql/api/src/main/scala/org/apache/spark/sql/Row.scala | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/sql/api/src/main/scala/org/apache/spark/sql/Row.scala
b/sql/api/src/main/scala/org/apache/spark/sql/Row.scala
index 78842a2df514..ef5c06d8a35c 100644
--- a/sql/api/src/main/scala/org/apache/spark/sql/Row.scala
+++ b/sql/api/src/main/scala/org/apache/spark/sql/Row.scala
@@ -304,7 +304,7 @@ trait Row extends Serializable {
def getDecimal(i: Int): java.math.BigDecimal = getAs[java.math.BigDecimal](i)
/**
- * Returns the value at position i of date type as
org.apache.spark.sql.types.Geometry.
+ * Returns the value at position i of geometry type as
org.apache.spark.sql.types.Geometry.
*
* @throws ClassCastException
* when data type does not match.
@@ -313,7 +313,7 @@ trait Row extends Serializable {
getAs[org.apache.spark.sql.types.Geometry](i)
/**
- * Returns the value at position i of date type as
org.apache.spark.sql.types.Geography.
+ * Returns the value at position i of geography type as
org.apache.spark.sql.types.Geography.
*
* @throws ClassCastException
* when data type does not match.
@@ -338,7 +338,7 @@ trait Row extends Serializable {
def getLocalDate(i: Int): java.time.LocalDate = getAs[java.time.LocalDate](i)
/**
- * Returns the value at position i of date type as java.sql.Timestamp.
+ * Returns the value at position i of timestamp type as java.sql.Timestamp.
*
* @throws ClassCastException
* when data type does not match.
@@ -346,7 +346,7 @@ trait Row extends Serializable {
def getTimestamp(i: Int): java.sql.Timestamp = getAs[java.sql.Timestamp](i)
/**
- * Returns the value at position i of date type as java.time.Instant.
+ * Returns the value at position i of timestamp type as java.time.Instant.
*
* @throws ClassCastException
* when data type does not match.
@@ -389,7 +389,7 @@ trait Row extends Serializable {
def getMap[K, V](i: Int): scala.collection.Map[K, V] = getAs[Map[K, V]](i)
/**
- * Returns the value at position i of array type as a `java.util.Map`.
+ * Returns the value at position i of map type as a `java.util.Map`.
*
* @throws ClassCastException
* when data type does not match.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
