szehon-ho commented on code in PR #7732:
URL: https://github.com/apache/iceberg/pull/7732#discussion_r1852955315


##########
docs/docs/spark-configuration.md:
##########
@@ -154,6 +154,10 @@ spark.read
     .table("catalog.db.table")
 ```
 
+Iceberg 1.8.0 and later support setting read options by Spark session 
configuration `spark.datasource.iceberg.<key>=<value>`

Review Comment:
   I still think we need new section like 'Configuration Priority' where we can 
explain the order of precedence:
   DataFrame Writes:
   - explicit dataframeWriter option
   - dataframe session default
   - if table exists, explicit table option
   - if table exists, table default
   
   DataFrame Reads:
   - explicit dataFrameReader option
   - dataframe session default
   - if table exists, explicit table option
   - if table exists, table default
   
   (please double check)



-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to