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 2022/03/21 23:37:28 UTC
[GitHub] [spark] dtenedor opened a new pull request #35928: [SPARK-38336][SQL] Check data source type for DEFAULT column values in ALTER TABLE ADD COLUMNS commands
dtenedor opened a new pull request #35928:
URL: https://github.com/apache/spark/pull/35928
### What changes were proposed in this pull request?
Add a quick check on the data source type in the presence of DEFAULT values in ALTER TABLE ADD COLUMNS commands. Such commands are rejected by default until explicitly allowlisted later, since each data source will be responsible for returning these values when the data is not present in storage (as opposed to stored NULL values).
### Why are the changes needed?
This provides a way to implement support for default values in each of the supported data sources as appropriate.
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
Unit 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.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
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 #35928: [SPARK-38336][SQL] Add class to assign DEFAULT column values to data source output rows during query execution
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #35928:
URL: https://github.com/apache/spark/pull/35928#issuecomment-1076330042
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.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
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