URLRepository

Author: nicktindall

modules/repository-url/src/main/java/org/elasticsearch/plugin/repository/url/URLRepositoryPlugin.java

@@ -63,7 +63,8 @@ public Map<String, Repository.Factory> getRepositories(
                 clusterService,
                 bigArrays,
                 recoverySettings,
-                httpClientFactory.updateAndGet(factory -> factory == null ? new URLHttpClient.Factory() : factory)
+                httpClientFactory.updateAndGet(factory -> factory == null ? new URLHttpClient.Factory() : factory),
+                snapshotMetrics
             )
         );
     }

modules/repository-url/src/main/java/org/elasticsearch/repositories/url/URLRepository.java

@@ -101,18 +101,10 @@ public URLRepository(
         ClusterService clusterService,
         BigArrays bigArrays,
         RecoverySettings recoverySettings,
-        URLHttpClient.Factory httpClientFactory
+        URLHttpClient.Factory httpClientFactory,
+        SnapshotMetrics snapshotMetrics
     ) {
-        super(
-            projectId,
-            metadata,
-            namedXContentRegistry,
-            clusterService,
-            bigArrays,
-            recoverySettings,
-            BlobPath.EMPTY,
-            SnapshotMetrics.NOOP
-        );
+        super(projectId, metadata, namedXContentRegistry, clusterService, bigArrays, recoverySettings, BlobPath.EMPTY, snapshotMetrics);
 
         if (URL_SETTING.exists(metadata.settings()) == false && REPOSITORIES_URL_SETTING.exists(environment.settings()) == false) {
             throw new RepositoryException(metadata.name(), "missing url");

modules/repository-url/src/test/java/org/elasticsearch/repositories/url/URLRepositoryTests.java

@@ -19,6 +19,7 @@
 import org.elasticsearch.env.TestEnvironment;
 import org.elasticsearch.indices.recovery.RecoverySettings;
 import org.elasticsearch.repositories.RepositoryException;
+import org.elasticsearch.repositories.SnapshotMetrics;
 import org.elasticsearch.repositories.blobstore.BlobStoreTestUtil;
 import org.elasticsearch.test.ESTestCase;
 import org.elasticsearch.xcontent.NamedXContentRegistry;
@@ -45,7 +46,8 @@ private URLRepository createRepository(Settings baseSettings, RepositoryMetadata
             BlobStoreTestUtil.mockClusterService(),
             MockBigArrays.NON_RECYCLING_INSTANCE,
             new RecoverySettings(baseSettings, new ClusterSettings(baseSettings, ClusterSettings.BUILT_IN_CLUSTER_SETTINGS)),
-            mock(URLHttpClient.Factory.class)
+            mock(URLHttpClient.Factory.class),
+            SnapshotMetrics.NOOP
         );
         assertThat(repository.getProjectId(), equalTo(projectId));
         return repository;