Skip to content

Conversation

@Peng-Lei
Copy link
Contributor

What changes were proposed in this pull request?

Quote the column name of view when SHOW CREATE TABLE for view.

Why are the changes needed?

follow up the #PR. Keep the consistent between table and view when SHOW CREATE TABLE.

Does this PR introduce any user-facing change?

Yes,It will change the result of SHOW CREATE TABLE for view.
eg:

"STRUCT<`_c0`, `_c1`>"  => "STRUCT<_c0, _c1>" 

How was this patch tested?

existed testcase.

@github-actions github-actions bot added the SQL label Jan 28, 2022
@Peng-Lei Peng-Lei changed the title [SPARK-37931][SQL][FOLLOWUP] Quote the column name of view [SPARK-37931][SQL][FOLLOWUP] Quote the column name of view if needed Jan 28, 2022
@Peng-Lei
Copy link
Contributor Author

@cloud-fan Could you take a look? Thank you very much.

@cloud-fan
Copy link
Contributor

thanks, merging to master!

@cloud-fan cloud-fan closed this in d150b7e Jan 28, 2022
senthh pushed a commit to senthh/spark-1 that referenced this pull request Feb 3, 2022
### What changes were proposed in this pull request?
Quote the column name of view when `SHOW CREATE TABLE` for view.

### Why are the changes needed?
follow up the [#PR](apache#35227). Keep the consistent between table and view when `SHOW CREATE TABLE`.

### Does this PR introduce _any_ user-facing change?
Yes,It will change the result of `SHOW CREATE TABLE` for view.
eg:
```
"STRUCT<`_c0`, `_c1`>"  => "STRUCT<_c0, _c1>"
```

### How was this patch tested?
existed testcase.

Closes apache#35351 from Peng-Lei/view-quote-columns.

Authored-by: PengLei <peng.8lei@gmail.com>
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants