Thanks Yufei. I'm still seeking more votes here. Manu
On Wed, May 14, 2025 at 1:37 AM Yufei Gu <flyrain...@gmail.com> wrote: > +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 >>> >>
