You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-dev@jackrabbit.apache.org by Christian Schneider <ch...@die-schneider.net> on 2017/10/30 08:57:26 UTC

Looking for small task starting in OAK .. DS conversion?

Hi all,

as I am just starting to work on OAK I am looking for a small task.
I found that there are still some components that use the old felix scr
annotations.
Does it make sense that I look into converting these to the DS ones so we
can remove support for felix scr in the build?

I have listed the classes below.
The main issue I see with the migration is that OAK uses the meta type
support of felix scr which is quite different to what DS 1.3 provides. So I
would need to migrate from the property based meta type descriptions to the
type safe ones of the DS 1.3 metatype support.

Anyway I would provide one module per PR so the reviewer does not have to
review one big commit at once.

Best
Christian

-- 
-- 
Christian Schneider
http://www.liquid-reality.de
<https://owa.talend.com/owa/redir.aspx?C=3aa4083e0c744ae1ba52bd062c5a7e46&URL=http%3a%2f%2fwww.liquid-reality.de>

Computer Scientist
http://www.adobe.com


---

oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/DefaultSyncConfigImpl.java:import
org.apache.felix.scr.annotations.Component;

oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/DefaultSyncHandler.java:import
org.apache.felix.scr.annotations.Component;

oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/ExternalIDPManagerImpl.java:import
org.apache.felix.scr.annotations.Component;

oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/ExternalLoginModuleFactory.java:import
org.apache.felix.scr.annotations.Component;

oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/principal/ExternalPrincipalConfiguration.java:import
org.apache.felix.scr.annotations.Component;

oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/SyncManagerImpl.java:import
org.apache.felix.scr.annotations.Component;

oak-auth-ldap/src/main/java/org/apache/jackrabbit/oak/security/authentication/ldap/impl/LdapIdentityProvider.java:import
org.apache.felix.scr.annotations.Component;

oak-auth-ldap/src/main/java/org/apache/jackrabbit/oak/security/authentication/ldap/impl/LdapProviderConfig.java:import
org.apache.felix.scr.annotations.Component;

oak-authorization-cug/src/main/java/org/apache/jackrabbit/oak/spi/security/authorization/cug/impl/CugConfiguration.java:import
org.apache.felix.scr.annotations.Component;

oak-authorization-cug/src/main/java/org/apache/jackrabbit/oak/spi/security/authorization/cug/impl/CugExcludeImpl.java:import
org.apache.felix.scr.annotations.Component;

oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/osgi/FileBlobStoreService.java:import
org.apache.felix.scr.annotations.Component;

oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/osgi/SplitBlobStoreService.java:import
org.apache.felix.scr.annotations.Component;

oak-blob-cloud/src/main/java/org/apache/jackrabbit/oak/blob/cloud/s3/AbstractS3DataStoreService.java:import
org.apache.felix.scr.annotations.Component;

oak-blob-cloud/src/main/java/org/apache/jackrabbit/oak/blob/cloud/s3/S3DataStoreService.java:import
org.apache.felix.scr.annotations.Component;

oak-blob-cloud/src/main/java/org/apache/jackrabbit/oak/blob/cloud/s3/SharedS3DataStoreService.java:import
org.apache.felix.scr.annotations.Component;

oak-blob-cloud-azure/src/main/java/org/apache/jackrabbit/oak/blob/cloud/azure/blobstorage/AzureDataStoreService.java:import
org.apache.felix.scr.annotations.Component;

oak-blob-plugins/src/main/java/org/apache/jackrabbit/oak/plugins/blob/ConsolidatedDataStoreCacheStats.java:import
org.apache.felix.scr.annotations.Component;

oak-blob-plugins/src/main/java/org/apache/jackrabbit/oak/plugins/blob/datastore/AbstractDataStoreService.java:import
org.apache.felix.scr.annotations.Component;

oak-blob-plugins/src/main/java/org/apache/jackrabbit/oak/plugins/blob/datastore/DbDataStoreService.java:import
org.apache.felix.scr.annotations.Component;

oak-blob-plugins/src/main/java/org/apache/jackrabbit/oak/plugins/blob/datastore/FileDataStoreService.java:import
org.apache.felix.scr.annotations.Component;

oak-blob-plugins/src/main/java/org/apache/jackrabbit/oak/plugins/blob/migration/BlobMigration.java:import
org.apache.felix.scr.annotations.Component;

oak-exercise/src/main/java/org/apache/jackrabbit/oak/exercise/security/authentication/CustomLoginModuleFactory.java:import
org.apache.felix.scr.annotations.Component;

oak-exercise/src/main/java/org/apache/jackrabbit/oak/exercise/security/authentication/external/CustomExternalIdentityProvider.java:import
org.apache.felix.scr.annotations.Component;

oak-exercise/src/main/java/org/apache/jackrabbit/oak/exercise/security/authorization/restriction/CustomRestrictionProvider.java:import
org.apache.felix.scr.annotations.Component;

oak-exercise/src/main/java/org/apache/jackrabbit/oak/exercise/security/principal/CustomPrincipalConfiguration.java:import
org.apache.felix.scr.annotations.Component;

oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/observation/ConsolidatedListenerMBeanImpl.java:import
org.apache.felix.scr.annotations.Component;

oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/osgi/RepositoryManager.java:import
org.apache.felix.scr.annotations.Component;

oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/IndexAugmentorFactory.java:import
org.apache.felix.scr.annotations.Component;

oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/LuceneIndexProviderService.java:import
org.apache.felix.scr.annotations.Component;

oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/score/impl/ScorerProviderFactoryImpl.java:import
org.apache.felix.scr.annotations.Component;

oak-remote/src/main/java/org/apache/jackrabbit/oak/remote/osgi/RemoteServletRegistration.java:import
org.apache.felix.scr.annotations.Component;

oak-search-mt/src/main/java/org/apache/jackrabbit/oak/plugins/index/mt/MTFulltextQueryTermsProviderFactory.java:import
org.apache.felix.scr.annotations.Component;

oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/osgi/SegmentNodeStoreServiceDeprecationError.java:import
org.apache.felix.scr.annotations.Component;

oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/osgi/StandbyStoreServiceDeprecationError.java:import
org.apache.felix.scr.annotations.Component;

oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/SegmentNodeStoreFactory.java:import
org.apache.felix.scr.annotations.Component;

oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/SegmentNodeStoreService.java:import
org.apache.felix.scr.annotations.Component;

oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/standby/store/StandbyStoreService.java:import
org.apache.felix.scr.annotations.Component;

oak-solr-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/solr/osgi/EmbeddedSolrServerConfigurationProvider.java:import
org.apache.felix.scr.annotations.Component;

oak-solr-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/solr/osgi/NodeStateSolrServersObserverService.java:import
org.apache.felix.scr.annotations.Component;

oak-solr-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/solr/osgi/OakSolrConfigurationProviderService.java:import
org.apache.felix.scr.annotations.Component;

oak-solr-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/solr/osgi/RemoteSolrServerConfigurationProvider.java:import
org.apache.felix.scr.annotations.Component;

oak-solr-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/solr/osgi/SolrIndexEditorProviderService.java:import
org.apache.felix.scr.annotations.Component;

oak-solr-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/solr/osgi/SolrQueryIndexProviderService.java:import
org.apache.felix.scr.annotations.Component;

oak-solr-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/solr/osgi/SolrServerProviderService.java:import
org.apache.felix.scr.annotations.Component;

oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/checks/NamespacePrefixNodestoreChecker.java:import
org.apache.felix.scr.annotations.Component;

oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/checks/NodeStoreChecksService.java:import
org.apache.felix.scr.annotations.Component;

oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/checks/NodeTypeDefinitionNodeStoreChecker.java:import
org.apache.felix.scr.annotations.Component;

oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/checks/NodeTypeMountedNodeStoreChecker.java:import
org.apache.felix.scr.annotations.Component;

oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/checks/UniqueIndexNodeStoreChecker.java:import
org.apache.felix.scr.annotations.Component;

oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/CompositeNodeStoreService.java:import
org.apache.felix.scr.annotations.Component;

oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/CrossMountReferenceValidatorProvider.java:import
org.apache.felix.scr.annotations.Component;

oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/MountInfoProviderService.java:import
org.apache.felix.scr.annotations.Component;

oak-store-composite/src/test/java/org/apache/jackrabbit/oak/composite/checks/NodeTypeDefinitionNodeStoreCheckerTest.java:import
org.apache.felix.scr.annotations.Component;

Re: Looking for small task starting in OAK .. DS conversion?

Posted by Julian Sedding <js...@gmail.com>.
Hi Christian

It's up to you. I have finished the implementation of the tool now. If
you like, you can build it and see if it helps.

Regards
Julian


On Tue, Oct 31, 2017 at 9:56 AM, Christian Schneider
<ch...@die-schneider.net> wrote:
> Hi Julian,
>
> I finished the conversion for the oak-auth-external module and created a
> PR. The tests all run fine.
> I will look into the comparison tool but I am not sure if it is needed. Of
> course it is possible that I introduce a bug with
> my PR but the comparison tool will also not guarantee that the conversion
> is bug free.
>
> Christian
>
> 2017-10-30 13:40 GMT+01:00 Julian Sedding <js...@gmail.com>:
>
>> Hi Christian
>>
>> I have worked on OAK-6741 before and there were some concerns
>> regarding my changes.
>>
>> To address these concerns, I started work on a tool that allows
>> diffing the OSGi DS and MetaType metadata of two bundles. It uses
>> Felix' SCR and MetaType implementations to parse the metadata and
>> should thus be able to compare on a semantic level rather than on a
>> purely syntactic level (i.e. diff all XML files, which comes with its
>> own challenges)[0].
>>
>> Note, that the tool is yet unfinished, as I don't currently have time
>> to complete it. Basically, what's left to do is implementing some
>> comparisons and possibly more rendering (see TODOs in
>> MetaDataDiff[1]). Fell free to fork, or I'm also happy grant you write
>> access on my repository.
>>
>> I hope you find this helpful!
>>
>> Regards
>> Julian
>>
>> [0] https://github.com/jsedding/osgi-ds-metatype-diff
>> [1] https://github.com/jsedding/osgi-ds-metatype-diff/blob/
>> master/src/main/java/net/distilledcode/tools/osgi/MetadataDiff.java
>>
>>
>> On Mon, Oct 30, 2017 at 10:28 AM, Alex Deparvu <st...@apache.org>
>> wrote:
>> > Hi Christian,
>> >
>> > Thanks for your interest in helping out in this area!
>> > You can look at OAK-6741 [0] to see what the status of this effort is,
>> > there's a few tasks created already waiting for some attention :)
>> >
>> > best,
>> > alex
>> >
>> > [0] https://issues.apache.org/jira/browse/OAK-6741
>> >
>> >
>> >
>> > On Mon, Oct 30, 2017 at 9:57 AM, Christian Schneider <
>> > chris@die-schneider.net> wrote:
>> >
>> >> Hi all,
>> >>
>> >> as I am just starting to work on OAK I am looking for a small task.
>> >> I found that there are still some components that use the old felix scr
>> >> annotations.
>> >> Does it make sense that I look into converting these to the DS ones so
>> we
>> >> can remove support for felix scr in the build?
>> >>
>> >> I have listed the classes below.
>> >> The main issue I see with the migration is that OAK uses the meta type
>> >> support of felix scr which is quite different to what DS 1.3 provides.
>> So I
>> >> would need to migrate from the property based meta type descriptions to
>> the
>> >> type safe ones of the DS 1.3 metatype support.
>> >>
>> >> Anyway I would provide one module per PR so the reviewer does not have
>> to
>> >> review one big commit at once.
>> >>
>> >> Best
>> >> Christian
>> >>
>> >> --
>> >> --
>> >> Christian Schneider
>> >> http://www.liquid-reality.de
>> >> <https://owa.talend.com/owa/redir.aspx?C=3aa4083e0c744ae1ba52bd062c5a7e
>> >> 46&URL=http%3a%2f%2fwww.liquid-reality.de>
>> >>
>> >> Computer Scientist
>> >> http://www.adobe.com
>> >>
>> >>
>> >> ---
>> >>
>> >> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
>> >> authentication/external/impl/DefaultSyncConfigImpl.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
>> >> authentication/external/impl/DefaultSyncHandler.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
>> >> authentication/external/impl/ExternalIDPManagerImpl.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
>> >> authentication/external/impl/ExternalLoginModuleFactory.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
>> >> authentication/external/impl/principal/ExternalPrincipalConfiguration
>> >> .java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
>> >> authentication/external/impl/SyncManagerImpl.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-auth-ldap/src/main/java/org/apache/jackrabbit/oak/
>> >> security/authentication/ldap/impl/LdapIdentityProvider.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-auth-ldap/src/main/java/org/apache/jackrabbit/oak/
>> >> security/authentication/ldap/impl/LdapProviderConfig.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-authorization-cug/src/main/java/org/apache/
>> >> jackrabbit/oak/spi/security/authorization/cug/impl/
>> >> CugConfiguration.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-authorization-cug/src/main/java/org/apache/
>> >> jackrabbit/oak/spi/security/authorization/cug/impl/
>> >> CugExcludeImpl.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/osgi/
>> >> FileBlobStoreService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/osgi/
>> >> SplitBlobStoreService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-blob-cloud/src/main/java/org/apache/jackrabbit/oak/blob/cloud/s3/
>> >> AbstractS3DataStoreService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-blob-cloud/src/main/java/org/apache/jackrabbit/oak/blob/cloud/s3/
>> >> S3DataStoreService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-blob-cloud/src/main/java/org/apache/jackrabbit/oak/blob/cloud/s3/
>> >> SharedS3DataStoreService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-blob-cloud-azure/src/main/java/org/apache/jackrabbit/
>> >> oak/blob/cloud/azure/blobstorage/AzureDataStoreService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-blob-plugins/src/main/java/org/apache/jackrabbit/oak/plugins/blob/
>> >> ConsolidatedDataStoreCacheStats.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-blob-plugins/src/main/java/org/apache/jackrabbit/
>> >> oak/plugins/blob/datastore/AbstractDataStoreService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-blob-plugins/src/main/java/org/apache/jackrabbit/
>> >> oak/plugins/blob/datastore/DbDataStoreService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-blob-plugins/src/main/java/org/apache/jackrabbit/
>> >> oak/plugins/blob/datastore/FileDataStoreService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-blob-plugins/src/main/java/org/apache/jackrabbit/
>> >> oak/plugins/blob/migration/BlobMigration.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-exercise/src/main/java/org/apache/jackrabbit/oak/exercise/security/
>> >> authentication/CustomLoginModuleFactory.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-exercise/src/main/java/org/apache/jackrabbit/oak/exercise/security/
>> >> authentication/external/CustomExternalIdentityProvider.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-exercise/src/main/java/org/apache/jackrabbit/oak/exercise/security/
>> >> authorization/restriction/CustomRestrictionProvider.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-exercise/src/main/java/org/apache/jackrabbit/oak/
>> >> exercise/security/principal/CustomPrincipalConfiguration.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/observation/
>> >> ConsolidatedListenerMBeanImpl.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/
>> >> osgi/RepositoryManager.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/
>> index/lucene/
>> >> IndexAugmentorFactory.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/
>> index/lucene/
>> >> LuceneIndexProviderService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/
>> >> index/lucene/score/impl/ScorerProviderFactoryImpl.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-remote/src/main/java/org/apache/jackrabbit/oak/remote/osgi/
>> >> RemoteServletRegistration.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-search-mt/src/main/java/org/apache/jackrabbit/oak/plugins/index/mt/
>> >> MTFulltextQueryTermsProviderFactory.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/osgi/
>> >> SegmentNodeStoreServiceDeprecationError.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/osgi/
>> >> StandbyStoreServiceDeprecationError.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/
>> >> SegmentNodeStoreFactory.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/
>> >> SegmentNodeStoreService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/
>> >> segment/standby/store/StandbyStoreService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
>> >> plugins/index/solr/osgi/EmbeddedSolrServerConfiguratio
>> >> nProvider.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
>> >> plugins/index/solr/osgi/NodeStateSolrServersObserverService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
>> >> plugins/index/solr/osgi/OakSolrConfigurationProviderService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
>> >> plugins/index/solr/osgi/RemoteSolrServerConfigurationP
>> rovider.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
>> >> plugins/index/solr/osgi/SolrIndexEditorProviderService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
>> >> plugins/index/solr/osgi/SolrQueryIndexProviderService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
>> >> plugins/index/solr/osgi/SolrServerProviderService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-store-composite/src/main/java/org/apache/jackrabbit/
>> >> oak/composite/checks/NamespacePrefixNodestoreChecker.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-store-composite/src/main/java/org/apache/jackrabbit/
>> >> oak/composite/checks/NodeStoreChecksService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-store-composite/src/main/java/org/apache/jackrabbit/
>> >> oak/composite/checks/NodeTypeDefinitionNodeStoreChecker.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-store-composite/src/main/java/org/apache/jackrabbit/
>> >> oak/composite/checks/NodeTypeMountedNodeStoreChecker.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-store-composite/src/main/java/org/apache/jackrabbit/
>> >> oak/composite/checks/UniqueIndexNodeStoreChecker.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/
>> >> CompositeNodeStoreService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/
>> >> CrossMountReferenceValidatorProvider.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/
>> >> MountInfoProviderService.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>> >> oak-store-composite/src/test/java/org/apache/jackrabbit/
>> >> oak/composite/checks/NodeTypeDefinitionNodeStoreCheckerTest.java:import
>> >> org.apache.felix.scr.annotations.Component;
>> >>
>>
>
>
>
> --
> --
> Christian Schneider
> http://www.liquid-reality.de
> <https://owa.talend.com/owa/redir.aspx?C=3aa4083e0c744ae1ba52bd062c5a7e46&URL=http%3a%2f%2fwww.liquid-reality.de>
>
> Computer Scientist
> http://www.adobe.com

Fwd: Looking for small task starting in OAK .. DS conversion?

Posted by Christian Schneider <ch...@die-schneider.net>.
Hi Julian,

I finished the conversion for the oak-auth-external module and created a
PR. The tests all run fine.
I will look into the comparison tool but I am not sure if it is needed. Of
course it is possible that I introduce a bug with
my PR but the comparison tool will also not guarantee that the conversion
is bug free.

Christian

2017-10-30 13:40 GMT+01:00 Julian Sedding <js...@gmail.com>:

> Hi Christian
>
> I have worked on OAK-6741 before and there were some concerns
> regarding my changes.
>
> To address these concerns, I started work on a tool that allows
> diffing the OSGi DS and MetaType metadata of two bundles. It uses
> Felix' SCR and MetaType implementations to parse the metadata and
> should thus be able to compare on a semantic level rather than on a
> purely syntactic level (i.e. diff all XML files, which comes with its
> own challenges)[0].
>
> Note, that the tool is yet unfinished, as I don't currently have time
> to complete it. Basically, what's left to do is implementing some
> comparisons and possibly more rendering (see TODOs in
> MetaDataDiff[1]). Fell free to fork, or I'm also happy grant you write
> access on my repository.
>
> I hope you find this helpful!
>
> Regards
> Julian
>
> [0] https://github.com/jsedding/osgi-ds-metatype-diff
> [1] https://github.com/jsedding/osgi-ds-metatype-diff/blob/
> master/src/main/java/net/distilledcode/tools/osgi/MetadataDiff.java
>
>
> On Mon, Oct 30, 2017 at 10:28 AM, Alex Deparvu <st...@apache.org>
> wrote:
> > Hi Christian,
> >
> > Thanks for your interest in helping out in this area!
> > You can look at OAK-6741 [0] to see what the status of this effort is,
> > there's a few tasks created already waiting for some attention :)
> >
> > best,
> > alex
> >
> > [0] https://issues.apache.org/jira/browse/OAK-6741
> >
> >
> >
> > On Mon, Oct 30, 2017 at 9:57 AM, Christian Schneider <
> > chris@die-schneider.net> wrote:
> >
> >> Hi all,
> >>
> >> as I am just starting to work on OAK I am looking for a small task.
> >> I found that there are still some components that use the old felix scr
> >> annotations.
> >> Does it make sense that I look into converting these to the DS ones so
> we
> >> can remove support for felix scr in the build?
> >>
> >> I have listed the classes below.
> >> The main issue I see with the migration is that OAK uses the meta type
> >> support of felix scr which is quite different to what DS 1.3 provides.
> So I
> >> would need to migrate from the property based meta type descriptions to
> the
> >> type safe ones of the DS 1.3 metatype support.
> >>
> >> Anyway I would provide one module per PR so the reviewer does not have
> to
> >> review one big commit at once.
> >>
> >> Best
> >> Christian
> >>
> >> --
> >> --
> >> Christian Schneider
> >> http://www.liquid-reality.de
> >> <https://owa.talend.com/owa/redir.aspx?C=3aa4083e0c744ae1ba52bd062c5a7e
> >> 46&URL=http%3a%2f%2fwww.liquid-reality.de>
> >>
> >> Computer Scientist
> >> http://www.adobe.com
> >>
> >>
> >> ---
> >>
> >> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
> >> authentication/external/impl/DefaultSyncConfigImpl.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
> >> authentication/external/impl/DefaultSyncHandler.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
> >> authentication/external/impl/ExternalIDPManagerImpl.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
> >> authentication/external/impl/ExternalLoginModuleFactory.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
> >> authentication/external/impl/principal/ExternalPrincipalConfiguration
> >> .java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
> >> authentication/external/impl/SyncManagerImpl.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-auth-ldap/src/main/java/org/apache/jackrabbit/oak/
> >> security/authentication/ldap/impl/LdapIdentityProvider.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-auth-ldap/src/main/java/org/apache/jackrabbit/oak/
> >> security/authentication/ldap/impl/LdapProviderConfig.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-authorization-cug/src/main/java/org/apache/
> >> jackrabbit/oak/spi/security/authorization/cug/impl/
> >> CugConfiguration.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-authorization-cug/src/main/java/org/apache/
> >> jackrabbit/oak/spi/security/authorization/cug/impl/
> >> CugExcludeImpl.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/osgi/
> >> FileBlobStoreService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/osgi/
> >> SplitBlobStoreService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-blob-cloud/src/main/java/org/apache/jackrabbit/oak/blob/cloud/s3/
> >> AbstractS3DataStoreService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-blob-cloud/src/main/java/org/apache/jackrabbit/oak/blob/cloud/s3/
> >> S3DataStoreService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-blob-cloud/src/main/java/org/apache/jackrabbit/oak/blob/cloud/s3/
> >> SharedS3DataStoreService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-blob-cloud-azure/src/main/java/org/apache/jackrabbit/
> >> oak/blob/cloud/azure/blobstorage/AzureDataStoreService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-blob-plugins/src/main/java/org/apache/jackrabbit/oak/plugins/blob/
> >> ConsolidatedDataStoreCacheStats.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-blob-plugins/src/main/java/org/apache/jackrabbit/
> >> oak/plugins/blob/datastore/AbstractDataStoreService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-blob-plugins/src/main/java/org/apache/jackrabbit/
> >> oak/plugins/blob/datastore/DbDataStoreService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-blob-plugins/src/main/java/org/apache/jackrabbit/
> >> oak/plugins/blob/datastore/FileDataStoreService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-blob-plugins/src/main/java/org/apache/jackrabbit/
> >> oak/plugins/blob/migration/BlobMigration.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-exercise/src/main/java/org/apache/jackrabbit/oak/exercise/security/
> >> authentication/CustomLoginModuleFactory.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-exercise/src/main/java/org/apache/jackrabbit/oak/exercise/security/
> >> authentication/external/CustomExternalIdentityProvider.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-exercise/src/main/java/org/apache/jackrabbit/oak/exercise/security/
> >> authorization/restriction/CustomRestrictionProvider.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-exercise/src/main/java/org/apache/jackrabbit/oak/
> >> exercise/security/principal/CustomPrincipalConfiguration.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/observation/
> >> ConsolidatedListenerMBeanImpl.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/
> >> osgi/RepositoryManager.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/
> index/lucene/
> >> IndexAugmentorFactory.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/
> index/lucene/
> >> LuceneIndexProviderService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/
> >> index/lucene/score/impl/ScorerProviderFactoryImpl.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-remote/src/main/java/org/apache/jackrabbit/oak/remote/osgi/
> >> RemoteServletRegistration.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-search-mt/src/main/java/org/apache/jackrabbit/oak/plugins/index/mt/
> >> MTFulltextQueryTermsProviderFactory.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/osgi/
> >> SegmentNodeStoreServiceDeprecationError.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/osgi/
> >> StandbyStoreServiceDeprecationError.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/
> >> SegmentNodeStoreFactory.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/
> >> SegmentNodeStoreService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/
> >> segment/standby/store/StandbyStoreService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
> >> plugins/index/solr/osgi/EmbeddedSolrServerConfiguratio
> >> nProvider.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
> >> plugins/index/solr/osgi/NodeStateSolrServersObserverService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
> >> plugins/index/solr/osgi/OakSolrConfigurationProviderService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
> >> plugins/index/solr/osgi/RemoteSolrServerConfigurationP
> rovider.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
> >> plugins/index/solr/osgi/SolrIndexEditorProviderService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
> >> plugins/index/solr/osgi/SolrQueryIndexProviderService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
> >> plugins/index/solr/osgi/SolrServerProviderService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-store-composite/src/main/java/org/apache/jackrabbit/
> >> oak/composite/checks/NamespacePrefixNodestoreChecker.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-store-composite/src/main/java/org/apache/jackrabbit/
> >> oak/composite/checks/NodeStoreChecksService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-store-composite/src/main/java/org/apache/jackrabbit/
> >> oak/composite/checks/NodeTypeDefinitionNodeStoreChecker.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-store-composite/src/main/java/org/apache/jackrabbit/
> >> oak/composite/checks/NodeTypeMountedNodeStoreChecker.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-store-composite/src/main/java/org/apache/jackrabbit/
> >> oak/composite/checks/UniqueIndexNodeStoreChecker.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/
> >> CompositeNodeStoreService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/
> >> CrossMountReferenceValidatorProvider.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/
> >> MountInfoProviderService.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
> >> oak-store-composite/src/test/java/org/apache/jackrabbit/
> >> oak/composite/checks/NodeTypeDefinitionNodeStoreCheckerTest.java:import
> >> org.apache.felix.scr.annotations.Component;
> >>
>



-- 
-- 
Christian Schneider
http://www.liquid-reality.de
<https://owa.talend.com/owa/redir.aspx?C=3aa4083e0c744ae1ba52bd062c5a7e46&URL=http%3a%2f%2fwww.liquid-reality.de>

Computer Scientist
http://www.adobe.com

Re: Looking for small task starting in OAK .. DS conversion?

Posted by Julian Sedding <js...@gmail.com>.
Hi Christian

I have worked on OAK-6741 before and there were some concerns
regarding my changes.

To address these concerns, I started work on a tool that allows
diffing the OSGi DS and MetaType metadata of two bundles. It uses
Felix' SCR and MetaType implementations to parse the metadata and
should thus be able to compare on a semantic level rather than on a
purely syntactic level (i.e. diff all XML files, which comes with its
own challenges)[0].

Note, that the tool is yet unfinished, as I don't currently have time
to complete it. Basically, what's left to do is implementing some
comparisons and possibly more rendering (see TODOs in
MetaDataDiff[1]). Fell free to fork, or I'm also happy grant you write
access on my repository.

I hope you find this helpful!

Regards
Julian

[0] https://github.com/jsedding/osgi-ds-metatype-diff
[1] https://github.com/jsedding/osgi-ds-metatype-diff/blob/master/src/main/java/net/distilledcode/tools/osgi/MetadataDiff.java


On Mon, Oct 30, 2017 at 10:28 AM, Alex Deparvu <st...@apache.org> wrote:
> Hi Christian,
>
> Thanks for your interest in helping out in this area!
> You can look at OAK-6741 [0] to see what the status of this effort is,
> there's a few tasks created already waiting for some attention :)
>
> best,
> alex
>
> [0] https://issues.apache.org/jira/browse/OAK-6741
>
>
>
> On Mon, Oct 30, 2017 at 9:57 AM, Christian Schneider <
> chris@die-schneider.net> wrote:
>
>> Hi all,
>>
>> as I am just starting to work on OAK I am looking for a small task.
>> I found that there are still some components that use the old felix scr
>> annotations.
>> Does it make sense that I look into converting these to the DS ones so we
>> can remove support for felix scr in the build?
>>
>> I have listed the classes below.
>> The main issue I see with the migration is that OAK uses the meta type
>> support of felix scr which is quite different to what DS 1.3 provides. So I
>> would need to migrate from the property based meta type descriptions to the
>> type safe ones of the DS 1.3 metatype support.
>>
>> Anyway I would provide one module per PR so the reviewer does not have to
>> review one big commit at once.
>>
>> Best
>> Christian
>>
>> --
>> --
>> Christian Schneider
>> http://www.liquid-reality.de
>> <https://owa.talend.com/owa/redir.aspx?C=3aa4083e0c744ae1ba52bd062c5a7e
>> 46&URL=http%3a%2f%2fwww.liquid-reality.de>
>>
>> Computer Scientist
>> http://www.adobe.com
>>
>>
>> ---
>>
>> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
>> authentication/external/impl/DefaultSyncConfigImpl.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
>> authentication/external/impl/DefaultSyncHandler.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
>> authentication/external/impl/ExternalIDPManagerImpl.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
>> authentication/external/impl/ExternalLoginModuleFactory.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
>> authentication/external/impl/principal/ExternalPrincipalConfiguration
>> .java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
>> authentication/external/impl/SyncManagerImpl.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-auth-ldap/src/main/java/org/apache/jackrabbit/oak/
>> security/authentication/ldap/impl/LdapIdentityProvider.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-auth-ldap/src/main/java/org/apache/jackrabbit/oak/
>> security/authentication/ldap/impl/LdapProviderConfig.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-authorization-cug/src/main/java/org/apache/
>> jackrabbit/oak/spi/security/authorization/cug/impl/
>> CugConfiguration.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-authorization-cug/src/main/java/org/apache/
>> jackrabbit/oak/spi/security/authorization/cug/impl/
>> CugExcludeImpl.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/osgi/
>> FileBlobStoreService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/osgi/
>> SplitBlobStoreService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-blob-cloud/src/main/java/org/apache/jackrabbit/oak/blob/cloud/s3/
>> AbstractS3DataStoreService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-blob-cloud/src/main/java/org/apache/jackrabbit/oak/blob/cloud/s3/
>> S3DataStoreService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-blob-cloud/src/main/java/org/apache/jackrabbit/oak/blob/cloud/s3/
>> SharedS3DataStoreService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-blob-cloud-azure/src/main/java/org/apache/jackrabbit/
>> oak/blob/cloud/azure/blobstorage/AzureDataStoreService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-blob-plugins/src/main/java/org/apache/jackrabbit/oak/plugins/blob/
>> ConsolidatedDataStoreCacheStats.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-blob-plugins/src/main/java/org/apache/jackrabbit/
>> oak/plugins/blob/datastore/AbstractDataStoreService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-blob-plugins/src/main/java/org/apache/jackrabbit/
>> oak/plugins/blob/datastore/DbDataStoreService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-blob-plugins/src/main/java/org/apache/jackrabbit/
>> oak/plugins/blob/datastore/FileDataStoreService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-blob-plugins/src/main/java/org/apache/jackrabbit/
>> oak/plugins/blob/migration/BlobMigration.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-exercise/src/main/java/org/apache/jackrabbit/oak/exercise/security/
>> authentication/CustomLoginModuleFactory.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-exercise/src/main/java/org/apache/jackrabbit/oak/exercise/security/
>> authentication/external/CustomExternalIdentityProvider.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-exercise/src/main/java/org/apache/jackrabbit/oak/exercise/security/
>> authorization/restriction/CustomRestrictionProvider.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-exercise/src/main/java/org/apache/jackrabbit/oak/
>> exercise/security/principal/CustomPrincipalConfiguration.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/observation/
>> ConsolidatedListenerMBeanImpl.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/
>> osgi/RepositoryManager.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/
>> IndexAugmentorFactory.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/
>> LuceneIndexProviderService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/
>> index/lucene/score/impl/ScorerProviderFactoryImpl.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-remote/src/main/java/org/apache/jackrabbit/oak/remote/osgi/
>> RemoteServletRegistration.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-search-mt/src/main/java/org/apache/jackrabbit/oak/plugins/index/mt/
>> MTFulltextQueryTermsProviderFactory.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/osgi/
>> SegmentNodeStoreServiceDeprecationError.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/osgi/
>> StandbyStoreServiceDeprecationError.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/
>> SegmentNodeStoreFactory.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/
>> SegmentNodeStoreService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/
>> segment/standby/store/StandbyStoreService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
>> plugins/index/solr/osgi/EmbeddedSolrServerConfiguratio
>> nProvider.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
>> plugins/index/solr/osgi/NodeStateSolrServersObserverService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
>> plugins/index/solr/osgi/OakSolrConfigurationProviderService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
>> plugins/index/solr/osgi/RemoteSolrServerConfigurationProvider.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
>> plugins/index/solr/osgi/SolrIndexEditorProviderService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
>> plugins/index/solr/osgi/SolrQueryIndexProviderService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
>> plugins/index/solr/osgi/SolrServerProviderService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-store-composite/src/main/java/org/apache/jackrabbit/
>> oak/composite/checks/NamespacePrefixNodestoreChecker.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-store-composite/src/main/java/org/apache/jackrabbit/
>> oak/composite/checks/NodeStoreChecksService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-store-composite/src/main/java/org/apache/jackrabbit/
>> oak/composite/checks/NodeTypeDefinitionNodeStoreChecker.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-store-composite/src/main/java/org/apache/jackrabbit/
>> oak/composite/checks/NodeTypeMountedNodeStoreChecker.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-store-composite/src/main/java/org/apache/jackrabbit/
>> oak/composite/checks/UniqueIndexNodeStoreChecker.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/
>> CompositeNodeStoreService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/
>> CrossMountReferenceValidatorProvider.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/
>> MountInfoProviderService.java:import
>> org.apache.felix.scr.annotations.Component;
>>
>> oak-store-composite/src/test/java/org/apache/jackrabbit/
>> oak/composite/checks/NodeTypeDefinitionNodeStoreCheckerTest.java:import
>> org.apache.felix.scr.annotations.Component;
>>

Re: Looking for small task starting in OAK .. DS conversion?

Posted by Alex Deparvu <st...@apache.org>.
Hi Christian,

Thanks for your interest in helping out in this area!
You can look at OAK-6741 [0] to see what the status of this effort is,
there's a few tasks created already waiting for some attention :)

best,
alex

[0] https://issues.apache.org/jira/browse/OAK-6741



On Mon, Oct 30, 2017 at 9:57 AM, Christian Schneider <
chris@die-schneider.net> wrote:

> Hi all,
>
> as I am just starting to work on OAK I am looking for a small task.
> I found that there are still some components that use the old felix scr
> annotations.
> Does it make sense that I look into converting these to the DS ones so we
> can remove support for felix scr in the build?
>
> I have listed the classes below.
> The main issue I see with the migration is that OAK uses the meta type
> support of felix scr which is quite different to what DS 1.3 provides. So I
> would need to migrate from the property based meta type descriptions to the
> type safe ones of the DS 1.3 metatype support.
>
> Anyway I would provide one module per PR so the reviewer does not have to
> review one big commit at once.
>
> Best
> Christian
>
> --
> --
> Christian Schneider
> http://www.liquid-reality.de
> <https://owa.talend.com/owa/redir.aspx?C=3aa4083e0c744ae1ba52bd062c5a7e
> 46&URL=http%3a%2f%2fwww.liquid-reality.de>
>
> Computer Scientist
> http://www.adobe.com
>
>
> ---
>
> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
> authentication/external/impl/DefaultSyncConfigImpl.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
> authentication/external/impl/DefaultSyncHandler.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
> authentication/external/impl/ExternalIDPManagerImpl.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
> authentication/external/impl/ExternalLoginModuleFactory.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
> authentication/external/impl/principal/ExternalPrincipalConfiguration
> .java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/
> authentication/external/impl/SyncManagerImpl.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-auth-ldap/src/main/java/org/apache/jackrabbit/oak/
> security/authentication/ldap/impl/LdapIdentityProvider.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-auth-ldap/src/main/java/org/apache/jackrabbit/oak/
> security/authentication/ldap/impl/LdapProviderConfig.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-authorization-cug/src/main/java/org/apache/
> jackrabbit/oak/spi/security/authorization/cug/impl/
> CugConfiguration.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-authorization-cug/src/main/java/org/apache/
> jackrabbit/oak/spi/security/authorization/cug/impl/
> CugExcludeImpl.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/osgi/
> FileBlobStoreService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-blob/src/main/java/org/apache/jackrabbit/oak/spi/blob/osgi/
> SplitBlobStoreService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-blob-cloud/src/main/java/org/apache/jackrabbit/oak/blob/cloud/s3/
> AbstractS3DataStoreService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-blob-cloud/src/main/java/org/apache/jackrabbit/oak/blob/cloud/s3/
> S3DataStoreService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-blob-cloud/src/main/java/org/apache/jackrabbit/oak/blob/cloud/s3/
> SharedS3DataStoreService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-blob-cloud-azure/src/main/java/org/apache/jackrabbit/
> oak/blob/cloud/azure/blobstorage/AzureDataStoreService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-blob-plugins/src/main/java/org/apache/jackrabbit/oak/plugins/blob/
> ConsolidatedDataStoreCacheStats.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-blob-plugins/src/main/java/org/apache/jackrabbit/
> oak/plugins/blob/datastore/AbstractDataStoreService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-blob-plugins/src/main/java/org/apache/jackrabbit/
> oak/plugins/blob/datastore/DbDataStoreService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-blob-plugins/src/main/java/org/apache/jackrabbit/
> oak/plugins/blob/datastore/FileDataStoreService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-blob-plugins/src/main/java/org/apache/jackrabbit/
> oak/plugins/blob/migration/BlobMigration.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-exercise/src/main/java/org/apache/jackrabbit/oak/exercise/security/
> authentication/CustomLoginModuleFactory.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-exercise/src/main/java/org/apache/jackrabbit/oak/exercise/security/
> authentication/external/CustomExternalIdentityProvider.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-exercise/src/main/java/org/apache/jackrabbit/oak/exercise/security/
> authorization/restriction/CustomRestrictionProvider.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-exercise/src/main/java/org/apache/jackrabbit/oak/
> exercise/security/principal/CustomPrincipalConfiguration.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/observation/
> ConsolidatedListenerMBeanImpl.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/
> osgi/RepositoryManager.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/
> IndexAugmentorFactory.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/
> LuceneIndexProviderService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/
> index/lucene/score/impl/ScorerProviderFactoryImpl.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-remote/src/main/java/org/apache/jackrabbit/oak/remote/osgi/
> RemoteServletRegistration.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-search-mt/src/main/java/org/apache/jackrabbit/oak/plugins/index/mt/
> MTFulltextQueryTermsProviderFactory.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/osgi/
> SegmentNodeStoreServiceDeprecationError.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/osgi/
> StandbyStoreServiceDeprecationError.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/
> SegmentNodeStoreFactory.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/
> SegmentNodeStoreService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/
> segment/standby/store/StandbyStoreService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
> plugins/index/solr/osgi/EmbeddedSolrServerConfiguratio
> nProvider.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
> plugins/index/solr/osgi/NodeStateSolrServersObserverService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
> plugins/index/solr/osgi/OakSolrConfigurationProviderService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
> plugins/index/solr/osgi/RemoteSolrServerConfigurationProvider.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
> plugins/index/solr/osgi/SolrIndexEditorProviderService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
> plugins/index/solr/osgi/SolrQueryIndexProviderService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-solr-core/src/main/java/org/apache/jackrabbit/oak/
> plugins/index/solr/osgi/SolrServerProviderService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-store-composite/src/main/java/org/apache/jackrabbit/
> oak/composite/checks/NamespacePrefixNodestoreChecker.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-store-composite/src/main/java/org/apache/jackrabbit/
> oak/composite/checks/NodeStoreChecksService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-store-composite/src/main/java/org/apache/jackrabbit/
> oak/composite/checks/NodeTypeDefinitionNodeStoreChecker.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-store-composite/src/main/java/org/apache/jackrabbit/
> oak/composite/checks/NodeTypeMountedNodeStoreChecker.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-store-composite/src/main/java/org/apache/jackrabbit/
> oak/composite/checks/UniqueIndexNodeStoreChecker.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/
> CompositeNodeStoreService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/
> CrossMountReferenceValidatorProvider.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/
> MountInfoProviderService.java:import
> org.apache.felix.scr.annotations.Component;
>
> oak-store-composite/src/test/java/org/apache/jackrabbit/
> oak/composite/checks/NodeTypeDefinitionNodeStoreCheckerTest.java:import
> org.apache.felix.scr.annotations.Component;
>