Hi Fede, Thanks for the KIP. LGTM.
@Divij I think the timestamp is timezone independent, so it should be fine. Luke On Thu, May 11, 2023 at 2:04 AM Divij Vaidya <divijvaidy...@gmail.com> wrote: > Thank you for the KIP. > > The current proposal has the limitation that it uses a duration syntax for > representation of a timestamp. Also, the syntax relies on the locale and > timezone of the caller machine. This makes it difficult to share the output > with others. As an example, let's say you want to share the state of the > quorum with me which was captured but "3s ago" gives me no information on > when it was executed. > > Alternatively, may I suggest introducing a parameter called > "--datetime-format=" which takes as value the ISO-8601 [1] format and > prints the timestamp based on the provided format. It would solve the > problem of readability (epochs are hard to read!) as well as the problem of > portability of output across machines. > > What do you think? > > [1] https://en.wikipedia.org/wiki/ISO_8601 > > -- > Divij Vaidya > > > > On Wed, May 10, 2023 at 6:10 PM Federico Valeri <fedeval...@gmail.com> > wrote: > > > Hi all, I'd like to start a new discussion thread on KIP-927: Improve > > the kafka-metadata-quorum output. > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-927%3A+Improve+the+kafka-metadata-quorum+output > > > > This KIP is small and proposes to add a new optional flag to have a > > human-readable timestamp output. > > > > Thank you! > > > > Br > > Fede > > >
