Ma77Ball opened a new issue, #4852:
URL: https://github.com/apache/texera/issues/4852

   ### Task Summary
   
   ### Summary
   
   `SqlServer.scala` currently opens a fresh PostgreSQL connection (TCP + 
SCRAM-SHA-256 auth) on every query. Add a HikariCP connection pool so queries 
reuse pre-authenticated connections instead.
   
   ### Tasks
   
   - [ ] Add HikariCP dependency if not already present
   - [ ] Replace direct `DSL.using(...)` setup in `SqlServer.scala` with a 
`HikariDataSource`-backed `DSLContext`
   - [ ] Configure pool defaults (max size, min idle, timeouts)
   - [ ] Add `close()` and update `clearInstance()` to call it so tests don't 
leak pool threads
   - [ ] Verify existing tests pass
   
   ### Acceptance Criteria
   
   - All queries go through the pool
   - Public API of `SqlServer` is unchanged
   - Existing tests pass
   
   ### Task Type
   
   - [x] Refactor / Cleanup
   - [ ] DevOps / Deployment / CI
   - [ ] Testing / QA
   - [ ] Documentation
   - [x] Performance
   - [ ] Other


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to