Github user wangtao605 commented on the issue:
https://github.com/apache/spark/pull/21766
> @wangtao605 Do you mind documenting our behavior in our Spark SQL doc?
Yes, it's ok.
---
-
To unsubscribe, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21766
@wangtao605 Do you mind documenting our behavior in our Spark SQL doc?
---
-
To unsubscribe, e-mail:
Github user wangtao605 commented on the issue:
https://github.com/apache/spark/pull/21766
@rxin In order to support sql syntax better and align SQL standards. I
think it is worth to add a numeric type as an alias of decimal.
---
Github user wangtao605 commented on the issue:
https://github.com/apache/spark/pull/21766
@dmateusp Actually i think "Numeric" has no essential difference with
"Decimal". May be just have it as an alias is better,i will add some tests if
you agree.
---
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/21766
Why did you need this change? Given it's very difficult to revert the
change (or introduce a proper numeric type if ever needed in the future), I
would not merge this pull request unless there are
Github user dmateusp commented on the issue:
https://github.com/apache/spark/pull/21766
Just checked out the PR,
```scala
scala> spark.sql("SELECT CAST(1 as NUMERIC)")
res0: org.apache.spark.sql.DataFrame = [CAST(1 AS DECIMAL(10,0)):
decimal(10,0)]
scala>
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21766
aha, I didn't know that and postgresql also uses them interchangeably, too.
---
-
To unsubscribe, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21766
They are not exactly the same in ANSI SQL, although MS SQL Server looks
like use them interchangeably.
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21766
@gatorsmile Is it worth adding a numeric type as an alias of decimal? Both
types seems to be in the SQL standard and some dbms (e.g., postgresql and sql
server) can parse both.
@wangtao605 Can
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21766
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
10 matches
Mail list logo