nastra commented on code in PR #12563:
URL: https://github.com/apache/iceberg/pull/12563#discussion_r2175102382
##########
core/src/main/java/org/apache/iceberg/util/ThreadPools.java:
##########
@@ -82,6 +87,20 @@ public static ExecutorService getDeleteWorkerPool() {
return DELETE_WORKER_POOL;
}
+ /**
+ * A shared {@link ScheduledExecutorService} that REST catalogs can use for
refreshing their
+ * authentication data.
+ */
+ public static ScheduledExecutorService getAuthRefreshPool() {
Review Comment:
```suggestion
public static ScheduledExecutorService authRefreshPool() {
```
I'm aware that we already have some methods in this class with a `get`
prefix, but those methods have been added at the very beginning and haven't
been renamed
--
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]
