[JDBC] Implement DataSource login timeout

Offen

#1,267 erstellt am 28.8.2026 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 4/5
Geschätzter Aufwand: 3-5 Tage
Anfängerfreundlichkeit: 66/100

Issue-Typ: Feature
Klarheit: Größtenteils klar
Aktivitätsstatus: Aktiv
Tech-Stack: java
Bereich: databases

Rechercherichtung

Beginne mit ArrowFlightJdbcDataSource.setLoginTimeout(int), getLoginTimeout() und dem DataSource.getConnection()-Pfad; vergleiche sie mit dem bestehenden connectTimeoutMs-Verhalten. Füge Abdeckung für den Standardwert, Setter/Getter, die Durchsetzung des Timeouts, SQLTimeoutException und die Bereinigung teilweise erstellter Ressourcen hinzu. Als abgeschlossen gilt die Aufgabe, wenn 0 das Standardverhalten beibehält und Timeouts ungleich null für den Login fehlgeschlagene Versuche ohne Leaks abbrechen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Describe the enhancement requested

ArrowFlightJdbcDataSource.setLoginTimeout(int) currently throws SQLFeatureNotSupportedException, and getLoginTimeout() always returns 0.

Implement the JDBC DataSource login-timeout contract:

Store and return the timeout configured for each ArrowFlightJdbcDataSource instance.
Apply a nonzero timeout to connection establishment through DataSource.getConnection().
Preserve the JDBC semantics of zero: use the default system timeout if one exists; otherwise, do not impose a timeout.
If the timeout expires, attempt to cancel the connection attempt and throw SQLTimeoutException.
Clean up any allocator, channel, or partially constructed client when the attempt is cancelled or times out.
Add tests for the default value, setter/getter behavior, timeout enforcement, exception type, and resource cleanup.

This is distinct from #494/#495. The existing connectTimeoutMs property configures Netty socket connection attempts; the JDBC login timeout applies to the overall database connection attempt exposed by DataSource.getConnection().

Context: #1027 and the follow-up discussion.

Related DriverManager work: https://github.com/apache/arrow-java/issues/1268