patnaikankit opened a new issue, #18442:
URL: https://github.com/apache/iceberg/issues/18442

   ### Apache Iceberg version
   
   None
   
   ### Query engine
   
   None
   
   ### Please describe the bug 🐞
   
   ## Description
   
   `BigQueryMetastoreClientImpl` configures retries for `ConnectException` and 
`UnknownHostException`, but its [[exception 
handler](https://github.com/apache/iceberg/blob/0f3131f02fc2560bd122f3b323d13ea935892eb3/bigquery/src/main/java/org/apache/iceberg/gcp/bigquery/BigQueryMetastoreClientImpl.java#L114-L122)](https://github.com/apache/iceberg/blob/0f3131f02fc2560bd122f3b323d13ea935892eb3/bigquery/src/main/java/org/apache/iceberg/gcp/bigquery/BigQueryMetastoreClientImpl.java#L114-L122)
 aborts on `RuntimeException`. The [[exception handler 
interceptor](https://github.com/apache/iceberg/blob/0f3131f02fc2560bd122f3b323d13ea935892eb3/bigquery/src/main/java/org/apache/iceberg/gcp/bigquery/BigQueryMetastoreClientImpl.java#L93-L103)](https://github.com/apache/iceberg/blob/0f3131f02fc2560bd122f3b323d13ea935892eb3/bigquery/src/main/java/org/apache/iceberg/gcp/bigquery/BigQueryMetastoreClientImpl.java#L93-L103)
 does not inspect wrapped causes.
   
   Meanwhile, 
[`[internalCreate(Table)](https://github.com/apache/iceberg/blob/0f3131f02fc2560bd122f3b323d13ea935892eb3/bigquery/src/main/java/org/apache/iceberg/gcp/bigquery/BigQueryMetastoreClientImpl.java#L349-L365)`](https://github.com/apache/iceberg/blob/0f3131f02fc2560bd122f3b323d13ea935892eb3/bigquery/src/main/java/org/apache/iceberg/gcp/bigquery/BigQueryMetastoreClientImpl.java#L349-L365)
 catches `IOException` from the BigQuery table insert and wraps it in 
`RuntimeIOException`.
   
   As a result, a transient connection or DNS failure during table creation 
fails immediately instead of following the configured retry policy.
   
   ### Expected Behavior
   
   Wrapped `ConnectException` and `UnknownHostException` causes should follow 
the configured retry policy, while unrelated runtime exceptions remain 
non-retryable.
   
   This report concerns retry classification only; it does not claim that table 
creation is retried after an ambiguous successful insert.
   
   ### Willingness to contribute
   
   - [x] I can contribute a fix for this bug independently
   - [ ] I would be willing to contribute a fix for this bug with guidance from 
the Iceberg community
   - [ ] I cannot contribute a fix for this bug at this time


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to