The GitHub Actions job "uv in /. - Update #1231451406" on 
iceberg-python.git/main has failed.
Run started by GitHub user dependabot[bot] (triggered by dependabot[bot]).

Head commit for run:
7e66ccbf16fc736df20f96c2ea030200fbe3a5e3 / geruh <[email protected]>
feat: Add tell() to OutputStream writers (#2998)

# Rationale for this change

Currently, PyIceberg writes one manifest per snapshot operation
regardless of manifest size. In order to eventually support this we need
to be able to track written bytes without closing the file, so that we
can roll to a new file once we hit target size.

We had some of this work done in #650, but we can keep this simple and
add writers as a follow up. The nice thing is that the underlying
streams we support already have a tell() method and we just need to
expose it.

With this change in the follow up we can do:

```
with write_manifest(...) as writer:
    writer.add_entry(entry)
    if writer.tell() >= target_file_size:
        # roll to new file
```

## Are these changes tested?

Yes, added a test :)

## Are there any user-facing changes?

No

Report URL: https://github.com/apache/iceberg-python/actions/runs/21615695451

With regards,
GitHub Actions via GitBox

Reply via email to