Abyss-lord commented on PR #6882: URL: https://github.com/apache/gravitino/pull/6882#issuecomment-2947900008
> > > > seems we could close this PR? > > > > > > > > > Why？ > > > > > > Prometheus doesn't support timestamps in metrics. > > @Abyss-lord WDYT? @jerqi @FANNG1 Thanks for the clarification! Would this kind of .timestamp suffix be considered acceptable in general? Could this pattern still be useful in some cases — for example, if we just want to know when a particular metric was last updated? Maybe it's not suitable for all metrics, but could it be acceptable to apply this suffix to a small number of selected metrics where update time is meaningful? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
