Skip to content
SonarQube Analysis

Address final review comments #20

Sign in to view logs

Address final review comments

Address final review comments #20

Triggered via push January 14, 2025 21:18
@mikaylathompsonmikaylathompson
pushed 8253d36
client-for-es-6
Status Success
Total duration 4m 7s
Artifacts 1

sonar-qube.yml

on: push
Run SonarQube Analysis
3m 56s
Run SonarQube Analysis
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 1 warning
java:S1611: RFS/src/main/java/org/opensearch/migrations/bulkload/common/LuceneDocumentsReader.java#L124
Remove the parentheses around the "leafReader" parameter
java:S1874: RFS/src/main/java/org/opensearch/migrations/bulkload/common/LuceneDocumentsReader.java#L295
Remove this use of "document"; it is deprecated.
java:S112: RFS/src/main/java/org/opensearch/migrations/bulkload/common/OpenSearchClientFactory.java#L49
Define and throw a dedicated exception instead of using a generic one.
java:S112: RFS/src/main/java/org/opensearch/migrations/bulkload/common/OpenSearchClientFactory.java#L62
Define and throw a dedicated exception instead of using a generic one.
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:S135: RFS/src/main/java/org/opensearch/migrations/bulkload/workcoordination/OpenSearchWorkCoordinator.java#L1033
Reduce the total number of break and continue statements in this loop to use at most one.
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.
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.65 KB