Skip to content

Merge pull request #15989 from cdapio/simplified-upgrades-list-API

097994c
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Draft

[Draft] Simplified upgrades WIP branch. #15964

Merge pull request #15989 from cdapio/simplified-upgrades-list-API
097994c
Select commit
Loading
Failed to load commit list.
GitHub Actions / Checkstyle completed Jun 26, 2025 in 0s

Checkstyle report

34 violation(s) found

Annotations

Check warning on line 310 in cdap-app-fabric/src/main/java/io/cdap/cdap/gateway/handlers/AppLifecycleHttpHandler.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.coding.MissingSwitchDefaultCheck

switch without "default" clause.

Check warning on line 503 in cdap-app-fabric/src/main/java/io/cdap/cdap/gateway/handlers/AppLifecycleHttpHandler.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.AtclauseOrderCheck

Javadoc comment at column 65 has parse error. Details: no viable alternative at input '</>' while parsing HTML_ELEMENT

Check warning on line 98 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/upgrade/MetadataApplicationPluginMappingFetcher.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

SeparatorWrapDot

'.' should be on a new line.

Check warning on line 99 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/upgrade/MetadataApplicationPluginMappingFetcher.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

SeparatorWrapDot

'.' should be on a new line.

Check warning on line 52 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/upgrade/DefaultUpgradeManager.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocMethodCheck

Missing a Javadoc comment.

Check warning on line 90 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/upgrade/DefaultUpgradeManager.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

SeparatorWrapDot

'.' should be on a new line.

Check warning on line 91 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/upgrade/DefaultUpgradeManager.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

SeparatorWrapDot

'.' should be on a new line.

Check warning on line 1 in cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/upgrade/DefaultUpgradeManagerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.NewlineAtEndOfFileCheck

File does not end with a newline.

Check warning on line 1 in cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/upgrade/MetadataApplicationPluginMappingFetcherTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.NewlineAtEndOfFileCheck

File does not end with a newline.

Check warning on line 137 in cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/AppLifecycleHttpHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck

Abbreviation in name 'setLCMFlag' must contain no more than '1' consecutive capital letters.

Check warning on line 379 in cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/AppLifecycleHttpHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck

Distance between variable 'request' declaration and its first usage is 4, but allowed 3.  Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).

Check warning on line 462 in cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/AppLifecycleHttpHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck

Abbreviation in name 'testDeployVersionedAndNonVersionedAppLCMFlagEnabled' must contain no more than '1' consecutive capital letters.

Check warning on line 573 in cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/AppLifecycleHttpHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck

Abbreviation in name 'testListAndGetLCMFlagEnabledInstanceUpgradeScenario' must contain no more than '1' consecutive capital letters.

Check warning on line 669 in cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/AppLifecycleHttpHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck

Abbreviation in name 'testListAndGetForPaginatedAPI' must contain no more than '1' consecutive capital letters.

Check warning on line 740 in cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/AppLifecycleHttpHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

RightCurlySame

'}' at column 7 should be on the same line as the next part of a multi-block statement (one that directly contains multiple blocks: if/else-if/else, do/while or try/catch/finally).

Check warning on line 811 in cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/AppLifecycleHttpHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck

Abbreviation in name 'testListAndGetForPaginatedAPIWithEmptyLastPage' must contain no more than '1' consecutive capital letters.

Check warning on line 852 in cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/AppLifecycleHttpHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck

Abbreviation in name 'testListAndGetForPaginatedAPIWithFiltering' must contain no more than '1' consecutive capital letters.

Check warning on line 898 in cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/AppLifecycleHttpHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck

Abbreviation in name 'testListAndGetForPaginatedAPIWithNameFilterType' must contain no more than '1' consecutive capital letters.

Check warning on line 941 in cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/AppLifecycleHttpHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck

Abbreviation in name 'testListAndGetForPaginatedAPIWithLatestOnlyLCMFlagEnabled' must contain no more than '1' consecutive capital letters.

Check warning on line 1011 in cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/AppLifecycleHttpHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck

Abbreviation in name 'testListAndGetLCMFlagEnabledForVersionHistory' must contain no more than '1' consecutive capital letters.

Check warning on line 1157 in cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/AppLifecycleHttpHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck

Abbreviation in name 'testListAndGetLCMFlagEnabled' must contain no more than '1' consecutive capital letters.

Check warning on line 1282 in cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/AppLifecycleHttpHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck

Distance between variable 'appReference' declaration and its first usage is 5, but allowed 3.  Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).

Check warning on line 1430 in cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/AppLifecycleHttpHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck

Abbreviation in name 'testDeleteLCMFlagEnabled' must contain no more than '1' consecutive capital letters.

Check warning on line 1445 in cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/AppLifecycleHttpHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck

Distance between variable 'applicationId' declaration and its first usage is 5, but allowed 3.  Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).

Check warning on line 1561 in cdap-app-fabric/src/test/java/io/cdap/cdap/internal/app/services/http/handlers/AppLifecycleHttpHandlerTest.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck

Abbreviation in name 'testDeleteVersionedAppLCMEnabled' must contain no more than '1' consecutive capital letters.