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

Fix reading without consumer from federation #9136

Conversation

niksaveliev
Copy link
Collaborator

@niksaveliev niksaveliev commented Sep 12, 2024

Changelog entry

Changelog category

  • Not for changelog (changelog entry is not required)

Additional information

...

Copy link

github-actions bot commented Sep 12, 2024

2024-09-12 10:14:45 UTC Pre-commit check linux-x86_64-relwithdebinfo for 7d4cbdc has started.
2024-09-12 10:15:09 UTC Artifacts will be uploaded here
2024-09-12 10:17:56 UTC Check cancelled

Copy link

github-actions bot commented Sep 12, 2024

2024-09-12 10:17:23 UTC Pre-commit check linux-x86_64-release-asan for 7d4cbdc has started.
2024-09-12 10:17:55 UTC Check cancelled

Copy link

github-actions bot commented Sep 12, 2024

2024-09-12 10:17:33 UTC Pre-commit check linux-x86_64-release-clang14 for 7d4cbdc has started.
2024-09-12 10:17:47 UTC Check cancelled

@niksaveliev niksaveliev marked this pull request as ready for review September 12, 2024 10:18
@niksaveliev niksaveliev requested a review from a team as a code owner September 12, 2024 10:18
Copy link

github-actions bot commented Sep 12, 2024

2024-09-12 10:18:34 UTC Pre-commit check linux-x86_64-relwithdebinfo for 9e57a5f has started.
2024-09-12 10:18:38 UTC Artifacts will be uploaded here
2024-09-12 10:21:25 UTC ya make is running...
🟡 2024-09-12 11:18:11 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14762 13113 0 6 1377 266

2024-09-12 11:19:32 UTC ya make is running... (failed tests rerun, try 2)
🟢 2024-09-12 11:29:48 UTC Tests successful.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
288 (only retried tests) 192 0 0 8 88

🟢 2024-09-12 11:29:56 UTC Build successful.
🟢 2024-09-12 11:30:37 UTC ydbd size 8.4 GiB changed* by -16 Bytes, which is <= 0 Bytes vs main: OK

ydbd size dash main: 4d137ad merge: 9e57a5f diff diff %
ydbd size 9 035 109 760 Bytes 9 035 109 744 Bytes -16 Bytes -0.000%
ydbd stripped size 488 206 024 Bytes 488 206 024 Bytes 0 Bytes 0.000%

*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 Sep 12, 2024

2024-09-12 10:19:18 UTC Pre-commit check linux-x86_64-release-clang14 for 9e57a5f has started.
2024-09-12 10:19:21 UTC Artifacts will be uploaded here
2024-09-12 10:22:06 UTC ya make is running...
🟢 2024-09-12 10:27:46 UTC Build successful.

Copy link

github-actions bot commented Sep 12, 2024

2024-09-12 10:21:12 UTC Pre-commit check linux-x86_64-release-asan for 9e57a5f has started.
2024-09-12 10:21:22 UTC Artifacts will be uploaded here
2024-09-12 10:24:00 UTC ya make is running...
🔴 2024-09-12 11:52:03 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
10309 10168 0 11 33 97

🟢 2024-09-12 11:53:00 UTC Build successful.
🟢 2024-09-12 11:53:34 UTC ydbd size 5.6 GiB changed* by -16 Bytes, which is <= 0 Bytes vs main: OK

ydbd size dash main: 4d137ad merge: 9e57a5f diff diff %
ydbd size 6 044 978 568 Bytes 6 044 978 552 Bytes -16 Bytes -0.000%
ydbd stripped size 1 512 804 336 Bytes 1 512 804 336 Bytes 0 Bytes 0.000%

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

@niksaveliev niksaveliev merged commit e8c9cca into ydb-platform:main Sep 12, 2024
10 of 12 checks passed
@shnikd shnikd mentioned this pull request Sep 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