cloud-fan commented on code in PR #58092:
URL: https://github.com/apache/spark/pull/58092#discussion_r3819817838


##########
docs/sql-ref-syntax-aux-describe-table.md:
##########
@@ -26,6 +26,13 @@ table. The metadata information includes column name, column 
type
 and column comment. Optionally a partition spec or column name may be specified
 to return the metadata pertaining to a partition or column respectively.
 
+If a table's declared partition columns do not match the last columns of its 
schema, the catalog
+metadata is inconsistent. `DESCRIBE TABLE` still returns the rest of the 
metadata, and reports
+both column lists under an `# Invalid Partition Information` section in place 
of the usual
+`# Partition Information` section. Repair such a table so that its declared 
partition columns
+match the last columns of its schema; until then, commands that require the 
partition schema,
+such as `DESCRIBE TABLE ... PARTITION`, `SHOW PARTITIONS` and `SHOW CREATE 
TABLE`, keep failing.

Review Comment:
   **Non-blocking:**
   
   These two examples do not generally keep failing: unfiltered `SHOW 
PARTITIONS` skips `partitionSchema` because normalization is inside `spec.map`, 
and `SHOW CREATE TABLE` renders `metadata.schema` and `partitionColumnNames` 
independently. Please remove them or qualify the exact forms that still assert.



-- 
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]

Reply via email to