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

add payload cryptographer for snapshots too #568

Merged
merged 1 commit into from
Apr 9, 2024

Conversation

DavidBadura
Copy link
Member

No description provided.

@DavidBadura DavidBadura added the enhancement New feature or request label Apr 9, 2024
@DavidBadura DavidBadura added this to the 3.0.0 milestone Apr 9, 2024
@DavidBadura DavidBadura requested a review from DanielBadura April 9, 2024 11:35
Copy link

github-actions bot commented Apr 9, 2024

Hello 👋

here is the most recent benchmark result:

SimpleSetupBench
================

+----------------------------------------+--------------------+--------------------+-----------+-----------------+------------+-------------+
|                                        | time (kde mode)                                     | memory                                     |
+----------------------------------------+--------------------+--------------------+-----------+-----------------+------------+-------------+
| subject                                | Tag: <current>     | Tag: base          | time-diff | Tag: <current>  | Tag: base  | memory-diff |
+----------------------------------------+--------------------+--------------------+-----------+-----------------+------------+-------------+
| benchLoad1Event ()                     | 1.314ms (±0.00%)   | 950.000μs (±0.00%) | +38.34%   | 34.265mb        | 34.202mb   | +0.18%      |
| benchLoad10000Events ()                | 54.075ms (±0.00%)  | 54.903ms (±0.00%)  | -1.51%    | 34.265mb        | 34.202mb   | +0.18%      |
| benchSave1Event ()                     | 1.118ms (±0.00%)   | 1.050ms (±0.00%)   | +6.41%    | 33.763mb        | 33.693mb   | +0.21%      |
| benchSave10000Events ()                | 231.285ms (±0.00%) | 235.510ms (±0.00%) | -1.79%    | 33.763mb        | 33.693mb   | +0.21%      |
| benchSave10000Aggregates ()            | 8.854s (±0.00%)    | 8.732s (±0.00%)    | +1.40%    | 33.763mb        | 33.693mb   | +0.21%      |
| benchSave10000AggregatesTransaction () | 6.884s (±0.00%)    | 6.860s (±0.00%)    | +0.35%    | 33.763mb        | 33.693mb   | +0.21%      |
+----------------------------------------+--------------------+--------------------+-----------+-----------------+------------+-------------+

SnapshotsBench
==============

+----------------------------------------+--------------------+--------------------+-----------+-----------------+------------+-------------+
|                                        | time (kde mode)                                     | memory                                     |
+----------------------------------------+--------------------+--------------------+-----------+-----------------+------------+-------------+
| subject                                | Tag: <current>     | Tag: base          | time-diff | Tag: <current>  | Tag: base  | memory-diff |
+----------------------------------------+--------------------+--------------------+-----------+-----------------+------------+-------------+
| benchLoad10000EventsMissingSnapshot () | 54.707ms (±0.00%)  | 56.348ms (±0.00%)  | -2.91%    | 33.764mb        | 33.763mb   | +0.00%      |
| benchLoad10000Events ()                | 893.200μs (±0.00%) | 847.300μs (±0.00%) | +5.42%    | 33.764mb        | 33.763mb   | +0.00%      |
+----------------------------------------+--------------------+--------------------+-----------+-----------------+------------+-------------+

SplitStreamBench
================

+-------------------------+--------------------+--------------------+-----------+-----------------+------------+-------------+
|                         | time (kde mode)                                     | memory                                     |
+-------------------------+--------------------+--------------------+-----------+-----------------+------------+-------------+
| subject                 | Tag: <current>     | Tag: base          | time-diff | Tag: <current>  | Tag: base  | memory-diff |
+-------------------------+--------------------+--------------------+-----------+-----------------+------------+-------------+
| benchLoad10000Events () | 5.074ms (±0.00%)   | 5.114ms (±0.00%)   | -0.80%    | 37.071mb        | 37.070mb   | +0.00%      |
| benchSave10000Events () | 394.212ms (±0.00%) | 396.557ms (±0.00%) | -0.59%    | 37.072mb        | 37.071mb   | +0.00%      |
+-------------------------+--------------------+--------------------+-----------+-----------------+------------+-------------+

SubscriptionEngineBench
=======================

+---------------------------+-----------------+-----------------+-----------+-----------------+------------+-------------+
|                           | time (kde mode)                               | memory                                     |
+---------------------------+-----------------+-----------------+-----------+-----------------+------------+-------------+
| subject                   | Tag: <current>  | Tag: base       | time-diff | Tag: <current>  | Tag: base  | memory-diff |
+---------------------------+-----------------+-----------------+-----------+-----------------+------------+-------------+
| benchHandle10000Events () | 3.123s (±0.00%) | 3.131s (±0.00%) | -0.26%    | 34.162mb        | 34.161mb   | +0.00%      |
+---------------------------+-----------------+-----------------+-----------+-----------------+------------+-------------+

PersonalDataBench
=================

+----------------------------------------+--------------------+--------------------+-----------+-----------------+------------+-------------+
|                                        | time (kde mode)                                     | memory                                     |
+----------------------------------------+--------------------+--------------------+-----------+-----------------+------------+-------------+
| subject                                | Tag: <current>     | Tag: base          | time-diff | Tag: <current>  | Tag: base  | memory-diff |
+----------------------------------------+--------------------+--------------------+-----------+-----------------+------------+-------------+
| benchLoad1Event ()                     | 832.500μs (±0.00%) | 835.200μs (±0.00%) | -0.32%    | 35.780mb        | 35.779mb   | +0.00%      |
| benchLoad10000Events ()                | 88.298ms (±0.00%)  | 90.112ms (±0.00%)  | -2.01%    | 35.780mb        | 35.779mb   | +0.00%      |
| benchSave1Event ()                     | 1.544ms (±0.00%)   | 1.572ms (±0.00%)   | -1.79%    | 35.780mb        | 35.779mb   | +0.00%      |
| benchSave10000Events ()                | 269.980ms (±0.00%) | 260.615ms (±0.00%) | +3.59%    | 35.782mb        | 35.781mb   | +0.00%      |
| benchSave10000Aggregates ()            | 13.064s (±0.00%)   | 13.161s (±0.00%)   | -0.74%    | 35.780mb        | 35.779mb   | +0.00%      |
| benchSave10000AggregatesTransaction () | 10.944s (±0.00%)   | 10.880s (±0.00%)   | +0.59%    | 36.282mb        | 36.280mb   | +0.00%      |
+----------------------------------------+--------------------+--------------------+-----------+-----------------+------------+-------------+

This comment gets update everytime a new commit comes in!

@DavidBadura DavidBadura force-pushed the add-snapshot-cryptographer branch from 94002a7 to 1a85c82 Compare April 9, 2024 12:09
@DavidBadura DavidBadura force-pushed the add-snapshot-cryptographer branch from 1a85c82 to 257d2b9 Compare April 9, 2024 12:23
@DavidBadura DavidBadura merged commit 6279164 into 3.0.x Apr 9, 2024
35 of 36 checks passed
@DavidBadura DavidBadura deleted the add-snapshot-cryptographer branch April 9, 2024 23:39
DavidBadura added a commit that referenced this pull request Apr 11, 2024
add payload cryptographer for snapshots too
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants