Makes sense to me, perhaps we should also add in a test that checks that the Datafile api object and the REst spec are always in sync?
On Mon, May 12, 2025 at 10:52 AM Amogh Jahagirdar <2am...@gmail.com> wrote: > Thanks Prashant, I definitely agree the first_row_id will need to be added > to the REST Spec. Commented on the PR, I also think we'll need to make sure > the first-row-id for Snapshots are also added as part of this. > > Thanks, > Amogh Jahagirdar > > On Fri, May 9, 2025 at 6:12 PM Prashant Singh <prashant010...@gmail.com> > wrote: > >> Hey folks, >> >> I recently stumbled on the first_row_id missing in the DataFile for the >> rest spec. The first_row_id information IMHO needs to be passed from server >> to client for reads, as it is inferred during planning via inheritance from >> Manifest and manifest list and not something client can infer on its own >> without them. >> >> Starting a discussion thread, to see how to incorporate this to rest >> spec. >> Meanwhile PR : https://github.com/apache/iceberg/pull/13010 >> >> Best, >> Prashant >> >>
