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

[WIP] test: add v8 serialize benchmark #8

Closed
wants to merge 2 commits into from
Closed

Conversation

fengmk2
Copy link
Member

@fengmk2 fengmk2 commented Oct 30, 2018

Checklist
  • npm test passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)
Description of change

@fengmk2
Copy link
Member Author

fengmk2 commented Oct 30, 2018

效果不错

image

@gxcsoccer
Copy link
Member

除了 Error,其他都可以完全兼容了(Function 本来就不考虑)

@fengmk2
Copy link
Member Author

fengmk2 commented Nov 11, 2018

@gxcsoccer 背景的是序列化性能很差。。。

image

@codecov
Copy link

codecov bot commented Nov 11, 2018

Codecov Report

Merging #8 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master      #8   +/-   ##
======================================
  Coverage    97.1%   97.1%           
======================================
  Files           3       3           
  Lines         207     207           
======================================
  Hits          201     201           
  Misses          6       6

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4854da3...7587da7. Read the comment docs.

@fengmk2 fengmk2 closed this Jan 6, 2023
@fengmk2 fengmk2 deleted the v8-serialize branch January 6, 2023 18:00
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