RyukR1 commented on code in PR #3457:
URL: https://github.com/apache/iceberg-python/pull/3457#discussion_r3385069728
##########
pyiceberg/table/inspect.py:
##########
@@ -333,6 +424,21 @@ def _update_partitions_map_from_manifest_entry(
partition_record_dict: dict[str, Any],
snapshot: Snapshot | None,
) -> None:
+ """Update the partition statistics map with data from a single
manifest entry.
+
+ Initialises a new partition row if the key has not been seen, then
+ increments count and size statistics based on the file content type.
+
+ Args:
+ partitions_map: Mutable dict keyed by hashable partition values,
+ accumulating per-partition statistics.
+ file: The data file entry from the manifest.
+ partition_record_dict: Maps partition field names to their values.
+ snapshot: The owning snapshot, used to track last-updated
timestamps.
+
+ Raises:
+ ValueError: If the file has an unknown ``DataFileContent`` type.
Review Comment:
Thanks for the feedback! I've updated the docstrings to remove the Raises:
ValueError section since it is a private method and an unreachable fallback
condition. I also fixed a typo in the commit message (closes instead of
closses). The changes are updated and ready for review.
--
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]
