Skip to content

feat(QTDI-2134): dynamic dependencies - nested loading #1135

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
undx wants to merge 13 commits into
base: master
Choose a base branch
from

feat(QTDI-2134): fix pom for remove dynamicdependencies feature.

ff3973c
Select commit
Loading
Failed to load commit list.
Open

feat(QTDI-2134): dynamic dependencies - nested loading #1135

feat(QTDI-2134): fix pom for remove dynamicdependencies feature.
ff3973c
Select commit
Loading
Failed to load commit list.
sonar-eks / SonarQube Code Analysis failed Dec 4, 2025 in 1m 20s

Quality Gate failed

Failed Quality Gate failed

  • 0.00% Coverage on New Code (is less than 80.00%)
  • 0.00% Security Hotspots Reviewed on New Code (is less than 100.00%)
  • 10 New Issues (is greater than 0)

Project ID: org.talend.sdk.component:component-runtime

View in SonarQube

Annotations

Check failure on line 1335 in component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/ComponentManager.java

See this annotation in the file changed.

@sonar-eks sonar-eks / SonarQube Code Analysis

component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/ComponentManager.java#L1335 

Refactor this method to reduce its Cognitive Complexity from 25 to the 15 allowed.

Check failure on line 916 in component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/ComponentManager.java

See this annotation in the file changed.

@sonar-eks sonar-eks / SonarQube Code Analysis

component-runtime-manager/src/main/java/org/talend/sdk/component/runtime/manager/ComponentManager.java#L916 

Refactor this method to reduce its Cognitive Complexity from 31 to the 15 allowed.

Check warning on line 415 in container/container-core/src/main/java/org/talend/sdk/component/container/ContainerManager.java

See this annotation in the file changed.

@sonar-eks sonar-eks / SonarQube Code Analysis

container/container-core/src/main/java/org/talend/sdk/component/container/ContainerManager.java#L415 

Extract this nested try block into a separate method.

Check failure on line 110 in container/container-core/src/main/java/org/talend/sdk/component/container/ContainerManager.java

See this annotation in the file changed.

@sonar-eks sonar-eks / SonarQube Code Analysis

container/container-core/src/main/java/org/talend/sdk/component/container/ContainerManager.java#L110 

Refactor this method to reduce its Cognitive Complexity from 18 to the 15 allowed.

Check notice on line 147 in container/container-core/src/main/java/org/talend/sdk/component/container/Container.java

See this annotation in the file changed.

@sonar-eks sonar-eks / SonarQube Code Analysis

container/container-core/src/main/java/org/talend/sdk/component/container/Container.java#L147 

Remove the parentheses around the "name" parameter

Check failure on line 564 in container/container-core/src/main/java/org/talend/sdk/component/classloader/ConfigurableClassLoader.java

See this annotation in the file changed.

@sonar-eks sonar-eks / SonarQube Code Analysis

container/container-core/src/main/java/org/talend/sdk/component/classloader/ConfigurableClassLoader.java#L564 

Use try-with-resources or close this "JarFile" in a "finally" clause.

Check notice on line 482 in container/container-core/src/main/java/org/talend/sdk/component/classloader/ConfigurableClassLoader.java

See this annotation in the file changed.

@sonar-eks sonar-eks / SonarQube Code Analysis

container/container-core/src/main/java/org/talend/sdk/component/classloader/ConfigurableClassLoader.java#L482 

Complete the task associated to this TODO comment.

Check warning on line 409 in container/container-core/src/main/java/org/talend/sdk/component/classloader/ConfigurableClassLoader.java

See this annotation in the file changed.

@sonar-eks sonar-eks / SonarQube Code Analysis

container/container-core/src/main/java/org/talend/sdk/component/classloader/ConfigurableClassLoader.java#L409 

"NullPointerException" will be thrown when invoking method "isNestedDependencyResource()".

Check warning on line 132 in container/container-core/src/main/java/org/talend/sdk/component/classloader/ConfigurableClassLoader.java

See this annotation in the file changed.

@sonar-eks sonar-eks / SonarQube Code Analysis

container/container-core/src/main/java/org/talend/sdk/component/classloader/ConfigurableClassLoader.java#L132 

Constructor has 8 parameters, which is greater than 7 authorized.

Check notice on line 126 in container/container-core/src/main/java/org/talend/sdk/component/classloader/ConfigurableClassLoader.java

See this annotation in the file changed.

@sonar-eks sonar-eks / SonarQube Code Analysis

container/container-core/src/main/java/org/talend/sdk/component/classloader/ConfigurableClassLoader.java#L126 

Remove the parentheses around the "name" parameter
View more details on sonar-eks