Skip to content

Commit

Permalink
Traces type consolidation (#180)
Browse files Browse the repository at this point in the history
* Move traces types and schemas to types package

* Clean up

* Refactor traces

* Linter

* Handle Date types from otel with the schemas

* Port rust api to be compatible with typescript api

* Fix types across the frontend

* Linter

* Fix migration

* Linter

* Biome

* Remove ts compat trace

* Add drizzle snapshot

* Fix migration

* Remove ts_compat

* wip

* Add untagged

* Clippy

* Remove start and end time on traces

* Make properties optional like in the ts api

* Fix case renaming

* Run biome

* Update client to reflect new paths

* coerce the timestamp into date too

* Run biome

* Fix ts error with timestamps

* More ts fixes

---------

Co-authored-by: Benno van den Berg <[email protected]>
Co-authored-by: Laurynas Keturakis <[email protected]>
  • Loading branch information
3 people authored Sep 9, 2024
1 parent 40294bd commit 2b5b114
Show file tree
Hide file tree
Showing 55 changed files with 763 additions and 508 deletions.
13 changes: 13 additions & 0 deletions api/drizzle/0014_worthless_mystique.sql
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
CREATE TABLE `otel_spans_new` (
`inner` text NOT NULL,
`span_id` text NOT NULL,
`trace_id` text NOT NULL
);
--> statement-breakpoint
INSERT INTO `otel_spans_new` (`inner`, `span_id`, `trace_id`)
SELECT `parsed_payload`, `span_id`, `trace_id`
FROM `otel_spans`;
--> statement-breakpoint
DROP TABLE `otel_spans`;
--> statement-breakpoint
ALTER TABLE `otel_spans_new` RENAME TO `otel_spans`;
2 changes: 1 addition & 1 deletion api/drizzle/meta/0012_snapshot.json
Original file line number Diff line number Diff line change
Expand Up @@ -485,4 +485,4 @@
"tables": {},
"columns": {}
}
}
}
2 changes: 1 addition & 1 deletion api/drizzle/meta/0013_snapshot.json
Original file line number Diff line number Diff line change
Expand Up @@ -482,4 +482,4 @@
"tables": {},
"columns": {}
}
}
}
Loading

0 comments on commit 2b5b114

Please sign in to comment.