dmmulroy:dmmulroy/fix-serializable-type - Branch - cloudflare/workerd

fix(types): filter symbol keys in Rpc.Serializable mapped type

#5804

Comparing

dmmulroy:dmmulroy/fix-serializable-type

(

161303e

) with

main

(

08419cb

)

-19%

Improvement: 1

Untouched: 69

Skipped: 129

Benchmarks

199 total

Encode_ASCII_32[TextEncoder][0/0/32]

src/workerd/tests/bench-text-encoder.c++

+13%

3.3 ms2.9 ms

bodyWithHeaders[Response]

src/workerd/tests/bench-response.c++

24.9 µs24.8 µs

simpleStringBody[Response]

src/workerd/tests/bench-response.c++

19.1 µs19.1 µs

EncodeInto_ASCII_8192[TextEncoder][1/0/8192]

src/workerd/tests/bench-text-encoder.c++

3.3 ms3.2 ms

request[GlobalScopeBenchmark]

src/workerd/tests/bench-global-scope.c++

51.4 ms51.4 ms

bm_Http_FullProtocol

external/+http+capnp-cpp/src/capnp/compat/http-over-capnp-bench.c++

76.1 µs76.1 µs

Encode_OneByte_256[TextEncoder][0/1/256]

src/workerd/tests/bench-text-encoder.c++

5.1 ms5.1 ms

EncodeInto_ASCII_1024[TextEncoder][1/0/1024]

src/workerd/tests/bench-text-encoder.c++

2.5 ms2.5 ms

request[RegExpBenchmark]

src/workerd/tests/bench-regex.c++

7.3 ms7.3 ms

EncodeInto_TwoByte_256[TextEncoder][1/2/256]

src/workerd/tests/bench-text-encoder.c++

5.3 ms5.3 ms

bm_Http_OverCapnpFullRPC

external/+http+capnp-cpp/src/capnp/compat/http-over-capnp-bench.c++

235.6 µs235.6 µs

EncodeInto_OneByte_256[TextEncoder][1/1/256]

src/workerd/tests/bench-text-encoder.c++

4.4 ms4.4 ms

Encode_TwoByte_256[TextEncoder][0/2/256]

src/workerd/tests/bench-text-encoder.c++

7 ms7 ms

Encode_OneByte_8192[TextEncoder][0/1/8192]

src/workerd/tests/bench-text-encoder.c++

84.8 ms84.8 ms

Encode_TwoByte_8192[TextEncoder][0/2/8192]

src/workerd/tests/bench-text-encoder.c++

146.1 ms146.1 ms

EncodeInto_TwoByte_1024[TextEncoder][1/2/1024]

src/workerd/tests/bench-text-encoder.c++

14.1 ms14.1 ms

constructor[ApiHeaders]

src/workerd/tests/bench-api-headers.c++

67.6 ms67.6 ms

Util_RecursivelyFreeze

src/workerd/tests/bench-util.c++

10.2 ms10.2 ms

bm_Coro_CoAwait_ImmediateCoroutine

external/+http+capnp-cpp/src/kj/async-bench.c++

6.1 µs6.1 µs

JsString_Utf8Length_Utf16_Invalid_NonFlat_256

src/workerd/tests/bench-jsstring.c++

8.8 µs8.8 µs

JsString_Utf8Length_Utf16_NonFlat_1024

src/workerd/tests/bench-jsstring.c++

16.1 µs16.1 µs

bm_Http_ClientWrapper

external/+http+capnp-cpp/src/capnp/compat/http-over-capnp-bench.c++

35.7 µs35.7 µs

bm_Http_Baseline

external/+http+capnp-cpp/src/capnp/compat/http-over-capnp-bench.c++

11.2 µs11.2 µs

JsString_Utf8Length_Utf16_NonFlat_8192

src/workerd/tests/bench-jsstring.c++

73.7 µs73.7 µs

bm_Coro_Fib10

external/+http+capnp-cpp/src/kj/async-bench.c++

14.8 µs14.8 µs

Commits

Click on a commit to change the comparison range

Base

main

08419cb

+13.38%

fix: update type tests and snapshots for Serializable change

5eb0216

3 months ago

by petebacondarwin

-31.88%

fixup snapshots

161303e

3 months ago

by petebacondarwin

Home Terms Privacy Docs