Skip to content

Switch CI to Java 21 #72

Switch CI to Java 21

Switch CI to Java 21 #72

Triggered via push January 12, 2024 11:43
Status Failure
Total duration 1m 59s
Artifacts 1

release.yaml

on: push
Fit to window
Zoom out
Zoom in

Annotations

1 error and 5 warnings
release
Process completed with exit code 1.
release: src/main/java/pl/net/was/trino/git/GitConnector.java#L53
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: GitTransactionHandle.
release: src/main/java/pl/net/was/trino/git/GitSplitManager.java#L112
Return value of methods returning Future must be checked. Ignoring returned Futures suppresses exceptions thrown from the code that completes the Future.
release: src/test/java/pl/net/was/trino/git/TestGitClient.java#L57
Streams that encapsulate a closeable resource should be closed using try-with-resources
release: src/test/java/pl/net/was/trino/git/TestGitClient.java#L72
Date has a bad API that leads to bugs; prefer java.time.Instant or LocalDate.
release: src/test/java/pl/net/was/trino/git/TestGitClient.java#L82
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 release Expired
246 KB