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

updated quaint to pass bench tests #3942

Closed
wants to merge 1 commit into from
Closed

updated quaint to pass bench tests #3942

wants to merge 1 commit into from

Conversation

sabify
Copy link
Contributor

@sabify sabify commented Feb 20, 2024

No description provided.

@weiznich weiznich requested a review from a team February 20, 2024 11:58
@sabify sabify closed this Feb 20, 2024
@sabify
Copy link
Contributor Author

sabify commented Feb 20, 2024

@weiznich Quaint is under very active development, and there are feature set inconsistencies in the crate that cause compile errors. It would be better if this crate were removed from diesel_bench, as it is a blocking feature for benchmarking postgres integration.

The postgres benchmarks of diesel metrics haven't received updates for months.

@weiznich
Copy link
Member

I know. I'm happy to accept a PR fixing that, otherwise there are just more important things for me to work on currently.

@sabify
Copy link
Contributor Author

sabify commented Feb 22, 2024

@weiznich please check out #3943.

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