Skip to content

Middleware based hydrator - #117

Merged
DavidBadura merged 3 commits into
2.0.xfrom
middleware-hydrator
Nov 29, 2025
Merged

Middleware based hydrator#117
DavidBadura merged 3 commits into
2.0.xfrom
middleware-hydrator

Conversation

@DavidBadura

@DavidBaduraDavidBadura commented Oct 29, 2025

Copy link
Copy Markdown
Member

No description provided.

@DavidBadura
DavidBadura changed the base branch from 1.12.x to 2.0.xOctober 29, 2025 13:51
@github-actions

github-actionsBot commented Oct 29, 2025•
edited
Loading

Copy link
Copy Markdown

Hello 👋

here is the most recent benchmark result:

HydratorWithCryptographyBench
=============================
+-------------------------------+-------------------+-------------------+-----------------+------------+
| | time (kde mode) | memory |
+-------------------------------+-------------------+-------------------+-----------------+------------+
| subject | Tag: <current> | Tag: base | Tag: <current> | Tag: base |
+-------------------------------+-------------------+-------------------+-----------------+------------+
| benchHydrate1Object () | 80.400μs (±0.00%) | 41.200μs (±0.00%) | 3.190mb | 3.202mb |
| benchExtract1Object () | 9.200μs (±0.00%) | 10.000μs (±0.00%) | 3.190mb | 3.202mb |
| benchHydrate1000Objects () | 3.856ms (±0.00%) | 4.291ms (±0.00%) | 3.414mb | 3.325mb |
| benchExtract1000Objects () | 5.383ms (±0.00%) | 6.277ms (±0.00%) | 3.388mb | 3.313mb |
| benchHydrate1000000Objects () | 2.124s (±0.00%) | 2.490s (±0.00%) | 3.414mb | 3.325mb |
| benchExtract1000000Objects () | 3.896s (±0.00%) | 4.462s (±0.00%) | 2.936mb | 2.851mb |
+-------------------------------+-------------------+-------------------+-----------------+------------+
HydratorBench
=============
+-------------------------------+-------------------+-------------------+-----------------+------------+
| | time (kde mode) | memory |
+-------------------------------+-------------------+-------------------+-----------------+------------+
| subject | Tag: <current> | Tag: base | Tag: <current> | Tag: base |
+-------------------------------+-------------------+-------------------+-----------------+------------+
| benchHydrate1Object () | 75.600μs (±0.00%) | 72.400μs (±0.00%) | 2.695mb | 2.695mb |
| benchExtract1Object () | 3.800μs (±0.00%) | 3.200μs (±0.00%) | 2.695mb | 2.695mb |
| benchHydrate1000Objects () | 2.494ms (±0.00%) | 2.103ms (±0.00%) | 2.787mb | 2.750mb |
| benchExtract1000Objects () | 2.687ms (±0.00%) | 2.113ms (±0.00%) | 2.695mb | 2.695mb |
| benchHydrate1000000Objects () | 1.518s (±0.00%) | 1.130s (±0.00%) | 2.787mb | 2.751mb |
| benchExtract1000000Objects () | 1.683s (±0.00%) | 1.349s (±0.00%) | 2.695mb | 2.695mb |
+-------------------------------+-------------------+-------------------+-----------------+------------+
HydratorWithLazyBench
=====================
+------------------------------------------+--------------------+--------------------+-----------------+------------+
| | time (kde mode) | memory |
+------------------------------------------+--------------------+--------------------+-----------------+------------+
| subject | Tag: <current> | Tag: base | Tag: <current> | Tag: base |
+------------------------------------------+--------------------+--------------------+-----------------+------------+
| benchHydrate1Object () | 0.800μs (±0.00%) | 0.600μs (±0.00%) | 2.695mb | 2.695mb |
| benchHydrate1ObjectTriggerInit () | 112.200μs (±0.00%) | 116.200μs (±0.00%) | 2.762mb | 2.761mb |
| benchHydrate1000Objects () | 425.000μs (±0.00%) | 439.333μs (±0.00%) | 2.695mb | 2.695mb |
| benchHydrate1000ObjectsTriggerInit () | 3.325ms (±0.00%) | 3.039ms (±0.00%) | 2.762mb | 2.761mb |
| benchHydrate1000000Objects () | 291.434ms (±0.00%) | 298.221ms (±0.00%) | 2.695mb | 2.695mb |
| benchHydrate1000000ObjectsTriggerInit () | 2.173s (±0.00%) | 2.013s (±0.00%) | 2.762mb | 2.761mb |
+------------------------------------------+--------------------+--------------------+-----------------+------------+

This comment gets update everytime a new commit comes in!

@DanielBaduraDanielBadura mentioned this pull request Nov 10, 2025
Open
@DavidBadura
DavidBaduraforce-pushed the middleware-hydrator branch 9 times, most recently from 3f32c8f to 9e54004CompareNovember 19, 2025 15:48
@DavidBaduraDavidBadura changed the title Middleware hydratorMiddleware based hydratorNov 19, 2025
@DavidBadura
DavidBaduraforce-pushed the middleware-hydrator branch 2 times, most recently from ade3a43 to 6e02329CompareNovember 19, 2025 15:58
@DavidBaduraDavidBadura added this to the 2.0.0 milestone Nov 19, 2025
@DavidBaduraDavidBadura added BC break enhancement New feature or request labels Nov 19, 2025
@DavidBadura
DavidBaduraforce-pushed the middleware-hydrator branch 3 times, most recently from 4557026 to 5a37a76CompareNovember 21, 2025 12:44
@DavidBadura
DavidBadura marked this pull request as ready for review November 21, 2025 12:45
@DavidBaduraDavidBadura mentioned this pull request Nov 21, 2025
Merged
DanielBadura
DanielBadura requested changes Nov 22, 2025
View reviewed changes

@DanielBaduraDanielBadura left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add a Unit Test for CryptographyMiddleware and more importantly TransformMiddleware? Otherwise LGTM

@DanielBadura

DanielBadura commented Nov 29, 2025

Copy link
Copy Markdown
Member

needs a rebase due to the upmerge of the updates in 1.x branch, sorry about that :D

@DavidBadura
DavidBadura merged commit 58e069f into 2.0.xNov 29, 2025
14 of 16 checks passed
@DavidBadura
DavidBadura deleted the middleware-hydrator branch November 29, 2025 20:08
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DanielBaduraDanielBaduraDanielBadura approved these changes

Assignees

@DavidBaduraDavidBadura

Labels

BC breakenhancementNew feature or request

Projects

None yet

Milestone

2.0.0

Development

Successfully merging this pull request may close these issues.

2 participants

@DavidBadura@DanielBadura