Compiling main for Trino 428 to backport https://github.com/snowlift/… #31
nightkrAnnotations
9 warnings
|
Deprecation notice: v1, v2, and v3 of the artifact actions
The following artifacts were uploaded using a version of actions/upload-artifact that is scheduled for deprecation: "test report maven-checks".
Please update your workflow to use v4 of the artifact actions.
Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/
|
|
maven-checks
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-java@v3, actions/upload-artifact@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
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.
|
|
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/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/JsonPlugin.java#L58
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#L116
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/ExcelPlugin.java#L52
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/StorageConnector.java#L62
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.
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
test report maven-checks
Expired
|
74.1 KB |
|