refactor names

Author: sfc-gh-prsingh

core/src/main/java/org/apache/iceberg/RestTable.java renamed to core/src/main/java/org/apache/iceberg/RESTTable.java

@@ -25,15 +25,15 @@
 import org.apache.iceberg.rest.RESTClient;
 import org.apache.iceberg.rest.ResourcePaths;
 
-public class RestTable extends BaseTable {
+public class RESTTable extends BaseTable {
   private final RESTClient client;
   private final String path;
   private final Supplier<Map<String, String>> headers;
   private final MetricsReporter reporter;
   private final ResourcePaths resourcePaths;
   private final TableIdentifier tableIdentifier;
 
-  public RestTable(
+  public RESTTable(
       TableOperations ops,
       String name,
       MetricsReporter reporter,
@@ -57,7 +57,7 @@ public TableScan newScan() {
     // correct snapshotId to use for point in time cases. When looking at spark
     // it seems it follows similar approach, see class SparkDistributedDataScan
 
-    return new RestTableScan(
+    return new RESTTableScan(
         this,
         schema(),
         ImmutableTableScanContext.builder().metricsReporter(reporter).build(),

core/src/main/java/org/apache/iceberg/RestTableScan.java renamed to core/src/main/java/org/apache/iceberg/RESTTableScan.java

@@ -36,7 +36,7 @@
 import org.apache.iceberg.types.Types;
 import org.apache.iceberg.util.ParallelIterable;
 
-class RestTableScan extends DataTableScan {
+class RESTTableScan extends DataTableScan {
   private final RESTClient client;
   private final String path;
   private final Supplier<Map<String, String>> headers;
@@ -48,7 +48,7 @@ class RestTableScan extends DataTableScan {
   // TODO revisit if this property should be configurable
   private static final int FETCH_PLANNING_SLEEP_DURATION_MS = 1000;
 
-  RestTableScan(
+  RESTTableScan(
       Table table,
       Schema schema,
       TableScanContext context,
@@ -71,7 +71,7 @@ class RestTableScan extends DataTableScan {
   @Override
   protected TableScan newRefinedScan(
       Table refinedTable, Schema refinedSchema, TableScanContext refinedContext) {
-    return new RestTableScan(
+    return new RESTTableScan(
         refinedTable,
         refinedSchema,
         refinedContext,

core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java

@@ -36,7 +36,7 @@
 import org.apache.iceberg.MetadataTableUtils;
 import org.apache.iceberg.MetadataUpdate;
 import org.apache.iceberg.PartitionSpec;
-import org.apache.iceberg.RestTable;
+import org.apache.iceberg.RESTTable;
 import org.apache.iceberg.Schema;
 import org.apache.iceberg.SortOrder;
 import org.apache.iceberg.Table;
@@ -259,10 +259,10 @@ public void initialize(String name, Map<String, String> unresolved) {
     this.reporter = CatalogUtil.loadMetricsReporter(mergedProps);
 
     this.reportingViaRestEnabled =
-            PropertyUtil.propertyAsBoolean(
-                    mergedProps,
-                    RESTCatalogProperties.METRICS_REPORTING_ENABLED,
-                    RESTCatalogProperties.METRICS_REPORTING_ENABLED_DEFAULT);
+        PropertyUtil.propertyAsBoolean(
+            mergedProps,
+            RESTCatalogProperties.METRICS_REPORTING_ENABLED,
+            RESTCatalogProperties.METRICS_REPORTING_ENABLED_DEFAULT);
     this.restServerPlanningEnabled =
         PropertyUtil.propertyAsBoolean(mergedProps, REST_SERVER_PLANNING_ENABLED, false);
     super.initialize(name, mergedProps);
@@ -469,7 +469,7 @@ public Table loadTable(SessionContext context, TableIdentifier identifier) {
 
     trackFileIO(ops);
 
-    RestTable restTable = tableSupportsRemoteScanPlanning(ops, finalIdentifier, tableClient);
+    RESTTable restTable = tableSupportsRemoteScanPlanning(ops, finalIdentifier, tableClient);
     if (restTable != null) {
       return restTable;
     }
@@ -486,13 +486,13 @@ public Table loadTable(SessionContext context, TableIdentifier identifier) {
     return table;
   }
 
-  private RestTable tableSupportsRemoteScanPlanning(
+  private RESTTable tableSupportsRemoteScanPlanning(
       TableOperations ops, TableIdentifier finalIdentifier, RESTClient restClient) {
     if (ops.current().properties().containsKey(REST_TABLE_SCAN_PLANNING_PROPERTY)) {
       boolean tableSupportsRemotePlanning =
           ops.current().propertyAsBoolean(REST_TABLE_SCAN_PLANNING_PROPERTY, false);
       if (tableSupportsRemotePlanning && restServerPlanningEnabled) {
-        return new RestTable(
+        return new RESTTable(
             ops,
             fullTableName(finalIdentifier),
             metricsReporter(paths.metrics(finalIdentifier), restClient),
@@ -573,7 +573,7 @@ public Table registerTable(
 
     trackFileIO(ops);
 
-    RestTable restTable = tableSupportsRemoteScanPlanning(ops, ident, tableClient);
+    RESTTable restTable = tableSupportsRemoteScanPlanning(ops, ident, tableClient);
     if (restTable != null) {
       return restTable;
     }
@@ -837,7 +837,7 @@ public Table create() {
 
       trackFileIO(ops);
 
-      RestTable restTable = tableSupportsRemoteScanPlanning(ops, ident, tableClient);
+      RESTTable restTable = tableSupportsRemoteScanPlanning(ops, ident, tableClient);
       if (restTable != null) {
         return restTable;
       }