szehon-ho commented on code in PR #6729:
URL: https://github.com/apache/iceberg/pull/6729#discussion_r1116402901
##########
docs/spark-writes.md:
##########
@@ -302,6 +302,14 @@ data.writeTo("prod.db.table")
.createOrReplace()
```
+You can specify an Iceberg table location such as the `LOCATION` clause in SQL
by add the `location` paramter to the `tableProperty`:
Review Comment:
Nit: I know there's a bad example above, but typically we avoid 'you' or
'me' in documentation. you can see other reviews, for example:
https://github.com/apache/iceberg/pull/4301#discussion_r823197799
Also, I'm not sure why we reference "LOCATION" clause in SQL, which is
documented elsewhere.
How about:
```The Iceberg table location can also be specified by the `location` table
configuration```
Let me know what you think
--
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]
