Skip to content

add ut for incorrect pragma parsing and return error #7153

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

Merged
merged 1 commit into from
Jul 26, 2024

Conversation

gridnevvvit
Copy link
Member

Changelog entry

add ut for incorrect pragma parsing and return error

Changelog category

  • Not for changelog (changelog entry is not required)

Additional information

...

@gridnevvvit gridnevvvit requested a review from a team as a code owner July 26, 2024 16:06
Copy link

github-actions bot commented Jul 26, 2024

2024-07-26 16:08:34 UTC Pre-commit check for c48760e has started.
2024-07-26 16:11:29 UTC Check linux-x86_64-release-clang14 is running...
🟢 2024-07-26 16:20:34 UTC Build successful.

Copy link

github-actions bot commented Jul 26, 2024

2024-07-26 16:10:36 UTC Pre-commit check for c48760e has started.
2024-07-26 16:13:33 UTC Check linux-x86_64-relwithdebinfo is running...
🟡 2024-07-26 18:26:01 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
71679 58118 0 2 13547 12

2024-07-26 18:33:43 UTC Failed tests rerun (try 2) linux-x86_64-relwithdebinfo is running...
🟡 2024-07-26 18:41:41 UTC Some tests failed, follow the links below. Going to retry failed tests...

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
13 (only retried tests) 3 0 2 0 8

2024-07-26 18:48:36 UTC Failed tests rerun (try 3) linux-x86_64-relwithdebinfo is running...
🔴 2024-07-26 18:56:18 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
11 (only retried tests) 1 0 2 0 8

🟢 2024-07-26 19:02:51 UTC Build successful.
🟢 2024-07-26 19:03:28 UTC ydbd size 8.1 GiB changed* by -1.1 KiB, which is <= 0 Bytes vs main: OK

ydbd size dash main: 9785f6a merge: c48760e diff diff %
ydbd size 8 668 833 544 Bytes 8 668 832 448 Bytes -1.1 KiB -0.000%
ydbd stripped size 471 575 616 Bytes 471 575 552 Bytes -64 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 Jul 26, 2024

2024-07-26 16:10:38 UTC Pre-commit check for c48760e has started.
2024-07-26 16:13:37 UTC Check linux-x86_64-release-asan is running...
🔴 2024-07-26 18:33:41 UTC Some tests failed, follow the links below.

Test history | Ya make output

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
10328 9728 0 87 339 174

🟢 2024-07-26 18:34:45 UTC Build successful.
🟢 2024-07-26 18:35:41 UTC ydbd size 5.2 GiB changed* by -1008 Bytes, which is <= 0 Bytes vs main: OK

ydbd size dash main: 9785f6a merge: c48760e diff diff %
ydbd size 5 565 674 232 Bytes 5 565 673 224 Bytes -1008 Bytes -0.000%
ydbd stripped size 1 198 426 800 Bytes 1 198 426 480 Bytes -320 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

@gridnevvvit gridnevvvit enabled auto-merge (squash) July 26, 2024 18:28
@gridnevvvit gridnevvvit merged commit 90e960b into ydb-platform:main Jul 26, 2024
9 of 13 checks passed
@gridnevvvit gridnevvvit deleted the spilling-nodes-support branch August 30, 2024 13:48
lll-phill-lll pushed a commit to lll-phill-lll/ydb that referenced this pull request Sep 9, 2024
lll-phill-lll pushed a commit to lll-phill-lll/ydb that referenced this pull request Sep 10, 2024
lll-phill-lll pushed a commit to lll-phill-lll/ydb that referenced this pull request Sep 11, 2024
lll-phill-lll pushed a commit to lll-phill-lll/ydb that referenced this pull request Sep 13, 2024
lll-phill-lll pushed a commit to lll-phill-lll/ydb that referenced this pull request Sep 13, 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