Skip to content

added a new admin endpoint - /logs to fetch log files#491

Open
akash-a-n wants to merge 1 commit into
mainfrom
qlogs/admin-endpoint
Open

added a new admin endpoint - /logs to fetch log files#491
akash-a-n wants to merge 1 commit into
mainfrom
qlogs/admin-endpoint

Conversation

@akash-a-n

@akash-a-n akash-a-n commented Jul 8, 2026

Copy link
Copy Markdown
Contributor
  • Implement GET /logs?type=<mlog|qlog>&connection_id=<hex> route.
  • Offload blocking file I/O (std::ifstream::read) to folly::getGlobalCPUExecutor() to avoid stalling the admin event loop.

This change is Reviewable

@akash-a-n
akash-a-n force-pushed the qlogs/admin-endpoint branch from 9bed433 to 74ba272 Compare July 9, 2026 06:47
@akash-a-n
akash-a-n force-pushed the qlogs/admin-endpoint branch from 74ba272 to c161002 Compare July 22, 2026 14:26
…ed bugfix

- Implement `GET /logs?type=<mlog|qlog>&connection_id=<hex>` route.
- Offload blocking file I/O (`std::ifstream::read`) to
  `folly::getGlobalCPUExecutor()` to avoid stalling the admin event loop.
- Bugfix: create log dir if not exists
@akash-a-n
akash-a-n force-pushed the qlogs/admin-endpoint branch from c161002 to 02c1462 Compare July 22, 2026 15:12
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.

1 participant