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 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: [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]
