laskoviymishka opened a new pull request, #2179:
URL: https://github.com/apache/iceberg-go/pull/2179
ManifestFile.FetchEntries loaded every entry into memory and took no
context. Use ManifestFile.Entries, which streams entries as an iterator:
```
for entry, err := range mf.Entries(fs, discardDeleted) {
if err != nil {
return err
}
// use entry
}
```
or ReadManifest when a slice is needed.
DataFile.DistinctValueCounts and DataFileBuilder.DistinctValueCounts exposed
distinct_counts (field 111), which the spec deprecates in every format version.
Writers already omitted the field, so the setter had no effect on the wire. The
Avro field stays on the data file struct, so legacy manifests that carry it
still decode, and writers continue to omit it. The tests now check both on the
struct, and a codec test checks that the entry schema has no field 111.
This is a breaking change for callers of the removed methods.
Part of the v1.0 API cleanup in #2062.
--
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]
