search

prestodb / presto

The official home of the Presto distributed SQL query engine for big data
http://prestodb.io
Apache License 2.0
16.06k stars 5.38k forks source link

Allow configuring HiveTableOperations retry parameters #21967

Closed ZacBlanco closed 3 months ago

ZacBlanco commented 9 months ago

Expected Behavior or Use Case

Currently the HiveTableOperations class uses a set of static parameters for retries when refreshing the table metadata. It would be good to make them configurable from the iceberg connector properties file.

https://github.com/prestodb/presto/blob/master/presto-iceberg/src/main/java/com/facebook/presto/iceberg/HiveTableOperations.java#L387-L392

Presto Component, Service, or Connector

Iceberg Connector

Possible Implementation

Add new field(s) to the Iceberg connector configuration.

Context

It would be useful for users in the case they ever need to edit the retry configuration for their cluster or for decrease test times.

steveburnett commented 9 months ago

When this is worked on, please document new properties in Configuration Properties of the Iceberg Connector doc.

BVK21 commented 3 months ago

Hello @ZacBlanco can you please assign this to me

ZacBlanco commented 3 months ago

@BVK21 this was actually closed already by #23174