Skip to content

Feature: add support for influx timestamp#37

Open
IvanShpotenko wants to merge 4 commits intolexmag:masterfrom
IvanShpotenko:fix-add-timestamp
Open

Feature: add support for influx timestamp#37
IvanShpotenko wants to merge 4 commits intolexmag:masterfrom
IvanShpotenko:fix-add-timestamp

Conversation

@IvanShpotenko
Copy link
Copy Markdown

@IvanShpotenko IvanShpotenko commented Sep 16, 2021

This PR addresses an #6

There is a stale PR #24 that seems abandoned.

The current PR is based on that work by @jbavari and @ebostijancic and addresses comments left on the original PR.

@lexmag / @hauleth you guys think we can get this reviewed and merged?

Copy link
Copy Markdown

@AndriyLazorenko AndriyLazorenko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@feld
Copy link
Copy Markdown

feld commented Mar 10, 2023

please merge this and make a new release, it would be extremely useful for me when I need to ingest backdated metrics with Elixir

@AndriyLazorenko
Copy link
Copy Markdown

@feld owners have some sloppy CI, unable to merge due to its failure, seems like runner is not maintained...

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

Successfully merging this pull request may close these issues.

3 participants