kevinjqliu commented on code in PR #1611:
URL: https://github.com/apache/iceberg-python/pull/1611#discussion_r1945565345


##########
mkdocs/docs/configuration.md:
##########
@@ -239,9 +240,9 @@ When several files are stored under the same prefix, cloud 
object stores such as
 resulting in slowdowns. The `ObjectStoreLocationProvider` counteracts this by 
injecting deterministic hashes, in the form of binary directories,
 into file paths, to distribute files across a larger number of object store 
prefixes.
 
-Paths still contain partitions just before the file name, in Hive-style, and a 
`data` directory beneath the table's location,
-in a similar manner to the 
[`SimpleLocationProvider`](configuration.md#simple-location-provider). For 
example, a table
-partitioned over a string column `category` might have a data file with 
location: (note the additional binary directories)
+Paths still are also prefixed by `{location}/data/`, where `location` comes 
from the [table 
metadata](https://iceberg.apache.org/spec/#table-metadata-fields), in a similar 
manner to the 
[`SimpleLocationProvider`](configuration.md#simple-location-provider). This can 
be overridden by setting [`write.data.path` table 
configuration](#write-options).

Review Comment:
   ```suggestion
   Paths are still prefixed by `{location}/data/`, where `location` comes from 
the [table metadata](https://iceberg.apache.org/spec/#table-metadata-fields), 
in a similar manner to the 
[`SimpleLocationProvider`](configuration.md#simple-location-provider). This can 
be overridden by setting [`write.data.path` table 
configuration](#write-options).
   ```



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