Thanks for your thoughts, JB. I agree with the concerns, but note that reading table metadata from the file system for each commit report is not cost-free. I'd rather use the system time instead of reading table metadata.
Regards, Manu On Wed, May 21, 2025 at 3:56 PM Jean-Baptiste Onofré <j...@nanthrax.net> wrote: > +0 > > I don't see much value compared to reading the table metadata via snapshot > id. > I'm not against it, but it seems a bit redundant. Giving direct access > via REST Catalog could be discussed also. > > Regards > JB > > On Thu, May 8, 2025 at 5:01 PM Manu Zhang <owenzhang1...@gmail.com> wrote: > > > > Hi all, > > > > I'd like to start a vote to add commit timestamp `timestamp-millis` to > CommitReport in PR 12990. > > The timestamp info is valuable to schedule maintenance jobs, but > currently we need to look it up from the snapshot metadata table. > > > > Please take a look at the PR and vote in the next 72 hours. > > > > [ ] +1 ok to add the field > > [ ] +0 > > [ ] -1 I have concerns or there's a better alternative > > > > Thanks, > > Manu >
