This is an automated email from the ASF dual-hosted git repository. reschke pushed a commit to branch OAK-6759 in repository https://gitbox.apache.org/repos/asf/jackrabbit-oak.git
commit d6eb069c5c4a1303b238e6f2fc36b5ad73addc11 Author: Julian Reschke <julian.resc...@gmx.de> AuthorDate: Mon Sep 11 11:40:27 2023 +0100 OAK-6759: Convert oak-blob-cloud-azure to OSGi R6 annotations --- oak-blob-cloud-azure/pom.xml | 4 ++-- .../cloud/azure/blobstorage/AbstractAzureDataStoreService.java | 8 ++++++++ .../oak/blob/cloud/azure/blobstorage/AzureDataStoreService.java | 6 +++--- 3 files changed, 13 insertions(+), 5 deletions(-) diff --git a/oak-blob-cloud-azure/pom.xml b/oak-blob-cloud-azure/pom.xml index 6900a0fba8..91d8d9a319 100644 --- a/oak-blob-cloud-azure/pom.xml +++ b/oak-blob-cloud-azure/pom.xml @@ -76,8 +76,8 @@ <scope>provided</scope> </dependency> <dependency> - <groupId>org.apache.felix</groupId> - <artifactId>org.apache.felix.scr.annotations</artifactId> + <groupId>org.osgi</groupId> + <artifactId>org.osgi.service.component.annotations</artifactId> <scope>provided</scope> </dependency> diff --git a/oak-blob-cloud-azure/src/main/java/org/apache/jackrabbit/oak/blob/cloud/azure/blobstorage/AbstractAzureDataStoreService.java b/oak-blob-cloud-azure/src/main/java/org/apache/jackrabbit/oak/blob/cloud/azure/blobstorage/AbstractAzureDataStoreService.java index 6e4a325e04..761c4784d1 100644 --- a/oak-blob-cloud-azure/src/main/java/org/apache/jackrabbit/oak/blob/cloud/azure/blobstorage/AbstractAzureDataStoreService.java +++ b/oak-blob-cloud-azure/src/main/java/org/apache/jackrabbit/oak/blob/cloud/azure/blobstorage/AbstractAzureDataStoreService.java @@ -19,10 +19,12 @@ package org.apache.jackrabbit.oak.blob.cloud.azure.blobstorage; +import org.osgi.service.component.annotations.Reference; import org.apache.jackrabbit.core.data.DataStore; import org.apache.jackrabbit.core.data.DataStoreException; import org.apache.jackrabbit.oak.plugins.blob.AbstractSharedCachingDataStore; import org.apache.jackrabbit.oak.plugins.blob.datastore.AbstractDataStoreService; +import org.apache.jackrabbit.oak.stats.StatisticsProvider; import org.osgi.framework.Constants; import org.osgi.framework.ServiceRegistration; import org.osgi.service.component.ComponentContext; @@ -37,6 +39,12 @@ public abstract class AbstractAzureDataStoreService extends AbstractDataStoreSer private ServiceRegistration delegateReg; + // OAK-6759: we redeclare "statisticsProvider" (from + // AbstractDataStoreService) so that we do not gave to rely on annotation + // inheritance + @Reference + private StatisticsProvider statisticsProvider; + @Override protected DataStore createDataStore(ComponentContext context, Map<String, Object> config) { Properties properties = new Properties(); diff --git a/oak-blob-cloud-azure/src/main/java/org/apache/jackrabbit/oak/blob/cloud/azure/blobstorage/AzureDataStoreService.java b/oak-blob-cloud-azure/src/main/java/org/apache/jackrabbit/oak/blob/cloud/azure/blobstorage/AzureDataStoreService.java index bdc7abe320..5fa57e88fb 100644 --- a/oak-blob-cloud-azure/src/main/java/org/apache/jackrabbit/oak/blob/cloud/azure/blobstorage/AzureDataStoreService.java +++ b/oak-blob-cloud-azure/src/main/java/org/apache/jackrabbit/oak/blob/cloud/azure/blobstorage/AzureDataStoreService.java @@ -19,10 +19,10 @@ package org.apache.jackrabbit.oak.blob.cloud.azure.blobstorage; -import org.apache.felix.scr.annotations.Component; -import org.apache.felix.scr.annotations.ConfigurationPolicy; +import org.osgi.service.component.annotations.Component; +import org.osgi.service.component.annotations.ConfigurationPolicy; -@Component(policy = ConfigurationPolicy.REQUIRE, name = AzureDataStoreService.NAME) +@Component(configurationPolicy = ConfigurationPolicy.REQUIRE, name = AzureDataStoreService.NAME) public class AzureDataStoreService extends AbstractAzureDataStoreService { public static final String NAME = "org.apache.jackrabbit.oak.plugins.blob.datastore.AzureDataStore"; }