Skip to content

Add OSIV connection pool test endpoint for dev testing#1253

Closed
bhou2 wants to merge 5 commits into
masterfrom
dev-snapshot
Closed

Add OSIV connection pool test endpoint for dev testing#1253
bhou2 wants to merge 5 commits into
masterfrom
dev-snapshot

Conversation

@bhou2
Copy link
Copy Markdown
Contributor

@bhou2 bhou2 commented Jan 30, 2026

Add test controller and local config to demonstrate OSIV behavior:

  • Test endpoint executes DB query then polls connection pool for 60s
  • Small connection pool (5) with 1s timeout to show exhaustion
  • Toggle spring.jpa.open-in-view to test behavior

Usage:
./gradlew :genie-app:bootRun --args='--spring.config.additional-location=file:genie-app/src/main/resources/application-local.yml' curl http://localhost:8081/api/v3/test/osiv/simple-test

Expected:

  • OSIV enabled: Connection held for full 60s
  • OSIV disabled: Connection released immediately after query

For dev testing only - DO NOT MERGE to master.

Binbing Hou added 5 commits January 30, 2026 12:03
Add test controller and local config to demonstrate OSIV behavior:
- Test endpoint executes DB query then polls connection pool for 60s
- Small connection pool (5) with 1s timeout to show exhaustion
- Toggle spring.jpa.open-in-view to test behavior

Usage:
  ./gradlew :genie-app:bootRun --args='--spring.config.additional-location=file:genie-app/src/main/resources/application-local.yml'
  curl http://localhost:8081/api/v3/test/osiv/simple-test

Expected:
  - OSIV enabled: Connection held for full 60s
  - OSIV disabled: Connection released immediately after query

For dev testing only - DO NOT MERGE to master.
- Add fetch-depth: 0 to checkout action
- Ensures nebula plugin infers correct version (5.2.0-SNAPSHOT)
- Fixes version defaulting to 0.1.0-SNAPSHOT in CI
- Remove dockerPush from dev-snapshot workflow
- Avoids Docker Hub credential errors
- JAR artifacts and snapshots still published
- Docker images not needed for Netflix internal deployment
- Replace deprecated openjdk:17-jdk-slim (no longer available)
- Use eclipse-temurin:17-jdk (official OpenJDK successor)
- Fix Docker warnings:
  - FROM (uppercase) instead of from
  - LABEL maintainer instead of deprecated MAINTAINER
@bhou2 bhou2 closed this Jan 31, 2026
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