You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2021/04/09 11:13:14 UTC
[GitHub] [spark] zheniantoushipashi opened a new pull request #32109: [SPARK-35007][SQL] support numeric type
zheniantoushipashi opened a new pull request #32109:
URL: https://github.com/apache/spark/pull/32109
What changes were proposed in this pull request?
Spark's numeric is a synonym of decimal
this pr add support for numeric type
Why are the changes needed?
my code need needs to use NUMERIC type in spark2.4 version but not support now
How was this patch tested?
Existing tests.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] zheniantoushipashi commented on pull request #32109: [SPARK-35007][SQL] support numeric type
Posted by GitBox <gi...@apache.org>.
zheniantoushipashi commented on pull request #32109:
URL: https://github.com/apache/spark/pull/32109#issuecomment-817081128
ok
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins commented on pull request #32109: [SPARK-35007][SQL] support numeric type
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #32109:
URL: https://github.com/apache/spark/pull/32109#issuecomment-816611876
Can one of the admins verify this patch?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] zheniantoushipashi commented on pull request #32109: [SPARK-35007][SQL] support numeric type
Posted by GitBox <gi...@apache.org>.
zheniantoushipashi commented on pull request #32109:
URL: https://github.com/apache/spark/pull/32109#issuecomment-816678252
retest this please
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] zheniantoushipashi commented on pull request #32109: [SPARK-35007][SQL] support numeric type
Posted by GitBox <gi...@apache.org>.
zheniantoushipashi commented on pull request #32109:
URL: https://github.com/apache/spark/pull/32109#issuecomment-817082356
> Please make pr against master branch:
> ![image](https://user-images.githubusercontent.com/5399861/114259240-10a6a900-99ff-11eb-940b-3e77944a4e3e.png)
Hello, my pr is developed based on branch 2.4, why should I submit it to the master branch? The master branch already supports numeric, I want to double commit to the 2.4 branch
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] zheniantoushipashi commented on pull request #32109: [SPARK-35007][SQL] support numeric type
Posted by GitBox <gi...@apache.org>.
zheniantoushipashi commented on pull request #32109:
URL: https://github.com/apache/spark/pull/32109#issuecomment-817111500
> Then, please use Spark 3.x. In general, `branch-2.4` only accepts bug fixes. Please see Versioning Policy: http://spark.apache.org/versioning-policy.html
ok
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins commented on pull request #32109: [SPARK-35007][SQL] support numeric type
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #32109:
URL: https://github.com/apache/spark/pull/32109#issuecomment-816757492
Can one of the admins verify this patch?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] wangyum commented on pull request #32109: [SPARK-35007][SQL] support numeric type
Posted by GitBox <gi...@apache.org>.
wangyum commented on pull request #32109:
URL: https://github.com/apache/spark/pull/32109#issuecomment-817080840
Please make pr against master branch:
![image](https://user-images.githubusercontent.com/5399861/114259240-10a6a900-99ff-11eb-940b-3e77944a4e3e.png)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] wangyum closed pull request #32109: [SPARK-35007][SQL] support numeric type
Posted by GitBox <gi...@apache.org>.
wangyum closed pull request #32109:
URL: https://github.com/apache/spark/pull/32109
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins removed a comment on pull request #32109: [SPARK-35007][SQL] support numeric type
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on pull request #32109:
URL: https://github.com/apache/spark/pull/32109#issuecomment-816611876
Can one of the admins verify this patch?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] wangyum commented on pull request #32109: [SPARK-35007][SQL] support numeric type
Posted by GitBox <gi...@apache.org>.
wangyum commented on pull request #32109:
URL: https://github.com/apache/spark/pull/32109#issuecomment-817107491
Then, please use Spark 3.x. In general, `branch-2.4` only accepts bug fixes. Please see Versioning Policy: http://spark.apache.org/versioning-policy.html
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] zheniantoushipashi removed a comment on pull request #32109: [SPARK-35007][SQL] support numeric type
Posted by GitBox <gi...@apache.org>.
zheniantoushipashi removed a comment on pull request #32109:
URL: https://github.com/apache/spark/pull/32109#issuecomment-817081128
ok
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org