[GitHub] [spark] dongjoon-hyun commented on a change in pull request #31993: [SPARK-34897][SQL] Add workaround to error message when OrcUtils.requestedColumnIds fails

[GitBox <gi...@apache.org>]

dongjoon-hyun commented on a change in pull request #31993:
URL: https://github.com/apache/spark/pull/31993#discussion_r603399343



##########
File path: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/orc/OrcUtils.scala
##########
@@ -157,7 +158,8 @@ object OrcUtils extends Logging {
         // In these cases we map the physical schema to the data schema by index.
         assert(orcFieldNames.length <= dataSchema.length, "The given data schema " +
           s"${dataSchema.catalogString} has less fields than the actual ORC physical schema, " +
-          "no idea which columns were dropped, fail to read.")
+          "no idea which columns were dropped, fail to read. Try to disable " +
+          s"${NESTED_SCHEMA_PRUNING_ENABLED.key} to workaround this issue.")

Review comment:
       Although `spark.sql.optimizer.nestedSchemaPruning.enabled` is true by default, can we hide this conf guide when `spark.sql.optimizer.nestedSchemaPruning.enabled=false`, @wangyum ?




-- 
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org