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

RUST-993 Implement Clone for Collection<T> #454

Merged
merged 1 commit into from Sep 9, 2021

Conversation

awitten1
Copy link
Contributor

@awitten1 awitten1 commented Sep 8, 2021

Because derive is too conservative, derive only implements Clone for Collection<T> if T is Clone.
Collection<T> does not actually store any value of type T (so T does not need to be Clone).

@awitten1 awitten1 changed the title Implement Clone for Collection<T> RUST-993 Implement Clone for Collection<T> Sep 8, 2021
@patrickfreed patrickfreed self-requested a review September 8, 2021 19:25
Copy link
Contributor

@patrickfreed patrickfreed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! I took a look at the failures in the evergreen patch and they look to be known flaky tests.

Tagging in the other members of the team for review.

@patrickfreed
Copy link
Contributor

Once this PR is ready to merge, just hit the "Squash and Merge" button and make sure the commit title matches the PR title (i.e. includes ticket number and brief description).

Copy link
Contributor

@abr-egn abr-egn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Contributor

@isabelatkinson isabelatkinson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@awitten1 awitten1 merged commit 660cdf7 into mongodb:master Sep 9, 2021
awitten1 added a commit to awitten1/mongo-rust-driver that referenced this pull request Sep 9, 2021
Because derive is too conservative, derive only implements Clone for Collection<T> if T is Clone.
Collection<T> does not actually store any value of type T (so T does not need to be Clone).
awitten1 added a commit to awitten1/mongo-rust-driver that referenced this pull request Sep 9, 2021
Because derive is too conservative, derive only implements Clone for Collection<T> if T is Clone.
Collection<T> does not actually store any value of type T (so T does not need to be Clone).
awitten1 added a commit to awitten1/mongo-rust-driver that referenced this pull request Sep 21, 2021
Because derive is too conservative, derive only implements Clone for Collection<T> if T is Clone.
Collection<T> does not actually store any value of type T (so T does not need to be Clone).
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.

None yet

4 participants