wrapup

mulesoft · Dec 13, 2024 · 302c4b4 · 302c4b4
1 parent 7d66c7b
commit 302c4b4
Show file tree

Hide file tree

Showing 3 changed files with 65 additions and 148 deletions.
diff --git a/core/api-changes.json b/core/api-changes.json
@@ -1,110 +1,66 @@
 {
-  "4.9.0": {
+  "4.10.0": {
     "revapi": {
       "differences": {
         "differences": [
-{
-  "ignore": true,
-  "code": "java.field.removedWithConstant",
-  "old": "field org.mule.runtime.api.lifecycle.Initialisable.PHASE_NAME @ org.mule.runtime.core.api.config.DefaultMuleConfiguration",
-  "justification": "ADD YOUR EXPLANATION FOR THE NECESSITY OF THIS CHANGE"
-  /*  "package": "org.mule.runtime.core.api.config",
-  "classQualifiedName": "org.mule.runtime.core.api.config.DefaultMuleConfiguration",
-  "classSimpleName": "DefaultMuleConfiguration",
-  "fieldName": "PHASE_NAME",
-  "elementKind": "field",
-  "oldArchive": "org.mule.runtime:mule-core:jar:4.8.0",
-  "oldArchiveRole": "primary",
-  "breaksVersioningRules": "true",
-  */
-
-},
-{
-  "ignore": true,
-  "code": "java.method.removed",
-  "old": "method void org.mule.runtime.core.api.config.DefaultMuleConfiguration::initialise() throws org.mule.runtime.api.lifecycle.InitialisationException",
-  "justification": "ADD YOUR EXPLANATION FOR THE NECESSITY OF THIS CHANGE"
-  /*  "package": "org.mule.runtime.core.api.config",
-  "classQualifiedName": "org.mule.runtime.core.api.config.DefaultMuleConfiguration",
-  "classSimpleName": "DefaultMuleConfiguration",
-  "methodName": "initialise",
-  "elementKind": "method",
-  "oldArchive": "org.mule.runtime:mule-core:jar:4.8.0",
-  "oldArchiveRole": "primary",
-  "breaksVersioningRules": "true",
-  */
-
-},
-{
-  "ignore": true,
-  "code": "java.class.noLongerImplementsInterface",
-  "old": "class org.mule.runtime.core.api.config.DefaultMuleConfiguration",
-  "new": "class org.mule.runtime.core.api.config.DefaultMuleConfiguration",
-  "interface": "org.mule.runtime.api.lifecycle.Initialisable",
-  "justification": "ADD YOUR EXPLANATION FOR THE NECESSITY OF THIS CHANGE"
-  /*  "package": "org.mule.runtime.core.api.config",
-  "classQualifiedName": "org.mule.runtime.core.api.config.DefaultMuleConfiguration",
-  "classSimpleName": "DefaultMuleConfiguration",
-  "elementKind": "class",
-  "oldArchive": "org.mule.runtime:mule-core:jar:4.8.0",
-  "oldArchiveRole": "primary",
-  "newArchive": "org.mule.runtime:mule-core:jar:4.9.0-SNAPSHOT",
-  "newArchiveRole": "primary",
-  "breaksVersioningRules": "true",
-  */
-
-},
-{
-  "ignore": true,
-  "code": "java.field.removedWithConstant",
-  "old": "field org.mule.runtime.api.lifecycle.Initialisable.PHASE_NAME @ org.mule.runtime.core.api.config.ReconfigurableMuleConfiguration",
-  "justification": "ADD YOUR EXPLANATION FOR THE NECESSITY OF THIS CHANGE"
-  /*  "package": "org.mule.runtime.core.api.config",
-  "classQualifiedName": "org.mule.runtime.core.api.config.ReconfigurableMuleConfiguration",
-  "classSimpleName": "ReconfigurableMuleConfiguration",
-  "fieldName": "PHASE_NAME",
-  "elementKind": "field",
-  "oldArchive": "org.mule.runtime:mule-core:jar:4.8.0",
-  "oldArchiveRole": "primary",
-  "breaksVersioningRules": "true",
-  */
-
-},
-{
-  "ignore": true,
-  "code": "java.class.noLongerImplementsInterface",
-  "old": "class org.mule.runtime.core.api.config.ReconfigurableMuleConfiguration",
-  "new": "class org.mule.runtime.core.api.config.ReconfigurableMuleConfiguration",
-  "interface": "org.mule.runtime.api.lifecycle.Initialisable",
-  "justification": "ADD YOUR EXPLANATION FOR THE NECESSITY OF THIS CHANGE"
-  /*  "package": "org.mule.runtime.core.api.config",
-  "classQualifiedName": "org.mule.runtime.core.api.config.ReconfigurableMuleConfiguration",
-  "classSimpleName": "ReconfigurableMuleConfiguration",
-  "elementKind": "class",
-  "oldArchive": "org.mule.runtime:mule-core:jar:4.8.0",
-  "oldArchiveRole": "primary",
-  "newArchive": "org.mule.runtime:mule-core:jar:4.9.0-SNAPSHOT",
-  "newArchiveRole": "primary",
-  "breaksVersioningRules": "true",
-  */
-
-},
           {
             "ignore": true,
             "code": "java.method.exception.runtimeAdded",
             "old": "method <T> T org.mule.runtime.core.api.Injector::inject(T) throws org.mule.runtime.api.exception.MuleException",
             "new": "method <T> T org.mule.runtime.core.api.Injector::inject(T) throws org.mule.runtime.core.api.registry.IllegalDependencyInjectionException",
             "exception": "org.mule.runtime.core.api.registry.IllegalDependencyInjectionException",
-            "justification": "ADD YOUR EXPLANATION FOR THE NECESSITY OF THIS CHANGE"
+            "justification": "W-10623235: [MULE-10061] - Rethink the lifecycle of the MuleContext"
           },
           {
             "ignore": true,
             "code": "java.method.exception.checkedRemoved",
             "old": "method <T> T org.mule.runtime.core.api.Injector::inject(T) throws org.mule.runtime.api.exception.MuleException",
             "new": "method <T> T org.mule.runtime.core.api.Injector::inject(T) throws org.mule.runtime.core.api.registry.IllegalDependencyInjectionException",
             "exception": "org.mule.runtime.api.exception.MuleException",
-            "justification": "ADD YOUR EXPLANATION FOR THE NECESSITY OF THIS CHANGE"
+            "justification": "W-10623235: [MULE-10061] - Rethink the lifecycle of the MuleContext"
+          },
+          {
+            "ignore": true,
+            "code": "java.field.removedWithConstant",
+            "old": "field org.mule.runtime.api.lifecycle.Initialisable.PHASE_NAME @ org.mule.runtime.core.api.config.DefaultMuleConfiguration",
+            "justification": "W-10623235: [MULE-10061] - Rethink the lifecycle of the MuleContext"
+          },
+          {
+            "ignore": true,
+            "code": "java.method.removed",
+            "old": "method void org.mule.runtime.core.api.config.DefaultMuleConfiguration::initialise() throws org.mule.runtime.api.lifecycle.InitialisationException",
+            "justification": "W-10623235: [MULE-10061] - Rethink the lifecycle of the MuleContext"
+          },
+          {
+            "ignore": true,
+            "code": "java.class.noLongerImplementsInterface",
+            "old": "class org.mule.runtime.core.api.config.DefaultMuleConfiguration",
+            "new": "class org.mule.runtime.core.api.config.DefaultMuleConfiguration",
+            "interface": "org.mule.runtime.api.lifecycle.Initialisable",
+            "justification": "W-10623235: [MULE-10061] - Rethink the lifecycle of the MuleContext"
+          },
+          {
+            "ignore": true,
+            "code": "java.field.removedWithConstant",
+            "old": "field org.mule.runtime.api.lifecycle.Initialisable.PHASE_NAME @ org.mule.runtime.core.api.config.ReconfigurableMuleConfiguration",
+            "justification": "W-10623235: [MULE-10061] - Rethink the lifecycle of the MuleContext"
           },
+          {
+            "ignore": true,
+            "code": "java.class.noLongerImplementsInterface",
+            "old": "class org.mule.runtime.core.api.config.ReconfigurableMuleConfiguration",
+            "new": "class org.mule.runtime.core.api.config.ReconfigurableMuleConfiguration",
+            "interface": "org.mule.runtime.api.lifecycle.Initialisable",
+            "justification": "W-10623235: [MULE-10061] - Rethink the lifecycle of the MuleContext"
+          }
+        ]
+      }
+    }
+  },
+  "4.9.0": {
+    "revapi": {
+      "differences": {
+        "differences": [
           {
             "ignore": true,
             "code": "java.class.removed",

diff --git a/modules/artifact/api-changes.json b/modules/artifact/api-changes.json
@@ -1,61 +1,28 @@
 {
+  "4.10.0": {
+    "revapi": {
+      "differences": {
+        "differences": [
+          {
+            "ignore": true,
+            "code": "java.method.removed",
+            "old": "method void org.mule.runtime.module.artifact.api.serializer.ArtifactObjectSerializer::initialise() throws org.mule.runtime.api.lifecycle.InitialisationException",
+            "justification": "W-10623235: [MULE-10061] - Rethink the lifecycle of the MuleContext"
+          },
+          {
+            "ignore": true,
+            "code": "java.method.removed",
+            "old": "method void org.mule.runtime.module.artifact.api.serializer.ArtifactObjectSerializer::setMuleContext(org.mule.runtime.core.api.MuleContext)",
+            "justification": "W-10623235: [MULE-10061] - Rethink the lifecycle of the MuleContext"
+          }
+        ]
+      }
+    }
+  },
   "4.9.0": {
     "revapi": {
       "differences": {
         "differences": [
-{
-  "ignore": true,
-  "code": "java.method.removed",
-  "old": "method void org.mule.runtime.module.artifact.api.serializer.ArtifactObjectSerializer::setMuleContext(org.mule.runtime.core.api.MuleContext)",
-  "justification": "ADD YOUR EXPLANATION FOR THE NECESSITY OF THIS CHANGE"
-  /*  "package": "org.mule.runtime.module.artifact.api.serializer",
-  "classQualifiedName": "org.mule.runtime.module.artifact.api.serializer.ArtifactObjectSerializer",
-  "classSimpleName": "ArtifactObjectSerializer",
-  "methodName": "setMuleContext",
-  "elementKind": "method",
-  "oldArchive": "org.mule.runtime:mule-module-artifact:jar:4.8.0",
-  "oldArchiveRole": "primary",
-  "breaksVersioningRules": "true",
-  */
-
-},
-{
-  "ignore": true,
-  "code": "java.method.removed",
-  "old": "method void org.mule.runtime.module.artifact.api.serializer.ArtifactObjectSerializer::initialise() throws org.mule.runtime.api.lifecycle.InitialisationException",
-  "justification": "ADD YOUR EXPLANATION FOR THE NECESSITY OF THIS CHANGE"
-  /*  "package": "org.mule.runtime.module.artifact.api.serializer",
-  "classQualifiedName": "org.mule.runtime.module.artifact.api.serializer.ArtifactObjectSerializer",
-  "classSimpleName": "ArtifactObjectSerializer",
-  "methodName": "initialise",
-  "elementKind": "method",
-  "oldArchive": "org.mule.runtime:mule-module-artifact:jar:4.8.0",
-  "oldArchiveRole": "primary",
-  "breaksVersioningRules": "true",
-  */
-
-},
-{
-  "ignore": true,
-  "code": "java.annotation.added",
-  "old": "method void org.mule.runtime.module.artifact.api.serializer.ArtifactObjectSerializer::setMuleContext(org.mule.runtime.core.api.MuleContext)",
-  "new": "method void org.mule.runtime.module.artifact.api.serializer.ArtifactObjectSerializer::setMuleContext(org.mule.runtime.core.api.MuleContext)",
-  "annotation": "@javax.inject.Inject",
-  "justification": "ADD YOUR EXPLANATION FOR THE NECESSITY OF THIS CHANGE"
-  /*  "annotationType": "javax.inject.Inject",
-  "package": "org.mule.runtime.module.artifact.api.serializer",
-  "classQualifiedName": "org.mule.runtime.module.artifact.api.serializer.ArtifactObjectSerializer",
-  "classSimpleName": "ArtifactObjectSerializer",
-  "methodName": "setMuleContext",
-  "elementKind": "method",
-  "oldArchive": "org.mule.runtime:mule-module-artifact:jar:4.8.0",
-  "oldArchiveRole": "primary",
-  "newArchive": "org.mule.runtime:mule-module-artifact:jar:4.9.0-SNAPSHOT",
-  "newArchiveRole": "primary",
-  "breaksVersioningRules": "true",
-  */
-
-},
           {
             "ignore": true,
             "code": "java.method.removed",

diff --git a/...c/main/java/org/mule/runtime/config/internal/registry/SpringRegistryLifecycleManager.java b/...c/main/java/org/mule/runtime/config/internal/registry/SpringRegistryLifecycleManager.java
@@ -116,12 +116,6 @@ public SpringContextInitialisePhase() {
 
     @Override
     public void applyLifecycle(Object o) throws LifecycleException {
-      // Initialization already done when the spring context is first refreshed
-      // try {
-      // o = getMuleContext().get().getInjector().inject(o);
-      // } catch (MuleException e) {
-      // throw new LifecycleException(e, o);
-      // }
       if (o instanceof Transformer) {
         String name = ((Transformer) o).getName();
         if (isNamedBean(name)) {