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

Changed column to retrieve documents. #2826

Merged
merged 1 commit into from May 29, 2017

Conversation

Projects
None yet
4 participants
@whitingjr
Contributor

whitingjr commented May 26, 2017

This PR solves why results for vertx-web and MongoDB are as they are. The vertx-web application has been changed to use the proper column for MongoDB queries.

@mention-bot

This comment has been minimized.

Show comment
Hide comment
@mention-bot

mention-bot May 26, 2017

Thanks @whitingjr for contributing to The Framework Benchmarks! @pmlopes and @garricko, code you've worked on has been modified. If you have the chance, please review. If you wish to unsubscribe from these notices, please open a Pull Request with the commit message [ci skip] and your github name added to the userBlacklist array in the .mention-bot file.

mention-bot commented May 26, 2017

Thanks @whitingjr for contributing to The Framework Benchmarks! @pmlopes and @garricko, code you've worked on has been modified. If you have the chance, please review. If you wish to unsubscribe from these notices, please open a Pull Request with the commit message [ci skip] and your github name added to the userBlacklist array in the .mention-bot file.

@pmlopes

This comment has been minimized.

Show comment
Hide comment
@pmlopes

pmlopes May 29, 2017

Contributor

This looks good for me and the CI failures are not related or from the vertx-web code at all.

Contributor

pmlopes commented May 29, 2017

This looks good for me and the CI failures are not related or from the vertx-web code at all.

@nbrady-techempower nbrady-techempower merged commit 40a6222 into TechEmpower:master May 29, 2017

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment