Port system-database migrations 115-120, and require the last migration

[devhawk]

Python, TypeScript and Go rebuilt three hot system-database indexes in migrations 115-120 (py#857, ts#1367, go#476). Java stopped at 114. The migration version is one counter that every SDK shares, so a database first migrated by Java never got these indexes, even when other SDKs' apps share it.

Migrations 115-120. The DDL is copied verbatim from Python, at the same numbers:

	Creates	Drops
115 / 116	idx_workflow_status_in_flight_v2	idx_workflow_status_in_flight
117 / 118	idx_workflow_status_partition_dequeue_v3	idx_workflow_status_partition_dequeue_v2
119 / 120	idx_operation_outputs_completed_at_function_name_v2	idx_operation_outputs_completed_at_function_name

Each new index carries application_name as an INCLUDE column, so app-scoped counts and dequeues run index-only. All six run online (CONCURRENTLY, outside a transaction) on Postgres and transactionally on CockroachDB, like the other index migrations. On a Postgres copy of the schema, EXPLAIN shows the in-flight count, the dequeue, the partitioned head probe and loose scan, and the step aggregates each using the matching new index.

Required schema version. MigrationManager.MINIMUM_SYSDB_VERSION, a hand-kept floor of 111, is replaced by latestMigrationVersion(), the SDK's last migration (now 120). An executor with migrate disabled, and every DBOSClient, now requires every migration this SDK defines. A schema migrated ahead by a newer SDK is still accepted.

This removes a public constant. The next release adds migrations that new code depends on (rewind, starting at 121), so the floor would have had to rise to the last migration then anyway.

Tests. A new test migrates a database to 114 and then to 120, and checks that the old indexes are dropped, the new ones exist with application_name as their only INCLUDE column (on CockroachDB, where it is reported as STORING), and a re-run is a no-op. Others cover a schema ahead of the SDK and the migration count being the same in every listen/notify and CockroachDB configuration. MigrationManagerTest and ClientSysDbVersionTest pass locally on Postgres and CockroachDB.

Fixes #573

🤖 Generated with Claude Code

[Copilot]

Copilot review overview

🟢 Approval recommended

The migrations, validation changes, and cross-database tests consistently implement the stated requirements.

Review effort: Balanced
Findings: None

What changed in this PR

Ports shared system-database migrations 115–120 and requires schemas to reach the SDK’s latest migration.

Changes:

• Rebuilds three indexes with application_name included.
• Uses the latest migration as the required schema version.
• Adds PostgreSQL and CockroachDB migration coverage.

File	Description
MigrationManager.java	Adds migrations 115–120 and latest-version validation.
DBOSClient.java	Updates schema-version documentation.
DBOS.java	Clarifies migration requirements.
QueuesDAO.java	Updates index references in documentation.
MigrationManagerTest.java	Tests index upgrades and version validation.
ClientSysDbVersionTest.java	Tests client enforcement of the latest version.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.