fluvio icon indicating copy to clipboard operation
fluvio copied to clipboard

Topic should show timestamp of last produced record

Open nacardin opened this issue 2 years ago • 7 comments

In the output of fluvio topic list

nacardin avatar Mar 25 '22 18:03 nacardin

Approach could be to consume latest record and use it's timestamp. Need to double check that record timestamp is accurate

nacardin avatar Apr 08 '22 18:04 nacardin

Stale issue message

github-actions[bot] avatar Jun 08 '22 11:06 github-actions[bot]

Might be nice to do this with a "--verbose" flag

digikata avatar Oct 18 '23 22:10 digikata

Might be nice to do this with a "--verbose" flag

maybe: timestamp, offset, key and value.

BTW, this is all possible with --format option.

My bad, please ignore.

ajhunyady avatar Oct 19 '23 01:10 ajhunyady

@digikata I just submitted a draft PR that queries the latest timestamp and includes it in the output. I haven't yet included the --verbose flag yet because I wanted to inquire further about what @ajhunyady just mentioned.

@ajhunyady, could you expand a bit on what you mentioned? Are you suggesting that instead of --verbose, we add support for --timestamp, --offset, --key, and --value for optional granular information about the latest record of a given topic?

Also could you tell me more about the --format option? Thank you!

solidiquis avatar Oct 19 '23 04:10 solidiquis

@solidiquis, sorry for the confusion, and ignore my comment. I thought we were discussing the consumer. The solution should be good enough.

BTW, we have a long standing bug in changing STATUS to a readable format: resolution::provisioned should be provisioned ... same for the other states.

It would be great if we could fix this as well.

Also instead of MODIFIED AT -> LAST MODIFIED

ajhunyady avatar Oct 19 '23 15:10 ajhunyady

Might be nice to do this with a "--verbose" flag

It's only 1 field, it's probably ok to add as is, otherwise we should probably remove other fields and add to verbose

ajhunyady avatar Oct 19 '23 15:10 ajhunyady

Awesome thanks for the input @ajhunyady! In that case I'll include LAST MODIFIED in the default output and carry on without adding --verbose. I'll also address the STATUS bug you just mentioned as well shortly.

solidiquis avatar Oct 19 '23 15:10 solidiquis