I thought sure I had a PR that added this, but I can't find it. +1 to adding `first_row_id`. Thanks, Prashant!
On Mon, May 12, 2025 at 9:22 AM Russell Spitzer <russell.spit...@gmail.com> wrote: > 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 >>> >>>
