Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add optional automatic time field population to ingest #101

Open
tmc opened this issue Sep 1, 2019 · 1 comment
Open

add optional automatic time field population to ingest #101

tmc opened this issue Sep 1, 2019 · 1 comment

Comments

@tmc
Copy link
Contributor

tmc commented Sep 1, 2019

Having a time field automatically populated with ingestion time if missing would be convenient.

@okayzed
Copy link
Collaborator

okayzed commented Sep 2, 2019

I'm conflicted on this. On the one hand, timestamps are not required (or are not required to be named "time") but they are generally useful to have in most datasets.

I think adding a flag that says "add timestamp if missing" would be a good compromise, but then most people probably wouldn't realize that the flag exists. OTH, having this be default behavior can lead to confusing situations

Thoughts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants