WIP : Preview Runner Push based model #16038

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

Draft

sidhdirenge wants to merge 11 commits into develop from sidhdirenge-preview-runner

Draft

WIP : Preview Runner Push based model #16038

Fix NPE during stop preview

GitHub Actions / Checkstyle completed Aug 21, 2025 in 0s

Checkstyle report

42 violation(s) found

Annotations

Check warning on line 16 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/preview/DefaultPreviewStore.java

github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.EmptyLineSeparatorCheck

'package' should be separated from previous line.

Check warning on line 391 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/preview/DefaultPreviewStore.java

github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.indentation.CommentsIndentationCheck

Comment has incorrect indentation level 0, expected is 4, indentation should be the same level as line 377.

Check warning on line 53 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/preview/PreviewRequestPollerService.java

github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocParagraphCheck

<p> tag should be placed immediately before the first word, with no space after.

Check warning on line 53 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/preview/PreviewRequestPollerService.java

github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocParagraphCheck

<p> tag should be preceded with an empty line.

Check warning on line 64 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/preview/PreviewRequestPollerService.java

github-actions / Checkstyle

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

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

Check warning on line 71 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/preview/PreviewRequestPollerService.java

github-actions / Checkstyle

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

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

Check warning on line 132 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/preview/DefaultPreviewManager.java

github-actions / Checkstyle

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

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

Check warning on line 133 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/preview/DefaultPreviewManager.java

github-actions / Checkstyle

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

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

Check warning on line 134 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/preview/DefaultPreviewManager.java

github-actions / Checkstyle

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

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

Check warning on line 138 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/preview/DefaultPreviewManager.java

github-actions / Checkstyle

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

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

Check warning on line 158 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/preview/DefaultPreviewManager.java

github-actions / Checkstyle

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

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

Check warning on line 159 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/preview/DefaultPreviewManager.java

github-actions / Checkstyle

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

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

Check warning on line 160 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/preview/DefaultPreviewManager.java

github-actions / Checkstyle

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

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

Check warning on line 161 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/preview/DefaultPreviewManager.java

github-actions / Checkstyle

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

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

Check warning on line 1 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/preview/PreviewRunnerHttpHandlerInternal.java

github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.NewlineAtEndOfFileCheck

File does not end with a newline.

Check warning on line 46 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/preview/PreviewRunnerHttpHandlerInternal.java

github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocParagraphCheck

<p> tag should be placed immediately before the first word, with no space after.

Check warning on line 46 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/preview/PreviewRunnerHttpHandlerInternal.java

github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.JavadocParagraphCheck

<p> tag should be preceded with an empty line.

Check warning on line 39 in cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/preview/PreviewDataSubscriberService.java

github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck

Wrong lexicographical order for 'io.cdap.cdap.messaging.context.MultiThreadMessagingContext' import. Should be before 'io.cdap.cdap.messaging.spi.MessagingService'.

Check warning on line 16 in cdap-app-fabric/src/main/java/io/cdap/cdap/app/store/preview/PreviewStore.java

github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.whitespace.EmptyLineSeparatorCheck

'package' should be separated from previous line.

Check warning on line 56 in cdap-app-fabric/src/main/java/io/cdap/cdap/app/store/preview/PreviewStore.java

github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck

First sentence of Javadoc is missing an ending period.

Check warning on line 63 in cdap-app-fabric/src/main/java/io/cdap/cdap/app/store/preview/PreviewStore.java

github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck

First sentence of Javadoc is missing an ending period.

Check warning on line 74 in cdap-app-fabric/src/main/java/io/cdap/cdap/app/store/preview/PreviewStore.java

github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck

First sentence of Javadoc is missing an ending period.

Check warning on line 83 in cdap-app-fabric/src/main/java/io/cdap/cdap/app/store/preview/PreviewStore.java

github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck

First sentence of Javadoc is missing an ending period.

Check warning on line 91 in cdap-app-fabric/src/main/java/io/cdap/cdap/app/store/preview/PreviewStore.java

github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck

First sentence of Javadoc is missing an ending period.

Check warning on line 110 in cdap-app-fabric/src/main/java/io/cdap/cdap/app/store/preview/PreviewStore.java

github-actions / Checkstyle

com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck

Summary javadoc is missing.

View more details on GitHub Actions