Skip to content
SonarQube Analysis

Sonarqube Fix - Ensure optional is present before get #353

Sign in to view logs

Sonarqube Fix - Ensure optional is present before get

Sonarqube Fix - Ensure optional is present before get #353

Triggered via pull request January 15, 2025 21:22
@mikaylathompsonmikaylathompson
synchronize #1233
mikaylathompson:client-factory-sonarqube-fix
Status Success
Total duration 4m 49s
Artifacts 1

sonar-qube.yml

on: pull_request
Run SonarQube Analysis
4m 39s
Run SonarQube Analysis
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 1 warning
java:S1874: RFS/src/main/java/org/opensearch/migrations/bulkload/common/LuceneDocumentsReader.java#L167
Remove this use of "document"; it is deprecated.
java:S3038: RFS/src/main/java/org/opensearch/migrations/bulkload/models/IndexMetadata.java#L33
"getName" is defined in the "Index" interface and can be removed from this class.
java:S1141: RFS/src/main/java/org/opensearch/migrations/bulkload/version_os_2_11/IndexCreator_OS_2_11.java#L47
Extract this nested try block into a separate method.
java:S1168: RFS/src/main/java/org/opensearch/migrations/bulkload/workcoordination/OpenSearchWorkCoordinator.java#L385
Return an empty collection instead of null.
java:S135: RFS/src/main/java/org/opensearch/migrations/bulkload/workcoordination/OpenSearchWorkCoordinator.java#L1010
Reduce the total number of break and continue statements in this loop to use at most one.
java:S1192: RFS/src/main/java/org/opensearch/migrations/bulkload/worker/IndexAndShardCursor.java#L20
Define a constant instead of duplicating this literal "Illegal work item name: '" 3 times.
java:S1141: RFS/src/main/java/org/opensearch/migrations/bulkload/worker/IndexRunner.java#L70
Extract this nested try block into a separate method.
java:S7158: TrafficCapture/captureOffloader/src/main/java/org/opensearch/migrations/trafficcapture/StreamChannelConnectionCaptureSerializer.java#L280
Use isEmpty() to check whether a string is empty or not.
java:S7158: TrafficCapture/captureOffloader/src/main/java/org/opensearch/migrations/trafficcapture/StreamChannelConnectionCaptureSerializer.java#L368
Use isEmpty() to check whether a string is empty or not.
python:S117: TrafficCapture/dockerSolution/src/main/docker/k8sConfigMapUtilScripts/configmap2yaml/format_services_yaml.py#L42
Rename this parameter "outStream" to match the regular expression ^[_a-z][a-z0-9_]*$.
Run SonarQube Analysis
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636

Artifacts

Produced during runtime
Name Size
sonar-reports
7.72 KB