CD #46
Annotations
18 warnings
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: marvinpinto/[email protected]. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
The following actions uses node12 which is deprecated and will be forced to run on node16: marvinpinto/[email protected]. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
Prepare release:
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.
|
Prepare release:
src/main/java/org/ebyhr/trino/storage/StorageRecordSetProvider.java#L68
Using stream::iterator creates a one-shot Iterable, which may cause surprising failures.
|
Prepare release:
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.
|
Prepare release:
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.
|
Prepare release:
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.
|
Prepare release:
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.
|
Prepare release:
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.
|
Publish JAR
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.
|
Publish JAR
Using stream::iterator creates a one-shot Iterable, which may cause surprising failures.
|
Publish JAR
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.
|
Publish JAR
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.
|
Publish JAR
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.
|
Publish JAR
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.
|
Publish JAR
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.
|
Run marvinpinto/[email protected]
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
Run marvinpinto/[email protected]
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
Loading