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

Bulk Embedded document resolution #1031

Closed
wants to merge 1 commit into from

Conversation

Amedeo91
Copy link

@Amedeo91 Amedeo91 commented Jun 20, 2017

With this PR, the goal is to improve the performance of the feature (only with the version feature turned off) by decreasing the round-trip to the database

Issue: #1029

@Amedeo91 Amedeo91 force-pushed the Emdebbed_perfomace branch 6 times, most recently from 0c46419 to ade1dfa Compare June 21, 2017 01:50
@Amedeo91 Amedeo91 force-pushed the Emdebbed_perfomace branch from ade1dfa to da83e7f Compare June 21, 2017 02:33
nicolaiarocci added a commit that referenced this pull request Jun 21, 2017
@nicolaiarocci
Copy link
Member

Merged with rebase, see dfb2225

Thanks!

Amedeo91 pushed a commit to Amedeo91/eve that referenced this pull request Jun 21, 2017
Amedeo91 pushed a commit to Amedeo91/eve that referenced this pull request Jun 21, 2017
lexhung added a commit to lexhung/eve that referenced this pull request Nov 1, 2017
* 'master' of https://github.com/pyeve/eve: (374 commits)
  Changelog for pyeve#1070
  use OrderedDict from backport_collections
  Minor changelog fixes for pyeve#1048
  Support Decimal type MongoDB
  Changelog: add reference to proper PR
  Changelog update for pyeve#1042
  Fix a serialization bug that randomly skips fields if "x_of" is encountered
  Amedeo Bussi
  Changelog for pyeve#1030
  pep/flake, and remove duplicate test
  documentation improvements and fixes
  Bulk delete
  MONGO_DBNAME can be now used along with MONGO_URI
  Vasilis Lolis
  typo
  Amedeo91
  Changelog for pyeve#1031
  Bulk Embedded document resolution
  Delete unnecessary code
  Fix insidious bug in tests.TestPost class
  ...
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