[
https://issues.apache.org/jira/browse/OAK-11244?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17895111#comment-17895111
]
Julian Reschke edited comment on OAK-11244 at 11/3/24 12:43 PM:
----------------------------------------------------------------
I actually under-estimated the number of classes:
{noformat}
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/AbstractRepositoryFactoryTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/ConfigTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/DocumentNodeStoreConfigTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/HybridIndexDisabledTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/JaasConfigSpiTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/JsonConfigRepFactoryTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/LuceneSupportTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/MBeanIntegrationTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/NodeStoreConfigTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/PropertyIndexReindexingTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/RepositoryClosedTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/RepositoryShutdownTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/SecondaryStoreConfigIT.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/SecurityProviderRegistrationTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/SegmentNodeStoreConfigTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/TarSegmentNodeStoreConfigTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/TokenAuthenticationTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/TrackerSupportTest.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/CdCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/CheckpointCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/ExportCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/ExportRelevantDocumentsCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/LsCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/LsdDocumentCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/LuceneCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/OakHelpCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/PnCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/PrintDocumentCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/RefreshCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/RetrieveCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/GroovyConsole.groovy
./oak-segment-tar/src/test/upgrade-it-scripts/create16store.groovy
./oak-segment-tar/target/upgrade-it/create16store.groovy
{noformat}
I encountered this when removing guava dependencies in oak-pojosr. Maybe we
should to thid in small steps, and I would start there.
was (Author: reschke):
I actually under-estimated the number of classes:
{noformat}
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/AbstractRepositoryFactoryTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/ConfigTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/DocumentNodeStoreConfigTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/HybridIndexDisabledTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/JaasConfigSpiTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/JsonConfigRepFactoryTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/LuceneSupportTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/MBeanIntegrationTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/NodeStoreConfigTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/PropertyIndexReindexingTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/RepositoryClosedTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/RepositoryShutdownTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/SecondaryStoreConfigIT.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/SecurityProviderRegistrationTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/SegmentNodeStoreConfigTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/TarSegmentNodeStoreConfigTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/TokenAuthenticationTest.groovy
./oak-pojosr/src/test/groovy/org/apache/jackrabbit/oak/run/osgi/TrackerSupportTest.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/CdCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/CheckpointCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/ExportCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/ExportRelevantDocumentsCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/LsCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/LsdDocumentCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/LuceneCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/OakHelpCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/PnCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/PrintDocumentCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/RefreshCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/commands/RetrieveCommand.groovy
./oak-run/src/main/groovy/org/apache/jackrabbit/oak/console/GroovyConsole.groovy
./oak-segment-tar/src/test/upgrade-it-scripts/create16store.groovy
./oak-segment-tar/target/upgrade-it/create16store.groovy
{noformat}
> Migrate Groovy code to Java
> ---------------------------
>
> Key: OAK-11244
> URL: https://issues.apache.org/jira/browse/OAK-11244
> Project: Jackrabbit Oak
> Issue Type: Task
> Reporter: Julian Reschke
> Priority: Major
>
> We have a fest classes written in Groovy.
> This poses problems when
> - people not fluent in Groovy need to make changes or debug the code
> - updating Java versions
> There are only a few, we should just convert them to proper Java code.
> [~stefanegli] [~mreutegg] [~angela] wdyt?
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
