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

Jump to bottom

fix(NODE-3309): remove redundant iteration of bulk write result #2815

Merged
merged 5 commits into from
May 25, 2021
Merged

fix(NODE-3309): remove redundant iteration of bulk write result #2815

merged 5 commits into from
May 25, 2021

Commits on May 24, 2021

  1. fix(NODE-3309): remove redundant iteration of bulk write result 

    Add caching for getters that build the result object.
Add benchmarking script.
Add unit test for getter caching.
    @nbbeeken
    nbbeeken committed May 24, 2021
    Configuration menu
    Copy the full SHA
    cfc8c05 View commit details
    Browse the repository at this point in the history

  2. refactor: clean up existence and defaulting checks

    @nbbeeken
    nbbeeken committed May 24, 2021
    Configuration menu
    Copy the full SHA
    cbec8e8 View commit details
    Browse the repository at this point in the history

  3. fix: better test name

    @nbbeeken
    nbbeeken committed May 24, 2021
    Configuration menu
    Copy the full SHA
    63f6d06 View commit details
    Browse the repository at this point in the history

Commits on May 25, 2021

  1. test: split unit tests

    @nbbeeken
    nbbeeken committed May 25, 2021
    Configuration menu
    Copy the full SHA
    35d2b92 View commit details
    Browse the repository at this point in the history

  2. fix: older versions of node do not have Symbol#description

    @nbbeeken
    nbbeeken committed May 25, 2021
    Configuration menu
    Copy the full SHA
    178b99d View commit details
    Browse the repository at this point in the history