fix: fix compilation by using DEFAULT_COLLECTION

Author: benbellick

core/src/main/java/io/substrait/extension/ExtensionCollector.java

@@ -30,7 +30,7 @@ private String getUriFromUrn(String urn) {
   }
 
   public ExtensionCollector() {
-    this(SimpleExtension.loadDefaults());
+    this(DefaultExtensionCatalog.DEFAULT_COLLECTION);
   }
 
   public ExtensionCollector(SimpleExtension.ExtensionCollection extensionCollection) {

core/src/main/java/io/substrait/extension/ImmutableExtensionLookup.java

@@ -35,7 +35,7 @@ public static class Builder {
     private final SimpleExtension.ExtensionCollection extensionCollection;
 
     public Builder() {
-      this.extensionCollection = SimpleExtension.loadDefaults();
+      this.extensionCollection = DefaultExtensionCatalog.DEFAULT_COLLECTION;
     }
 
     public Builder(SimpleExtension.ExtensionCollection extensionCollection) {

core/src/main/java/io/substrait/plan/PlanProtoConverter.java

@@ -1,5 +1,6 @@
 package io.substrait.plan;
 
+import io.substrait.extension.DefaultExtensionCatalog;
 import io.substrait.extension.ExtensionCollector;
 import io.substrait.extension.SimpleExtension;
 import io.substrait.proto.Plan;
@@ -16,7 +17,7 @@ public class PlanProtoConverter {
   private final SimpleExtension.ExtensionCollection extensionCollection;
 
   public PlanProtoConverter() {
-    this(SimpleExtension.loadDefaults());
+    this(DefaultExtensionCatalog.DEFAULT_COLLECTION);
   }
 
   public PlanProtoConverter(SimpleExtension.ExtensionCollection extensionCollection) {

core/src/test/java/io/substrait/extension/UriUrnMigrationEndToEndTest.java

@@ -26,9 +26,6 @@
  */
 public class UriUrnMigrationEndToEndTest {
 
-  private final SimpleExtension.ExtensionCollection defaultExtensions =
-      SimpleExtension.loadDefaults();
-
   /** Load a proto Plan from a JSON resource file using JsonFormat */
   private Plan loadPlanFromJson(String resourcePath) throws IOException {
     try (InputStream inputStream = getClass().getClassLoader().getResourceAsStream(resourcePath)) {
@@ -81,10 +78,12 @@ public void testUriUrnMigrationEndToEnd() throws IOException {
       Plan inputPlan = loadPlanFromJson(inputPath);
       Plan expectedPlan = loadPlanFromJson(expectedPath);
 
-      ProtoPlanConverter protoToPojo = new ProtoPlanConverter(defaultExtensions);
+      ProtoPlanConverter protoToPojo =
+          new ProtoPlanConverter(DefaultExtensionCatalog.DEFAULT_COLLECTION);
       io.substrait.plan.Plan pojoPlan = protoToPojo.from(inputPlan);
 
-      PlanProtoConverter pojoToProto = new PlanProtoConverter(defaultExtensions);
+      PlanProtoConverter pojoToProto =
+          new PlanProtoConverter(DefaultExtensionCatalog.DEFAULT_COLLECTION);
       Plan actualPlan = pojoToProto.toProto(pojoPlan);
 
       assertEquals(expectedPlan, actualPlan);
@@ -95,7 +94,8 @@ public void testUriUrnMigrationEndToEnd() throws IOException {
   public void testUnresolvableUriThrowsException() throws IOException {
     Plan inputPlan = loadPlanFromJson("uri-urn-migration/unresolvable-uri-plan.json");
 
-    ProtoPlanConverter protoToPojo = new ProtoPlanConverter(defaultExtensions);
+    ProtoPlanConverter protoToPojo =
+        new ProtoPlanConverter(DefaultExtensionCatalog.DEFAULT_COLLECTION);
 
     IllegalStateException exception =
         assertThrows(