Skip to content
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.

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

[CDAP-20993] Periodic refresh implementation of source control metadata for SCM sync status #15626

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

added implementation of peridic refresh for source control metadata

17956c8
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Open

[CDAP-20993] Periodic refresh implementation of source control metadata for SCM sync status #15626

added implementation of peridic refresh for source control metadata
17956c8
Select commit
Loading
Failed to load commit list.
GitHub Actions / Checkstyle completed May 6, 2024 in 0s

Checkstyle report

37 violation(s) found

Annotations

Check warning on line 61 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/services/ApplicationLifecycleService.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.imports.UnusedImportsCheck

Unused import - io.cdap.cdap.common.ForbiddenException.

Check warning on line 454 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/services/ApplicationLifecycleService.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck

All overloaded methods should be placed next to each other. Placing non-overloaded methods in between overloaded methods with the same type is a violation. Previous overloaded method located at line '425'.

Check warning on line 47 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/DefaultStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.imports.UnusedImportsCheck

Unused import - io.cdap.cdap.common.utils.ImmutablePair.

Check warning on line 571 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/DefaultStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck

All overloaded methods should be placed next to each other. Placing non-overloaded methods in between overloaded methods with the same type is a violation. Previous overloaded method located at line '557'.

Check warning on line 632 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/DefaultStore.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 633 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/DefaultStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck

'else' is not followed by whitespace.

Check warning on line 633 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/DefaultStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck

WhitespaceAround: 'else' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3)

Check warning on line 633 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/DefaultStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck

WhitespaceAround: '{' is not preceded with whitespace.

Check warning on line 634 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/DefaultStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck

'if' is not followed by whitespace.

Check warning on line 634 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/DefaultStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck

WhitespaceAround: 'if' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3)

Check warning on line 658 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/DefaultStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

RightCurlySame

'}' at column 9 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 659 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/DefaultStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck

'else' is not followed by whitespace.

Check warning on line 659 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/DefaultStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck

WhitespaceAround: 'else' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3)

Check warning on line 659 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/DefaultStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck

WhitespaceAround: '{' is not preceded with whitespace.

Check warning on line 660 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/DefaultStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck

'if' is not followed by whitespace.

Check warning on line 660 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/DefaultStore.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAroundCheck

WhitespaceAround: 'if' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3)

Check warning on line 382 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 355 in cdap-app-fabric/src/main/java/io/cdap/cdap/app/store/Store.java

See this annotation in the file changed.

@github-actions github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck

All overloaded methods should be placed next to each other. Placing non-overloaded methods in between overloaded methods with the same type is a violation. Previous overloaded method located at line '322'.

Check warning on line 113 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.whitespace.EmptyLineSeparatorCheck

'METHOD_DEF' should be separated from previous line.

Check warning on line 119 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 362 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 445 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 556 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 652 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 693 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.