Skip to content

Run tests

Run tests #847

Triggered via pull request September 25, 2024 21:34
@ebyhrebyhr
opened #342
ebi/ci3
Status Success
Total duration 2m 53s
Artifacts 1

main.yml

on: pull_request

Annotations

7 warnings
maven-checks: src/main/java/org/ebyhr/trino/storage/StorageConnector.java#L63
Importing nested classes/static methods/static fields with commonly-used names can make code harder to read, because it may not be clear from the context exactly which type is being referred to. Qualifying the name with that of the containing class can make the code clearer. Here we recommend using qualified class: StorageTransactionHandle.
maven-checks: src/main/java/org/ebyhr/trino/storage/StorageRecordSetProvider.java#L68
Using stream::iterator creates a one-shot Iterable, which may cause surprising failures.
maven-checks: src/main/java/org/ebyhr/trino/storage/operator/ExcelPlugin.java#L51
It is very rare for LinkedList to out-perform ArrayList or ArrayDeque. Avoid it unless you're willing to invest a lot of time into benchmarking. Caveat: LinkedList supports null elements, but ArrayDeque does not.
maven-checks: src/main/java/org/ebyhr/trino/storage/operator/JsonPlugin.java#L63
Implicit use of the platform default charset, which can result in differing behaviour between JVM executions or incorrect behavior if the encoding of the data source doesn't match expectations.
maven-checks: src/main/java/org/ebyhr/trino/storage/operator/JsonPlugin.java#L121
Implicit use of the platform default charset, which can result in differing behaviour between JVM executions or incorrect behavior if the encoding of the data source doesn't match expectations.
maven-checks: src/main/java/org/ebyhr/trino/storage/operator/RawPlugin.java#L42
Implicit use of the platform default charset, which can result in differing behaviour between JVM executions or incorrect behavior if the encoding of the data source doesn't match expectations.
maven-checks: src/main/java/org/ebyhr/trino/storage/operator/TextPlugin.java#L40
Implicit use of the platform default charset, which can result in differing behaviour between JVM executions or incorrect behavior if the encoding of the data source doesn't match expectations.

Artifacts

Produced during runtime
Name Size
test report maven-checks Expired
21.9 KB