+1 I'm OK to add it as long as it's optional. Yufei
On Mon, May 12, 2025 at 8:47 PM Manu Zhang <owenzhang1...@gmail.com> wrote: > Hi all, > > The background is that we schedule maintenance jobs based on commit > reports for Iceberg tables, and we want to know *when commits happen*. > Adding timestamp to the commit report would save us from loading metadata > of every table from the filesystem. > > Please take a look at the PR and cast your vote. Feel free to ping me if > you have any questions. > > Thanks, > Manu > > > On Fri, May 9, 2025 at 12:01 AM 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 <https://github.com/apache/iceberg/pull/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 >> >
