You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@streampark.apache.org by "shangeyao (via GitHub)" <gi...@apache.org> on 2026/07/08 03:27:40 UTC

[PR] [CI] SonarCloud CPD exclusions and issue suppressions for StreamPark 3.0 [streampark]

shangeyao opened a new pull request, #4424:
URL: https://github.com/apache/streampark/pull/4424

   ## Summary
   
   - Add `.sonarcloud.properties` and `sonar-project.properties` with ASF license headers
   - Configure CPD exclusions for versioned shims, connector, kubernetes, and packer modules
   - Document multicriteria issue suppressions for intentional Scala-to-Java migration patterns
   
   Closes #4423
   
   ## Motivation
   
   SonarCloud reports hundreds of issues on migration PRs (#4418, #4419, #4420) that are **configuration-level** concerns (shims naming, connector `Serializable` fields, deprecated Flink API bridges, SQL dialect regex, client exception stubs). Keeping these suppressions in feature PRs makes review harder and couples unrelated changes.
   
   This PR isolates Sonar configuration so feature PRs can focus on code fixes only.
   
   ## Suppression rules added
   
   | Rule | Scope | Reason |
   |------|-------|--------|
   | `javasecurity:S2083` | `SafePathUtils` | Validated user config paths |
   | `java:S100` | `streampark-flink-shims*/**` | Scala-compat `$method` names |
   | `java:S1948` | `streampark-flink-connector/**` | Connector config serialization |
   | `java:S6355` / `S1123` / `S1133` | Shims + `FlinkStreamTableTrait` | Deprecated Flink API bridges |
   | `java:S8786` | `SqlConvertUtils` | MySQL→PG dialect regex |
   | `java:S112` | `streampark-*-client*/**` | Client exception propagation |
   | `java:S1186` | `Silent*.java` | Intentional no-op watchers |
   
   ## Test plan
   
   - [ ] SonarCloud scan on this PR shows only the two property files changed
   - [ ] After merge, re-run Sonar on #4418 and confirm open-issue count drops without QG regressions from config drift
   - [ ] Verify `.sonarcloud.properties` is picked up by automatic analysis
   
   
   Made with [Cursor](https://cursor.com)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@streampark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [CI] SonarCloud CPD exclusions and issue suppressions for StreamPark 3.0 [streampark]

Posted by "shangeyao (via GitHub)" <gi...@apache.org>.
shangeyao closed pull request #4424: [CI] SonarCloud CPD exclusions and issue suppressions for StreamPark 3.0
URL: https://github.com/apache/streampark/pull/4424


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@streampark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [CI] SonarCloud CPD exclusions and issue suppressions for StreamPark 3.0 [streampark]

Posted by "sonarqubecloud[bot] (via GitHub)" <gi...@apache.org>.
sonarqubecloud[bot] commented on PR #4424:
URL: https://github.com/apache/streampark/pull/4424#issuecomment-4911112151

   ## [![Quality Gate Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png 'Quality Gate Passed')](https://sonarcloud.io/dashboard?id=apache_incubator-streampark&pullRequest=4424) **Quality Gate passed**  
   Issues  
   ![](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/passed-16px.png '') [0 New issues](https://sonarcloud.io/project/issues?id=apache_incubator-streampark&pullRequest=4424&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)  
   ![](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/accepted-16px.png '') [0 Accepted issues](https://sonarcloud.io/project/issues?id=apache_incubator-streampark&pullRequest=4424&issueStatuses=ACCEPTED)
   
   Measures  
   ![](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/passed-16px.png '') [0 Security Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache_incubator-streampark&pullRequest=4424&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)  
   ![](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/passed-16px.png '') [0.0% Coverage on New Code](https://sonarcloud.io/component_measures?id=apache_incubator-streampark&pullRequest=4424&metric=new_coverage&view=list)  
   ![](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/passed-16px.png '') [0.0% Duplication on New Code](https://sonarcloud.io/component_measures?id=apache_incubator-streampark&pullRequest=4424&metric=new_duplicated_lines_density&view=list)  
     
   <!-- sqra-placement-anchor -->
   [See analysis details on SonarQube Cloud](https://sonarcloud.io/dashboard?id=apache_incubator-streampark&pullRequest=4424)
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@streampark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [CI] SonarCloud CPD exclusions and issue suppressions for StreamPark 3.0 [streampark]

Posted by "sonarqubecloud[bot] (via GitHub)" <gi...@apache.org>.
sonarqubecloud[bot] commented on PR #4424:
URL: https://github.com/apache/streampark/pull/4424#issuecomment-4946087048

   ## [![Quality Gate Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png 'Quality Gate Passed')](https://sonarcloud.io/dashboard?id=apache_incubator-streampark&pullRequest=4424) **Quality Gate passed**  
   Issues  
   ![](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/passed-16px.png '') [0 New issues](https://sonarcloud.io/project/issues?id=apache_incubator-streampark&pullRequest=4424&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)  
   ![](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/accepted-16px.png '') [0 Accepted issues](https://sonarcloud.io/project/issues?id=apache_incubator-streampark&pullRequest=4424&issueStatuses=ACCEPTED)
   
   Measures  
   ![](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/passed-16px.png '') [0 Security Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache_incubator-streampark&pullRequest=4424&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)  
   ![](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/passed-16px.png '') [0.0% Coverage on New Code](https://sonarcloud.io/component_measures?id=apache_incubator-streampark&pullRequest=4424&metric=new_coverage&view=list)  
   ![](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/passed-16px.png '') [0.0% Duplication on New Code](https://sonarcloud.io/component_measures?id=apache_incubator-streampark&pullRequest=4424&metric=new_duplicated_lines_density&view=list)  
     
   <!-- sqra-placement-anchor -->
   [See analysis details on SonarQube Cloud](https://sonarcloud.io/dashboard?id=apache_incubator-streampark&pullRequest=4424)
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@streampark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org