-
Notifications
You must be signed in to change notification settings - Fork 93
Don't recommend catching java.lang.Exception #309
Copy link
Copy link
Closed
Labels
api: storageIssues related to the googleapis/java-storage API.Issues related to the googleapis/java-storage API.documentationImprovements or additions to documentationImprovements or additions to documentationgood first issueThis issue is a good place to started contributing to this repository.This issue is a good place to started contributing to this repository.help wantedWe'd love to have community involvement on this issue.We'd love to have community involvement on this issue.priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: docsImprovement to the documentation for an API.Improvement to the documentation for an API.
Metadata
Metadata
Assignees
Labels
api: storageIssues related to the googleapis/java-storage API.Issues related to the googleapis/java-storage API.documentationImprovements or additions to documentationImprovements or additions to documentationgood first issueThis issue is a good place to started contributing to this repository.This issue is a good place to started contributing to this repository.help wantedWe'd love to have community involvement on this issue.We'd love to have community involvement on this issue.priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: docsImprovement to the documentation for an API.Improvement to the documentation for an API.
https://googleapis.dev/java/google-cloud-storage/latest/com/google/cloud/storage/Blob.html#writer-com.google.cloud.storage.Storage.BlobWriteOption...-
sample code includes a catch block for java.lang.Exception. Per Effective Java et al, this is an antipattern. The sample code should catch only the actual exceptions expected to be thrown.