googleapis / googleapis/google-cloud-java

[java-bigquery] Don't throw exception in BigQueryResultSet.close()

Offen
#12,164 0 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen
api: bigquery priority: p2 type: feature request
Vorherrschende Sprache
Java
Sterne
2.1k
Forks
1.2k
Ø Merge
1 T. 23 Std.
Gemergte PRs (30 T.)
154

Beschreibung

**Is your feature request related to a problem? Please describe.**
`Connection.execute*()` nethods return BigQueryResult, which contains BigQueryResultSet the inheritor of ResultSet.
Most of the methods of BigQueryResultSet are not implemented including `close()`.
Unfortunately, IntelliJ Idea strongly suggests wrapping `ResultSet` into `try-with-resources` block, which we can't do because it will trow an exception.

**Describe the solution you'd like**
Don't throw 'Not implemented' exception in the close method.

**Describe alternatives you've considered**
Fully implement `close` method to free underlying resources.

**Additional context**
IntelliJ Idea's waring:
![image](https://github.com/googleapis/google-cloud-java/assets/4300612/c946c9bb-044d-40cd-9019-d16dd3a29d5a)

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.