This is an automated email from the ASF dual-hosted git repository.
maxgekk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 8cfd5bf1ca4 [SPARK-42430][DOC][FOLLOW-UP] Revise the java doc for
TimestampNTZ & ANSI interval types
8cfd5bf1ca4 is described below
commit 8cfd5bf1ca4042541232ef1787349ddb876adcfa
Author: Gengliang Wang <[email protected]>
AuthorDate: Sat Feb 18 10:30:32 2023 +0300
[SPARK-42430][DOC][FOLLOW-UP] Revise the java doc for TimestampNTZ & ANSI
interval types
### What changes were proposed in this pull request?
As https://github.com/apache/spark/pull/40005#pullrequestreview-1299089504
pointed out, the java doc for data type recommends using factory methods
provided in org.apache.spark.sql.types.DataTypes.
Since the ANSI interval types missed the `DataTypes` as well, this PR also
revise their doc.
### Why are the changes needed?
Unify the data type doc
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
Local preview
<img width="826" alt="image"
src="https://user-images.githubusercontent.com/1097932/219821685-321c2fd1-6248-4930-9c61-eec68f0dcb50.png";>
Closes #40074 from gengliangwang/reviseNTZDoc.
Authored-by: Gengliang Wang <[email protected]>
Signed-off-by: Max Gekk <[email protected]>
---
docs/sql-ref-datatypes.md | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/docs/sql-ref-datatypes.md b/docs/sql-ref-datatypes.md
index 3095a196a35..64c6d335429 100644
--- a/docs/sql-ref-datatypes.md
+++ b/docs/sql-ref-datatypes.md
@@ -128,7 +128,7 @@ You can access them by doing
|**BinaryType**|Array[Byte]|BinaryType|
|**BooleanType**|Boolean|BooleanType|
|**TimestampType**|java.sql.Timestamp|TimestampType|
-|**TimestampNTZType**|java.time.LocalDateTime| TimestampNTZType|
+|**TimestampNTZType**|java.time.LocalDateTime|TimestampNTZType|
|**DateType**|java.sql.Date|DateType|
|**YearMonthIntervalType**|java.time.Period|YearMonthIntervalType|
|**DayTimeIntervalType**|java.time.Duration|DayTimeIntervalType|
@@ -159,10 +159,10 @@ please use factory methods provided in
|**BinaryType**|byte[]|DataTypes.BinaryType|
|**BooleanType**|boolean or Boolean|DataTypes.BooleanType|
|**TimestampType**|java.sql.Timestamp|DataTypes.TimestampType|
-|**TimestampNTZType**|java.time.LocalDateTime| TimestampNTZType|
+|**TimestampNTZType**|java.time.LocalDateTime|DataTypes.TimestampNTZType|
|**DateType**|java.sql.Date|DataTypes.DateType|
-|**YearMonthIntervalType**|java.time.Period|YearMonthIntervalType|
-|**DayTimeIntervalType**|java.time.Duration|DayTimeIntervalType|
+|**YearMonthIntervalType**|java.time.Period|DataTypes.YearMonthIntervalType|
+|**DayTimeIntervalType**|java.time.Duration|DataTypes.DayTimeIntervalType|
|**ArrayType**|java.util.List|DataTypes.createArrayType(*elementType*)<br/>**Note:**
The value of *containsNull* will be
true.<br/>DataTypes.createArrayType(*elementType*, *containsNull*).|
|**MapType**|java.util.Map|DataTypes.createMapType(*keyType*,
*valueType*)<br/>**Note:** The value of *valueContainsNull* will be
true.<br/>DataTypes.createMapType(*keyType*, *valueType*, *valueContainsNull*)|
|**StructType**|org.apache.spark.sql.Row|DataTypes.createStructType(*fields*)<br/>**Note:**
*fields* is a List or an array of StructFields.Also, two fields with the same
name are not allowed.|
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
![https://user-images.githubusercontent.com/1097932/219821685-321c2fd1-6248-4930-9c61-eec68f0dcb50.png"](https://user-images.githubusercontent.com/1097932/219821685-321c2fd1-6248-4930-9c61-eec68f0dcb50.png"){kind=link}