fix incorrect nullability for datetime type when writing to arrow #1097

Annotations

5 errors and 25 warnings

linking with `cc` failed: exit status: 1

could not compile `connectorx` due to previous error

Process completed with exit code 101.

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions-rs/toolchain@v1, actions/cache@v2, actions-rs/cargo@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

unused import: `get_partition_range_query`

`connectorx` (lib) generated 1 warning

unused import: `SourceType`

unused imports: `get_partition_range_query`, `single_col_partition_query`

unused import: `url::Url`

unreachable expression

unused variable: `query`

unused variable: `col`

unused variable: `query`

this operation has no effect

very complex type used. Consider factoring parts into `type` definitions

use of `expect` followed by a function call

use of `unwrap_or` followed by a function call

this `to_owned` call clones the std::borrow::Cow<str> itself and does not cause the std::borrow::Cow<str> contents to become owned

you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`

this lifetime isn't used in the impl

redundant clone

explicit lifetimes given in parameter types where they could be elided (or replaced with `'_` if needed by type declaration)

this lifetime isn't used in the impl

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/