stevenzwu commented on code in PR #12315:
URL: https://github.com/apache/iceberg/pull/12315#discussion_r1960689270


##########
docs/docs/aws.md:
##########
@@ -385,6 +385,7 @@ Note, the path resolution logic for 
`ObjectStoreLocationProvider` is `write.data
 However, for the older versions up to 0.12.0, the logic is as follows:
 - before 0.12.0, `write.object-storage.path` must be set.
 - at 0.12.0, `write.object-storage.path` then `write.folder-storage.path` then 
`<tableLocation>/data`.
+- at 2.0.0 `write.object-storage.path` and `write.folder-storage.path` will be 
removed

Review Comment:
   it seems that the existing line 387 above missed one config 
`write.data.path`. 
   
   or maybe `write.data.path` was added after 0.12.0. then we missed a 
line/version altogether.



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