Skip to content
This repository has been archived by the owner on Jan 17, 2024. It is now read-only.

Update TraceID to use time.Time instead of uint64 #31

Merged
merged 1 commit into from
Jan 30, 2023
Merged

Update TraceID to use time.Time instead of uint64 #31

merged 1 commit into from
Jan 30, 2023

Conversation

vortegatorres
Copy link
Contributor

TraceID changes to use time.Time instead of uint64 in its timestamp done in PR #137 reproduced in xk6-distributed-tracing TraceID. Both TraceID implementation should be compatible.

Issue https://github.com/grafana/k6-cloud-crocospans/issues/133

@vortegatorres vortegatorres self-assigned this Jan 27, 2023
@CLAassistant
Copy link

CLAassistant commented Jan 30, 2023 •
edited

CLA assistant check
All committers have signed the CLA.

@vortegatorres vortegatorres merged commit 1326ddc into master Jan 30, 2023
@Blinkuu Blinkuu deleted the change_trace_id_to_use_int64 branch January 30, 2023 15:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Blinkuu Blinkuu Blinkuu approved these changes

Assignees

@vortegatorres vortegatorres

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@vortegatorres @CLAassistant @Blinkuu