Skip to content

refactor: modernize query service#122

Open
BK1031 wants to merge 3 commits intomainfrom
bk1031/query-refactor
Open

refactor: modernize query service#122
BK1031 wants to merge 3 commits intomainfrom
bk1031/query-refactor

Conversation

@BK1031
Copy link
Contributor

@BK1031 BK1031 commented Mar 14, 2026

  • Migrate from Poetry to uv (hatchling build backend)
  • Upgrade Python 3.10 → 3.12
  • Switch from MySQL (pymysql) to PostgreSQL (psycopg2)
  • Add mapache-py and rincon library dependencies, replace inline RinconService
  • Add Dockerfile.dev with uvicorn --reload hot reload for docker-compose
  • Use FastAPI lifespan for startup initialization (db, rincon, auth)
  • Fix SQL injection in query_signals with parameterized queries
  • Replace per-signal DataFrame filtering with single pivot_table call
  • Deduplicate merge_to_smallest/merge_to_largest into unified merge_signals
  • Fix df.equals() identity bug that could silently drop duplicate signals
  • Handle UTC timestamps consistently at the query layer

- Migrate from Poetry to uv with hatchling build backend
- Upgrade Python from 3.10 to 3.12
- Switch database driver from MySQL (pymysql) to PostgreSQL (psycopg2)
- Add mapache-py and rincon library dependencies
- Replace inline RinconService with rincon library client
- Add Dockerfile.dev with uvicorn hot reload for docker-compose
- Use FastAPI lifespan for startup initialization
- Fix SQL injection in query_signals with parameterized queries
- Replace per-signal DataFrame filtering with single pivot_table call
- Deduplicate merge_to_smallest/merge_to_largest into merge_signals
- Fix df.equals() identity bug that could silently drop signals
- Handle UTC timestamps consistently at the query layer
@netlify
Copy link

netlify bot commented Mar 14, 2026

Deploy Preview for gr-mapache canceled.

Name Link
🔨 Latest commit 3cc8c3c
🔍 Latest deploy log https://app.netlify.com/projects/gr-mapache/deploys/69b73c4ede66670008112926

BK1031 added 2 commits March 14, 2026 23:08
Replace uuid4 with gr-ulid prefixed IDs (qlog_, qtk_) to match
the convention used across other Mapache services.
When SKIP_AUTH_CHECK=true, skip Sentinel JWKS initialization and
bypass auth checks in all routes with a mock user, matching the
pattern used in the auth service.
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