v8.deserialize is very slow, is it normal?

[navegador5]

• Version: tested on bth v16.0.0-pre. and v12.18.2
• Platform: linux
• Subsystem:ubuntu

What steps will reproduce the bug?

My application sometimes need to keep the undefined/[Circular] when do deepcopy of a object,
so i tried v8.deserialize, it work, but very slow, is this normal or abnormal ?

    const v8 = require("v8")
    
    function deep_copy(o) {
        let buf = v8.serialize(o)
        let nobj = v8.deserialize(buf)
        return(nobj)
    }
    
    
    function orig_dcp(j) {
        return(JSON.parse(JSON.stringify(j)))
    }
    
    function slow_dcp(j) {
        return(deep_copy(j))
    }

    var json = {1:[1,2,3],2:{'a':undefined}}

    function tst(times,f,...args) {
        let start = perf_hooks.performance.nodeTiming.duration
        c= 0
        while(c<times) {f(...args);c=c+1}
        let end = perf_hooks.performance.nodeTiming.duration
        console.log(end-start)
    }
    

> orig_dcp(json)
{ '1': [ 1, 2, 3 ], '2': {} }
>
undefined
> slow_dcp(json)
{ '1': [ 1, 2, 3 ], '2': { a: undefined } }
>
undefined
> tst(50000,orig_dcp,json)
151.65841102600098
undefined
> tst(50000,slow_dcp,json)
2831.4293529987335
undefined
>

How often does it reproduce? Is there a required condition?

always

What is the expected behavior?

I dont know ,just too slow

What do you see instead?

nearly 20 times slow

Additional information

[BridgeAR]

@nodejs/v8 PTAL

@navegador5 the algorithm is definitely more complex than JSON and will always require more time. There might still be room for improvement but I did not look at it as Node.js just provides access to the v8 c++ implementation.

[camillobruni]

Would it be possible to have the serialize/stringify and deserialize/parse numbers separately?

[MoLow]

as written in #48137 - node does not do much besides passing the call to v8, so this issue should be reported to v8