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.

Sign up for GitHub

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

Adds feature for users to customize connections created by the pool #89

Merged
merged 8 commits into from
Dec 14, 2020
+99 −3
Merged

Adds feature for users to customize connections created by the pool #89

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
f45ac47
Adds feature for users to customize connections created by the pool
agersant Dec 12, 2020
df8139f
Fixed typo
agersant Dec 12, 2020
343e8e0
Fixed typo
agersant Dec 12, 2020
dd0859d
Cosmetic change
agersant Dec 12, 2020
9cb79fb
on_acquire_connection now uses mutable reference to connection
agersant Dec 14, 2020
cb219ed
Remove on_release support
agersant Dec 14, 2020
a340e34
Updated doc comment
agersant Dec 14, 2020
dfedd71
Stylistic changes
agersant Dec 14, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fixed typo
@agersant
agersant committed Dec 12, 2020
commit 343e8e087fdb006066687a60653453fada53747f
2 changes: 1 addition & 1 deletion bb8/tests/test.rs
View file
Original file line number Diff line number Diff line change
@@ -782,7 +782,7 @@ async fn test_customize_connection_acquire() {
assert_eq!(connection_2.custom_field, 2);
}

// Connection don't get customized again on re-use
// Connections don't get customized again on re-use
let connection_1_or_2 = pool.get().await.unwrap();
assert!(connection_1_or_2.custom_field == 1 || connection_1_or_2.custom_field == 2);
}