linkedin / avro-util

Collection of utilities to allow writing java code that operates across a wide range of avro versions.
BSD 2-Clause "Simplified" License
76 stars 59 forks source link

[fast-avro] Added logic to split large method for fast serializer #514

Closed gaojieliu closed 1 year ago

gaojieliu commented 1 year ago

In the past, we updated fast deserializer to support large method splitting, and this code change is leveraging the similar logic to split large method in fast serializer.

codecov-commenter commented 1 year ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (c16ffa1) 45.82% compared to head (d3bc89e) 45.83%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #514 +/- ## ========================================= Coverage 45.82% 45.83% - Complexity 4438 4440 +2 ========================================= Files 398 398 Lines 28040 28040 Branches 4622 4622 ========================================= + Hits 12850 12851 +1 Misses 13634 13634 + Partials 1556 1555 -1 ``` [see 7 files with indirect coverage changes](https://app.codecov.io/gh/linkedin/avro-util/pull/514/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=linkedin)

:umbrella: View full report in Codecov by Sentry.

:loudspeaker: Have feedback on the report? Share it here.

gaojieliu commented 1 year ago

LGTM, can you update the title of PR method->record.

It is large method actually. Large method can't be optimized by JIT by default.