MaxGekk opened a new pull request #30641:
URL: https://github.com/apache/spark/pull/30641


   ### What changes were proposed in this pull request?
   Invoke the check `DDLUtils.verifyPartitionProviderIsHive()` from V1 
implementation of `SHOW TABLE EXTENDED` when partition specs are specified.
   
   This PR is some kind of follow up https://github.com/apache/spark/pull/16373 
and https://github.com/apache/spark/pull/15515.
   
   ### Why are the changes needed?
   To output an user friendly error with recommendation like
   **"
   ... partition metadata is not stored in the Hive metastore. To import this 
information into the metastore, run `msck repair table tableName`
   "**
   instead of silently output an empty result.
   
   ### Does this PR introduce _any_ user-facing change?
   Yes.
   
   ### How was this patch tested?
   By running the affected test suites, in particular:
   ```
   $ build/sbt -Phive-2.3 -Phive-thriftserver "test:testOnly 
*HiveCatalogedDDLSuite"
   $ build/sbt -Phive-2.3 -Phive-thriftserver "hive/test:testOnly 
*PartitionProviderCompatibilitySuite"
   ```
   
   Authored-by: Max Gekk <[email protected]>
   Signed-off-by: HyukjinKwon <[email protected]>
   (cherry picked from commit 29096a8869c95221dc75ce7fd3d098680bef4f55)
   Signed-off-by: Max Gekk <[email protected]>


----------------------------------------------------------------
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:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to