Skip to content

Conversation

@MrLolthe1st
Copy link
Collaborator

No description provided.

@github-actions
Copy link

github-actions bot commented Jan 18, 2024

2024-01-18 16:41:39 UTC Pre-commit check for 3c3146c has started.
2024-01-18 16:41:42 UTC Build linux-x86_64-release-asan is running...
🟢 2024-01-18 16:43:26 UTC Build successful.
2024-01-18 16:43:41 UTC Tests are running...
🔴 2024-01-18 16:57:40 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15901 15838 0 27 25 11

@github-actions
Copy link

github-actions bot commented Jan 18, 2024

2024-01-18 16:41:08 UTC Pre-commit check for 3c3146c has started.
2024-01-18 16:41:10 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-01-18 16:42:49 UTC Build successful.
2024-01-18 16:43:07 UTC Tests are running...
🟢 2024-01-18 17:07:14 UTC Tests successful.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
59958 50727 0 0 9221 10

Copy link
Member

@rvu1024 rvu1024 left a comment

Choose a reason for hiding this comment

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

нужен тест

@MrLolthe1st MrLolthe1st requested a review from rvu1024 January 18, 2024 14:39
#endif

TFailureInjector::Reach("dq_rpc_reader_read_err_when_empty", [&res_] {
res_ = NYT::TErrorOr<NYT::TSharedRef>(NYT::TError("failrule injected"));
Copy link
Member

Choose a reason for hiding this comment

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

failure - misspelling

@MrLolthe1st MrLolthe1st force-pushed the fix-rpc-reader-data-loss branch from 5b2c70d to 3c3146c Compare January 18, 2024 16:40
@MrLolthe1st MrLolthe1st merged commit 58f43ff into ydb-platform:main Jan 18, 2024
@niksaveliev niksaveliev mentioned this pull request Jan 29, 2024
@vitstn vitstn mentioned this pull request Jan 29, 2024
@starlinskiy starlinskiy mentioned this pull request Feb 12, 2024
@vitstn vitstn mentioned this pull request Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants