Peng-Lei opened a new pull request #35204:
URL: https://github.com/apache/spark/pull/35204
### What changes were proposed in this pull request?
1. Add `quoted(identifier: TableIdentifier)` to quoted the table name of V1
command(SHOW CREATE TABLE[AS SERDE]) to match
V2 behavior. It just work when `quoteIfNeeded`
2. Change `addV2TableProperties` of `V1Table`. Just when `external == true`,
we will add `location` property.
3. Change `V1Table`.`Schema`, re-construct the original schema from the
string.
4. Move the `convertTableMetadata` from `tables.scala` to `ddl.scala`
5. Use V2 command as default for `SHOW CRATE TABLE`
6. Change V2 behavior `ShowTablePartitions` to match V1 behavior.
7. Add special logic to V2 command for handle hive related logic
### Why are the changes needed?
It's been a while since we introduced the v2 commands, and it seems
reasonable to use v2 commands by default even for the session catalog, with a
legacy config to fall back to the v1 commands.
### Does this PR introduce _any_ user-facing change?
use V2 command as default for `show create table`
if LEGACY_USE_V1_COMMAND == true
will use V1 command
### How was this patch tested?
build/sbt -Phive-2.3 -Phive-thriftserver "test:testOnly
*ShowCreateTableSuite"
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
