8000 feat: add Kafka topic setup and recent spans polling endpoint by koxudaxi · Pull Request #516 · Mirascope/lilypad · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: add Kafka topic setup and recent spans polling endpoint #516

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 64 commits into from
Jun 10, 2025
Merged

feat: add Kafka topic setup and recent spans polling endpoint #516

merged 64 commits into from
Jun 10, 2025

Conversation

koxudaxi
Copy link
Collaborator
@koxudaxi koxudaxi commented Jun 3, 2025

No description provided.

koxudaxi added 17 commits June 3, 2025 22:06
@koxudaxi
feat: add Kafka topic setup and recent spans polling endpoint
04cdd63
@koxudaxi
fix: update trace status assertion in test_traces_api.py
a0c83a2
@koxudaxi
feat: add response models for recent spans and traces queue
aca882e
@koxudaxi
fix: fix sdk response
77fee81
@koxudaxi
fix: refactor Kafka service methods to synchronous execution and impr…
c16a551 
…ove error handling
@koxudaxi
feat: add user ID to span and batch span sending methods in Kafka ser…
d992002 
…vice
@koxudaxi
feat: migrate Kafka service to use aiokafka for asynchronous message …
1a78828 
…handling
@koxudaxi
feat: add reset singletons fixture and recent spans endpoint to spans…
06b8bcb 
… API
@koxudaxi
fix: simplify exception handling in Kafka service cleanup methods
4a94dfb
@koxudaxi
fix: update logging to clarify trace URL availability with Kafka queue
bbb2cc0
@koxudaxi
feat: implement recent spans retrieval for both sync and async clients
2a4158a
@koxudaxi
fix: add type ignore comments for pyright in various files
ff08b9a
@koxudaxi
fix: adjust recursion depth argument for child spans loading in spans.py
01f63bf
@koxudaxi
feat: refactor Kafka connection tests to use aiokafka for async support
a4f7bbe
@koxudaxi
fix: update datetime usage to use timezone.utc for consistency in spa…
93115cf 
…ns API
@koxudaxi
feat: migrate Kafka setup to use aiokafka for asynchronous topic mana…
1931ee4 
…gement
@koxudaxi
fix: update Kafka topic setup to await async method directly in async…
50dd67e 
… context
@koxudaxi koxudaxi marked this pull request as ready for review June 5, 2025 09:02
@koxudaxi koxudaxi requested review from brenkao and willbakst June 5, 2025 09:02
@koxudaxi koxudaxi requested a review from brenkao June 6, 2025 09:09
brenkao
brenkao reviewed Jun 7, 2025
View reviewed changes
@koxudaxi koxudaxi requested a review from brenkao June 9, 2025 03:22
brenkao
brenkao reviewed Jun 9, 2025
View reviewed changes
@koxudaxi koxudaxi requested a review from brenkao June 10, 2025 06:08
@koxudaxi koxudaxi merged commit e091b4d into main Jun 10, 2025
16 checks passed
@koxudaxi koxudaxi deleted the use-kafka-topic-for-spans branch June 10, 2025 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brenkao brenkao brenkao approved these changes

@willbakst willbakst Awaiting requested review from willbakst

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@koxudaxi @codecov-commenter @brenkao
0