danielcweeks commented on a change in pull request #206: Add option to load
column stats with data files.
URL: https://github.com/apache/incubator-iceberg/pull/206#discussion_r291276021
##########
File path: api/src/main/java/org/apache/iceberg/DataFile.java
##########
@@ -138,6 +138,14 @@ static StructType getType(StructType partitionType) {
*/
DataFile copy();
+ /**
+ * Copies this {@link DataFile data file} without file stats. Manifest
readers can reuse data file instances; use
+ * this method to copy data without stats when collecting files.
+ *
+ * @return a copy of this data file
+ */
+ DataFile slimCopy();
Review comment:
I don't particularly love the terminology here. Why not simply add
`copy(boolean stats)` or a copy with an enum to indicate what portions of the
datafile to include? At some point we may want just some of the values (e.g.
CBO may want counts, but not lower/upper bounds). Just a thought.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
