-
Notifications
You must be signed in to change notification settings - Fork 83
Open
Description
Description
Based on the discussion in PR #1275 (#1275), we've identified that the current logging in search routes may be unnecessary since Fastify already automatically logs route information for each incoming request.
Background
During the review of PR #1275 which adds logging to Presto search routes, it was discovered that:
- Fastify already prints route URLs in the logs for each incoming request
- The additional manual logging of route arguments may be redundant
- This affects both regular search routes and the newly proposed Presto search routes
Tasks
- Review current logging in regular search routes (
components/webui/server/src/routes/api/search/) - Review logging in Presto search routes (
components/webui/server/src/routes/api/presto-search/) - Remove unnecessary log statements that duplicate information already provided by Fastify
- Ensure any truly necessary debugging information is retained
References
- Original discussion: feat(webui): Add logs to presto search routes. #1275
- Requested by: @davemarco in feat(webui): Add logs to presto search routes. #1275 (comment)
Metadata
Metadata
Assignees
Labels
No labels