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

Speedup debezium parse #196

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Speedup debezium parse #196

wants to merge 1 commit into from

Conversation

laskoviymishka
Copy link
Contributor

Before:

BenchmarkReceiverTest
BenchmarkReceiverTest/parse
BenchmarkReceiverTest/parse/tz
BenchmarkReceiverTest/parse/tz-10      	   53989	     22082 ns/op	  15.58 MB/s	    9008 B/op	      55 allocs/op
BenchmarkReceiverTest/parse/no-tz
BenchmarkReceiverTest/parse/no-tz-10   	   54669	     21947 ns/op	  15.67 MB/s	    9007 B/op	      55 allocs/op
PASS

After

BenchmarkReceiverTest
BenchmarkReceiverTest/parse
BenchmarkReceiverTest/parse/citext
BenchmarkReceiverTest/parse/citext-10  	  134011	      8474 ns/op	  39.30 MB/s	   12149 B/op	      36 allocs/op
BenchmarkReceiverTest/parse/tz
BenchmarkReceiverTest/parse/tz-10      	  130683	      9110 ns/op	  37.76 MB/s	   12679 B/op	      36 allocs/op
BenchmarkReceiverTest/parse/no-tz
BenchmarkReceiverTest/parse/no-tz-10   	  135462	      8913 ns/op	  38.60 MB/s	   12679 B/op	      36 allocs/op

Main gain from faster json parse, total gain x3

Before:
```
```

After
```
BenchmarkReceiverTest
BenchmarkReceiverTest/parse
BenchmarkReceiverTest/parse/citext
BenchmarkReceiverTest/parse/citext-10  	  134011	      8474 ns/op	  39.30 MB/s	   12149 B/op	      36 allocs/op
BenchmarkReceiverTest/parse/tz
BenchmarkReceiverTest/parse/tz-10      	  130683	      9110 ns/op	  37.76 MB/s	   12679 B/op	      36 allocs/op
BenchmarkReceiverTest/parse/no-tz
BenchmarkReceiverTest/parse/no-tz-10   	  135462	      8913 ns/op	  38.60 MB/s	   12679 B/op	      36 allocs/op
```

Main gain from faster json parse, total gain x3
@laskoviymishka
Copy link
Contributor Author

:shipit:

Copy link

robot-magpie bot commented Feb 3, 2025

@laskoviymishka has imported your pull request. If you are a member of Transfer team, you can view this diff.

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.

1 participant