googleapis / googleapis/google-cloud-java

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

Ouverte
#12,164 0 commentaires 1 réaction 0 personnes assignées Voir sur GitHub
api: bigquery priority: p2 type: feature request
Langage dominant
Java
Étoiles
2.1k
Forks
1.2k
Merge moyen
1 j 23 h
PR mergées (30 j)
154

Description

**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)

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.