From 868c883e929331031895ee6d94fa12ec2e4ddf53 Mon Sep 17 00:00:00 2001 From: Robert Stupp Date: Tue, 19 Jul 2022 21:06:12 +0200 Subject: [PATCH] Clarify --- .../java/org/apache/iceberg/nessie/NessieExtCatalog.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/clients/iceberg-views/src/main/java/org/apache/iceberg/nessie/NessieExtCatalog.java b/clients/iceberg-views/src/main/java/org/apache/iceberg/nessie/NessieExtCatalog.java index 822569f1cb6..9fa27b56900 100644 --- a/clients/iceberg-views/src/main/java/org/apache/iceberg/nessie/NessieExtCatalog.java +++ b/clients/iceberg-views/src/main/java/org/apache/iceberg/nessie/NessieExtCatalog.java @@ -105,7 +105,10 @@ public void setConf(Configuration conf) { } public void refresh() throws NessieNotFoundException { - // TODO only used in a single test + // TODO This NessieExtCatalog is a *hack* to get some Iceberg view support into Nessie. + // The refactoring of `NessieCatalog` (https://github.com/apache/iceberg/pull/4491) + // removed the explicit ability to refresh the catalog. Client code should actually NEVER + // call this function explicitly. // tableCatalog.refresh(); viewCatalog.refresh(); }