Skip to content

Add record timestamps #2288

Description

@morenol

Ensure every batch has correct timestamp

Activity

  1. self-assigned this
    on Apr 8, 2022
  2. added this to the 0.9.24 milestone on Apr 8, 2022
  3. morenol commented on Apr 19, 2022

    @morenol
    CollaboratorAuthor

    Currently all timestamp related fields in Records and Batches are set to 0. We need to update the producer to populate the timestamp fields

  4. changed the title [-]Validate record timestamps[/-] [+]Add record timestamps[/+] on Apr 19, 2022
  5. modified the milestones: 0.9.24, 0.9.25 on Apr 21, 2022
  6. morenol commented on May 2, 2022

    @morenol
    CollaboratorAuthor

    Regarding CLI:

    Can we leverage https://docs.rs/chrono/0.4.19/chrono/format/strftime/index.html#specifiers?

    @sehz since we are using Handlebars, we need to pass {{ name }}, for every specifier that we want. Do we want to be able to specify them with something like {{ Y }}, {{ C }}? I think that for CLI it is enough to follow a specific RFC format

  7. added a commit that references this issue on May 2, 2022
  8. modified the milestones: 0.9.25, 0.9.26 on May 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions