Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(example/sqlite): use Type::UNKNOWN when Column::dec_type returns None. #163

Merged
merged 5 commits into from
Mar 5, 2024

Conversation

KKould
Copy link
Contributor

@KKould KKould commented Mar 4, 2024

login info: LoginInfo { user: Some("28156"), database: Some("28156"), host: "127.0.0.1" }
thread 'tokio-runtime-worker' panicked at examples\sqlite.rs:242:37:
called `Result::unwrap()` on an `Err` value: PoisonError { .. }
stack backtrace:
   0: std::panicking::begin_panic_handler
             at /rustc/25f8d01fd8bda339612d0c0a8844173a09205f7c/library\std\src\panicking.rs:647
   1: core::panicking::panic_fmt

After I fixed the previous error, there was a keep live column during the connection, causing an Option::unwrap error.
image

examples/sqlite.rs Outdated Show resolved Hide resolved
@KKould KKould changed the title fix: lock throw PoisonError and unwrap fix(example/sqlite): lock throw PoisonError and unwrap Mar 4, 2024
examples/sqlite.rs Outdated Show resolved Hide resolved
@KKould KKould changed the title fix(example/sqlite): lock throw PoisonError and unwrap fix(example/sqlite): use Type::UNKNOWN when Column::dec_type returns None. Mar 4, 2024
@KKould KKould requested a review from sunng87 March 4, 2024 12:39
@sunng87
Copy link
Owner

sunng87 commented Mar 4, 2024

@KKould Great! Please fix the format issue and I will merge this. Thank you for your patient!

@KKould
Copy link
Contributor Author

KKould commented Mar 4, 2024

@KKould Great! Please fix the format issue and I will merge this. Thank you for your patient!

Sorry I forgot to execute cargo fmt😭😭😭

@sunng87 sunng87 merged commit f108053 into sunng87:master Mar 5, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants