Skip to content

Conversation

serbel324
Copy link
Collaborator

@serbel324 serbel324 commented Aug 9, 2024

  • Wrap ctor args in structs
  • Fix UT-mocks

Changelog entry

Refactor constructors of DSProxy requests

Changelog category

  • Improvement

Additional information

...

Copy link

github-actions bot commented Aug 9, 2024

2024-08-09 17:32:50 UTC Pre-commit check for 9beb42e has started.
2024-08-09 17:35:33 UTC Check linux-x86_64-release-asan is running...
🔴 2024-08-09 19:51:41 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
9940 9893 0 10 24 13

🟢 2024-08-09 19:52:49 UTC Build successful.
🟢 2024-08-09 19:53:19 UTC ydbd size 5.4 GiB changed* by +56.5 KiB, which is < 100.0 KiB vs main: OK

ydbd size dash main: 49e62b4 merge: 9beb42e diff diff %
ydbd size 5 844 143 624 Bytes 5 844 201 504 Bytes +56.5 KiB +0.001%
ydbd stripped size 1 468 161 680 Bytes 1 468 177 488 Bytes +15.4 KiB +0.001%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

Copy link

github-actions bot commented Aug 9, 2024

2024-08-09 17:36:40 UTC Pre-commit check for 9beb42e has started.
2024-08-09 17:39:21 UTC Check linux-x86_64-release-clang14 is running...
🟢 2024-08-09 17:52:01 UTC Build successful.

Copy link

github-actions bot commented Aug 9, 2024

2024-08-09 17:37:17 UTC Pre-commit check for 9beb42e has started.
2024-08-09 17:40:09 UTC Check linux-x86_64-relwithdebinfo is running...
🟡 2024-08-09 19:22:27 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14268 12913 0 2 1338 15

2024-08-09 19:24:17 UTC Failed tests rerun (try 2) linux-x86_64-relwithdebinfo is running...
🟢 2024-08-09 19:32:14 UTC Tests successful.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
20 (only retried tests) 10 0 0 1 9

🟢 2024-08-09 19:32:21 UTC Build successful.
🟡 2024-08-09 19:33:00 UTC ydbd size 8.1 GiB changed* by +392.9 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: ee6b772 merge: 9beb42e diff diff %
ydbd size 8 689 057 208 Bytes 8 689 459 568 Bytes +392.9 KiB +0.005%
ydbd stripped size 472 758 216 Bytes 472 773 640 Bytes +15.1 KiB +0.003%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@serbel324 serbel324 merged commit 4efd471 into ydb-platform:main Aug 12, 2024
VPolka added a commit to VPolka/ydb that referenced this pull request Nov 5, 2024
VPolka added a commit to VPolka/ydb that referenced this pull request Nov 5, 2024
serbel324 added a commit to serbel324/ydb that referenced this pull request Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants