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

perf(query-runner): use Date.now() intead of +new Date() #10811

Open

Samuron wants to merge 1 commit into typeorm:master from Samuron:date_now

Samuron commented Apr 2, 2024

Description of change

Replaced +new Date() with Date.now() in query runners to avoid two Date objects allocations per query.

Pull-Request Checklist

Code is up-to-date with the master branch
npm run format to apply prettier formatting
npm run test passes with this change
This pull request links relevant issues as Fixes #0000
There are new or updated unit tests validating the change
Documentation has been updated to reflect this change
The new commits follow conventions explained in CONTRIBUTING.md


          perf(query-runner): use Date.now() intead of +new Date()

b48d5cd

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment